Fix installation object parsing logic

Please provide a description of this PR:
The original parsing logic may not be very robust,
for example:
apiVersion: v1
kind: ConfigMap
metadata:
  name: myconfigmap
  namespace: default
data:
  mydata: |-
    First line of data
    Second line of data                # with some spaces

# some random comments
 # some random comments
apiVersion: v1
kind: ConfigMap
metadata:
  name: myconfigmap
  namespace: default # some random comments should be deleted
data:
  mydata: |-
    First line of data # some random comments should not be deleted
    Second line of data

are not handled correctly.
This PR is to make the parsing logic more robust to handle objects parsing correctly.

@hanxiaop: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

gencheck_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

gencheck_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test gencheck

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

gencheck_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test gencheck

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

unit-tests_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

gencheck_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test gencheck

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

unit-tests_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests

integ-operator-controller_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-operator-controller

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

gencheck_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test gencheck

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

unit-tests_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests

integ-operator-controller_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-operator-controller

integ-ds_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ds

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

gencheck_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test gencheck

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

unit-tests_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests

integ-operator-controller_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-operator-controller

integ-ds_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ds

integ-ipv6_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ipv6

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

gencheck_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test gencheck

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

unit-tests_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests

integ-operator-controller_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-operator-controller

integ-ds_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ds

integ-ipv6_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ipv6

integ-telemetry-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-istiodremote

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

gencheck_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test gencheck

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

unit-tests_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests

integ-operator-controller_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-operator-controller

integ-ds_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ds

integ-ipv6_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ipv6

integ-telemetry-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-istiodremote

integ-pilot-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

unit-tests-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

gencheck_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test gencheck

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

unit-tests_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests

integ-operator-controller_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-operator-controller

integ-ds_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ds

integ-ipv6_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ipv6

integ-telemetry-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-istiodremote

integ-pilot-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-multicluster

integ-pilot_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

gencheck_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test gencheck

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

unit-tests_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests

integ-operator-controller_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-operator-controller

integ-ds_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ds

integ-ipv6_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ipv6

integ-telemetry-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-istiodremote

integ-pilot-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-multicluster

integ-pilot_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot

unit-tests-arm64_istio
2042627744119ba700a6840b82ebcfe7ab6673cb
link
true
/test unit-tests-arm64

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-basic-arm64_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-basic-arm64

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

unit-tests_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test unit-tests

integ-operator-controller_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-operator-controller

integ-ds_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ds

integ-ipv6_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ipv6

integ-telemetry-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-istiodremote

integ-pilot-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-multicluster

integ-pilot_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot

unit-tests-arm64_istio
2042627744119ba700a6840b82ebcfe7ab6673cb
link
true
/test unit-tests-arm64

gencheck_istio
2042627744119ba700a6840b82ebcfe7ab6673cb
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hanxiaop: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-distroless_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-distroless

integ-ambient_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ambient

integ-security_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security

integ-cni_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-cni

integ-security-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-multicluster

integ-pilot-istiodremote-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote-mc

integ-telemetry_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry

integ-telemetry-mc_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-mc

integ-pilot-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-istiodremote

integ-security-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-security-istiodremote

integ-operator-controller_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-operator-controller

integ-ds_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ds

integ-ipv6_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-ipv6

integ-telemetry-istiodremote_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-telemetry-istiodremote

integ-pilot-multicluster_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot-multicluster

integ-pilot_istio
bfd23c1d93dda1a762e38dfb98f93129e55c1fed
link
true
/test integ-pilot

unit-tests-arm64_istio
2042627744119ba700a6840b82ebcfe7ab6673cb
link
true
/test unit-tests-arm64

gencheck_istio
2042627744119ba700a6840b82ebcfe7ab6673cb
link
true
/test gencheck

unit-tests_istio
2042627744119ba700a6840b82ebcfe7ab6673cb
link
true
/test unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

So before:

We pass raw YAML on, unless we overlay it in which case we re-format it

Now:

We always reformat

is that accurate?

@howardjohn Yes this is correct, overlay will do the same kind of unmarshal and marshal. Is there any concerns?
I think the reformat/overlay parts are all manipulating the yaml field of the object, and the actual apply logic is using the unstructured object directly, so the order won't be a issue.
// K8sObject is an in-memory representation of a k8s object, used for moving between different representations
// (Unstructured, JSON, YAML) with cached rendering.
type K8sObject struct {
	object *unstructured.Unstructured

	Group     string
	Kind      string
	Name      string
	Namespace string

	json []byte
	yaml []byte
}

I'm not sure if there are historical reasons that we had the cleaning raw YAML logic, but I think it's a bug which may potentially cause parse errors.

I actually think the parse/overlay logic is too complicated with extra operations, tpath, etc...

Kindly ping @howardjohn


WG5 association event (from Linked Pasts 6)

Working group: Proposal to add an Associated Event property to the SNAP disambiguator class
Members: Gabriel Bodard, Tom Gheldof, Richard Light, Rada Varga
Proposal
A new snap:associatedEvent will be a sub-property of snap:disambiguator (and therefore sibling of snap:associatedPlace, snap:associatedDate and snap:occupation).
The object of snap:associatedEvent property is a URI, defining or describing the event in some form. Examples given in the Cookbook could include Wikipedia/Wikidata pages and CIDOC-CRM RDF.
The SNAP Cookbook will have no further opinion as to the structure, format or complexity of the construct pointed to by this property, although consistency is to be encouraged in the long run, of course.

This proposal is currently included in the SNAP Cookbook 2022 at https://github.com/SNAP-DRGN/Cookbook/wiki/RDF-8.4-Event
Discussion, correction and improvement is welcome.


Update Developer Guides about DPNP integration

Status: contains all text.
To fix:

[x] Links
[x] Code blocks
[ ] Add API references

Added a lot of garbage by mistake. Will clean it.


Heap buffer overflow when loading some SVG image

Tested versions
master[e5b4ef8e9522e950033cbece39a31a4a76da19c1]
System information
Arch Linux
Issue description
Open AssetLib in the editor, and address sanitizer decides to crash the editor.
ASan Output
==4963==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x5190024df0fd at pc 0x75ecfa4818df bp 0x75ecc75ff540 sp 0x75ecc75fece8
READ of size 1002 at 0x5190024df0fd thread T23
    #0 0x75ecfa4818de in __interceptor_strlen /usr/src/debug/gcc/gcc/libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc:461
    #1 0x55d2960687ca in tvg::strDuplicate(char const*, unsigned long) thirdparty/thorvg/src/common/tvgStr.cpp:219
    #2 0x55d2960ab2d8 in simpleXmlParseCSSAttribute(char const*, unsigned int, char**, char**, char const**, unsigned int*) thirdparty/thorvg/src/loaders/svg/tvgXmlParser.cpp:564
    #3 0x55d296087edc in _svgLoaderParserXmlCssStyle thirdparty/thorvg/src/loaders/svg/tvgSvgLoader.cpp:3164
    #4 0x55d29608860e in _svgLoaderParser thirdparty/thorvg/src/loaders/svg/tvgSvgLoader.cpp:3208
    #5 0x55d2960aa633 in simpleXmlParse(char const*, unsigned int, bool, bool (*)(void*, SimpleXMLType, char const*, unsigned int), void const*) thirdparty/thorvg/src/loaders/svg/tvgXmlParser.cpp:454
    #6 0x55d29608ac01 in SvgLoader::run(unsigned int) thirdparty/thorvg/src/loaders/svg/tvgSvgLoader.cpp:3557
    #7 0x55d296114394 in tvg::Task::operator()(unsigned int) thirdparty/thorvg/src/renderer/tvgTaskScheduler.h:64
    #8 0x55d296115631 in tvg::TaskSchedulerImpl::run(unsigned int) thirdparty/thorvg/src/renderer/tvgTaskScheduler.cpp:153
    #9 0x55d296114e66 in tvg::TaskSchedulerImpl::TaskSchedulerImpl(unsigned int)::{lambda()#1}::operator()() const thirdparty/thorvg/src/renderer/tvgTaskScheduler.cpp:120
    #10 0x55d296116a57 in void std::__invoke_impl<void, tvg::TaskSchedulerImpl::TaskSchedulerImpl(unsigned int)::{lambda()#1}>(std::__invoke_other, tvg::TaskSchedulerImpl::TaskSchedulerImpl(unsigned int)::{lambda()#1}&&) /usr/include/c++/13.2.1/bits/invoke.h:61
    #11 0x55d296116a1a in std::__invoke_result<tvg::TaskSchedulerImpl::TaskSchedulerImpl(unsigned int)::{lambda()#1}>::type std::__invoke<tvg::TaskSchedulerImpl::TaskSchedulerImpl(unsigned int)::{lambda()#1}>(tvg::TaskSchedulerImpl::TaskSchedulerImpl(unsigned int)::{lambda()#1}&&) /usr/include/c++/13.2.1/bits/invoke.h:96
    #12 0x55d2961169c7 in void std::thread::_Invoker<std::tuple<tvg::TaskSchedulerImpl::TaskSchedulerImpl(unsigned int)::{lambda()#1}> >::_M_invoke<0ul>(std::_Index_tuple<0ul>) /usr/include/c++/13.2.1/bits/std_thread.h:292
    #13 0x55d29611699b in std::thread::_Invoker<std::tuple<tvg::TaskSchedulerImpl::TaskSchedulerImpl(unsigned int)::{lambda()#1}> >::operator()() /usr/include/c++/13.2.1/bits/std_thread.h:299
    #14 0x55d29611697f in std::thread::_State_impl<std::thread::_Invoker<std::tuple<tvg::TaskSchedulerImpl::TaskSchedulerImpl(unsigned int)::{lambda()#1}> > >::_M_run() /usr/include/c++/13.2.1/bits/std_thread.h:244
    #15 0x55d2a023f0b2 in execute_native_thread_routine (/home/timothy/repos/godot-master/bin/godot.linuxbsd.editor.dev.x86_64.san+0x1524d0b2) (BuildId: 00782fbb3b377b7ae3d6d6a3814246024b24a92c)
    #16 0x75ecfa1bd559  (/usr/lib/libc.so.6+0x8b559) (BuildId: c0caa0b7709d3369ee575fcd7d7d0b0fc48733af)
    #17 0x75ecfa23aa3b  (/usr/lib/libc.so.6+0x108a3b) (BuildId: c0caa0b7709d3369ee575fcd7d7d0b0fc48733af)

0x5190024df0fd is located 0 bytes after 1149-byte region [0x5190024dec80,0x5190024df0fd)
allocated by thread T0 here:
    #0 0x75ecfa4e1359 in __interceptor_malloc /usr/src/debug/gcc/gcc/libsanitizer/asan/asan_malloc_linux.cpp:69
    #1 0x55d29608cda1 in SvgLoader::open(char const*, unsigned int, bool) thirdparty/thorvg/src/loaders/svg/tvgSvgLoader.cpp:3675
    #2 0x55d2960f995a in LoaderMgr::loader(char const*, unsigned int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, bool) thirdparty/thorvg/src/renderer/tvgLoader.cpp:364
    #3 0x55d29610c2d0 in tvg::Picture::Impl::load(char const*, unsigned int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, bool) thirdparty/thorvg/src/renderer/tvgPicture.h:163
    #4 0x55d29610b551 in tvg::Picture::load(char const*, unsigned int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&, bool) thirdparty/thorvg/src/renderer/tvgPicture.cpp:183
    #5 0x55d29616c58b in ImageLoaderSVG::create_image_from_utf8_buffer(Ref<Image>, unsigned char const*, int, float, bool) modules/svg/image_loader_svg.cpp:85
    #6 0x55d29616c158 in ImageLoaderSVG::load_mem_svg(unsigned char const*, int, float) modules/svg/image_loader_svg.cpp:74
    #7 0x55d2983e7c57 in EditorAssetLibrary::_image_update(bool, bool, Vector<unsigned char> const&, int) editor/plugins/asset_library_editor_plugin.cpp:852
    #8 0x55d2983ebcc3 in EditorAssetLibrary::_request_image(ObjectID, int, String, EditorAssetLibrary::ImageType, int) editor/plugins/asset_library_editor_plugin.cpp:1025
    #9 0x55d2983f51ed in EditorAssetLibrary::_http_request_completed(int, int, Vector<String> const&, Vector<unsigned char> const&) editor/plugins/asset_library_editor_plugin.cpp:1399
    #10 0x55d29841ce83 in void call_with_variant_args_helper<EditorAssetLibrary, int, int, Vector<String> const&, Vector<unsigned char> const&, 0ul, 1ul, 2ul, 3ul>(EditorAssetLibrary*, void (EditorAssetLibrary::*)(int, int, Vector<String> const&, Vector<unsigned char> const&), Variant const**, Callable::CallError&, IndexSequence<0ul, 1ul, 2ul, 3ul>) core/variant/binder_common.h:304
    #11 0x55d29841aa02 in void call_with_variant_args<EditorAssetLibrary, int, int, Vector<String> const&, Vector<unsigned char> const&>(EditorAssetLibrary*, void (EditorAssetLibrary::*)(int, int, Vector<String> const&, Vector<unsigned char> const&), Variant const**, int, Callable::CallError&) core/variant/binder_common.h:418
    #12 0x55d298410d19 in CallableCustomMethodPointer<EditorAssetLibrary, int, int, Vector<String> const&, Vector<unsigned char> const&>::call(Variant const**, int, Variant&, Callable::CallError&) const core/object/callable_method_pointer.h:103
    #13 0x55d29ee1cf78 in Callable::callp(Variant const**, int, Variant&, Callable::CallError&) const core/variant/callable.cpp:57
    #14 0x55d29f65d745 in Object::emit_signalp(StringName const&, Variant const**, int) core/object/object.cpp:1201
    #15 0x55d2995d9558 in Node::emit_signalp(StringName const&, Variant const**, int) scene/main/node.cpp:3904
    #16 0x55d29951c169 in Error Object::emit_signal<int, int, Vector<String>, Vector<unsigned char> >(StringName const&, int, int, Vector<String>, Vector<unsigned char>) core/object/object.h:933
    #17 0x55d299512cbe in HTTPRequest::_request_done(int, int, Vector<String> const&, Vector<unsigned char> const&) scene/main/http_request.cpp:483
    #18 0x55d29952f6b8 in void call_with_variant_args_helper<HTTPRequest, int, int, Vector<String> const&, Vector<unsigned char> const&, 0ul, 1ul, 2ul, 3ul>(HTTPRequest*, void (HTTPRequest::*)(int, int, Vector<String> const&, Vector<unsigned char> const&), Variant const**, Callable::CallError&, IndexSequence<0ul, 1ul, 2ul, 3ul>) core/variant/binder_common.h:304
    #19 0x55d29952b5d4 in void call_with_variant_args<HTTPRequest, int, int, Vector<String> const&, Vector<unsigned char> const&>(HTTPRequest*, void (HTTPRequest::*)(int, int, Vector<String> const&, Vector<unsigned char> const&), Variant const**, int, Callable::CallError&) core/variant/binder_common.h:418
    #20 0x55d2995278f1 in CallableCustomMethodPointer<HTTPRequest, int, int, Vector<String> const&, Vector<unsigned char> const&>::call(Variant const**, int, Variant&, Callable::CallError&) const core/object/callable_method_pointer.h:103
    #21 0x55d29ee1cf78 in Callable::callp(Variant const**, int, Variant&, Callable::CallError&) const core/variant/callable.cpp:57
    #22 0x55d29f644b09 in CallQueue::_call_function(Callable const&, Variant const*, int, bool) core/object/message_queue.cpp:221
    #23 0x55d29f645e9d in CallQueue::flush() core/object/message_queue.cpp:326
    #24 0x55d2996698f5 in SceneTree::process(double) scene/main/scene_tree.cpp:530
    #25 0x55d293696cf9 in Main::iteration() main/main.cpp:4008
    #26 0x55d2934aa66d in OS_LinuxBSD::run() platform/linuxbsd/os_linuxbsd.cpp:938
    #27 0x55d293498876 in main platform/linuxbsd/godot_linuxbsd.cpp:85
    #28 0x75ecfa157ccf  (/usr/lib/libc.so.6+0x25ccf) (BuildId: c0caa0b7709d3369ee575fcd7d7d0b0fc48733af)

Thread T23 created by T0 here:
    #0 0x75ecfa44a497 in __interceptor_pthread_create /usr/src/debug/gcc/gcc/libsanitizer/asan/asan_interceptors.cpp:208
    #1 0x55d2a023f199 in std::thread::_M_start_thread(std::unique_ptr<std::thread::_State, std::default_delete<std::thread::_State> >, void (*)()) (/home/timothy/repos/godot-master/bin/godot.linuxbsd.editor.dev.x86_64.san+0x1524d199) (BuildId: 00782fbb3b377b7ae3d6d6a3814246024b24a92c)
    #2 0x55d2961150a2 in tvg::TaskSchedulerImpl::TaskSchedulerImpl(unsigned int) thirdparty/thorvg/src/renderer/tvgTaskScheduler.cpp:120
    #3 0x55d29611414f in tvg::TaskScheduler::init(unsigned int) thirdparty/thorvg/src/renderer/tvgTaskScheduler.cpp:201
    #4 0x55d2960f8083 in tvg::Initializer::init(tvg::CanvasEngine, unsigned int) thirdparty/thorvg/src/renderer/tvgInitializer.cpp:129
    #5 0x55d296171990 in initialize_svg_module(ModuleInitializationLevel) modules/svg/register_types.cpp:52
    #6 0x55d2936e2da1 in initialize_modules(ModuleInitializationLevel) modules/register_module_types.gen.cpp:166
    #7 0x55d293686915 in Main::setup2() main/main.cpp:3035
    #8 0x55d29367c1ad in Main::setup(char const*, int, char**, bool) main/main.cpp:2404
    #9 0x55d29349881c in main platform/linuxbsd/godot_linuxbsd.cpp:74
    #10 0x75ecfa157ccf  (/usr/lib/libc.so.6+0x25ccf) (BuildId: c0caa0b7709d3369ee575fcd7d7d0b0fc48733af)

SUMMARY: AddressSanitizer: heap-buffer-overflow /usr/src/debug/gcc/gcc/libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc:461 in __interceptor_strlen
Shadow bytes around the buggy address:
  0x5190024dee00: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x5190024dee80: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x5190024def00: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x5190024def80: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x5190024df000: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
=>0x5190024df080: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00[05]
  0x5190024df100: fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa fa
  0x5190024df180: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x5190024df200: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x5190024df280: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
  0x5190024df300: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00
Shadow byte legend (one shadow byte represents 8 application bytes):
  Addressable:           00
  Partially addressable: 01 02 03 04 05 06 07
  Heap left redzone:       fa
  Freed heap region:       fd
  Stack left redzone:      f1
  Stack mid redzone:       f2
  Stack right redzone:     f3
  Stack after return:      f5
  Stack use after scope:   f8
  Global redzone:          f9
  Global init order:       f6
  Poisoned by user:        f7
  Container overflow:      fc
  Array cookie:            ac
  Intra object redzone:    bb
  ASan internal:           fe
  Left alloca redzone:     ca
  Right alloca redzone:    cb
==4963==ABORTING

Steps to reproduce
Make sure your editor build has Address Sanitizer enabled.
Open AssetLib tab inside the editor, search for "Remote Transform" and wait.
Or extract and open the MRP with the editor.
Minimal reproduction project (MRP)
svg_heap_buffer_overflow.zip

Do you know if there's more information about the svg I can pass to the thorvg folks @hermet

Narrowed down to the icon of this plugin.
Extract svg_heap_buffer_overflow.zip. Open the project in a sanitizer build crashes with heap-buffer-overflow during the asset import process.

@fire thorvg svg maintainers: @JSUYA @mgrudzinska  We will check our side as well, thanks.

Hi
I tested the attached svg images(icon.svg, RemoteTransformControl.svg) with version 0.12.10 (e69d63688b) of thorvg, but was unable to reproduce the problem.
Recently, various issues in the svg loader module have been fixed. Can you check whether the same problem occurs in the latest version of thorvg?

@JSUYA Hi
In the parser code, I find thorvg uses lots of string functions like strlen and strchr.
However, SvgLoader::open() accepts a buffer pointer & size, it's not necessarily null terminated.
https://github.com/godotengine/godot/blob/029aadef563fb69cf49aa9795b62f27171f8c3f4/thirdparty/thorvg/src/loaders/svg/tvgSvgLoader.cpp#L3697-L3711
Making the function allocate size + 1 and put null at content[size] fixes the problem :tada:

Hello, @JSUYA has handled the issue. The fix will be applied in thorvg v0.13.1. thanks.


Local vendor repository

To allow a project to e.g. commit dependencies to git.
Removes negative effects of when the repo of a dependency is removed.
Contrary to the .opa/dependencies directory, the vendor directory should contain the full set of non-modified dependencies.

Consider creating a odm vendor command (possibly with sub-commands) for creating and updating the vendor directory.
Alternatively/complementary, make this configurable in opa.project.


hubble/peer: drop irrelevant change notifications

This commit optimizes the node update handler in two ways:

Change notifications from the node manager that do not change the
name or address are now ignored as they are irrelevant to the peer
service.
If the name is changed, a DELETE notification with the old name is
sent followed by an ADD notification with the new name.

Note that if the name has not changed but the address has, an UPDATE
notification is sent. This behavior is unchanged.
This set of changes will should reduce the amount of unnecessary update
notifications that result in overhead for the main consumer of the peer
service (Hubble Relay). Also, this way of handling a name change is
easier to process.

test-me-please

Coverage increased (+0.03%) to 36.946% when pulling 6801f8424ac6ab5de89fb787302f99d6dfefdccb on pr/rolinh/hubble-peer-service-fix into ea8b727d7b663c4482f26348cbf613c29811879f on master.


For question database, math symbols should be displayed correctly.

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

No changes in code, just need to populate the database with the correct character. E.g. for super script, copy the supercript from here and paste. https://symbl.cc/en/2075/


Ian - Setting the package status

Ian is a customer who would like to set a specific status of his package to Fragile so the delivery driver knows to be careful with it.

Acceptance Criteria:

Fragile status can be set and read

Ian is able to specify the status of his package to fragile through the ordering process on the website, which will then notify the driver of the status of the package.

Reviewed User Story:
As a customer I want to be able to set the status of my package to Fragile so the delivery drivers knows to handle the package with care.


Full refactor to make testing in future easier and more robust

Changelog

Full refactor
Add branch destroyer

issue fix: https://github.com/Spazzy757/paul/issues/18

/cat


Compliance: ITSG-22 Network Security Zones and ITSG-38 Network Security Zoning - Design Considerations for Placement of Services

We have been focused on ITSG-33 security controls until 202208 - we need to verify our compliance with Network Zoning   and ZIPs (Zone Interface Point)
ITSG-22 https://cyber.gc.ca/en/guidance/baseline-security-requirements-network-security-zones-version-20-itsp80022 and https://cyber.gc.ca/sites/default/files/cyber/publications/itsp80022-e.pdf
ITSG-38 Placement of Services with Zones - https://open.canada.ca/data/en/dataset/7ef76a62-bb53-4e9c-b2a4-03e5c53570a1 and
https://cyber.gc.ca/en/guidance/network-security-zoning-design-considerations-placement-services-within-zones-itsg-38
SSC 2020 https://wiki.gccollab.ca/images/9/9d/Network_Security_Zoning_Reference_Architecture.pdf
Flows, firewall demarcation, encryption in transit levels l3/l4 (default internal) + l7
Expand on https://cloud.google.com/architecture/landing-zones/decide-network-design#option-2 in https://cloud.google.com/architecture/landing-zones#what-is-a-google-cloud-landing-zone
Google Front End Service (reverse proxy) - https://cloud.google.com/docs/security/infrastructure/design#google_front_end_service
20220920
Network separation question:
We have a couple options to start -
1 - shared VPC (host) - with service subnets - this works best for PaaS workloads (with some sharing - where separation is at the k8s namespace and/or service level) - (aka transit gateway)
2 - workload (non-shared) VPC's peered to the shared perimeter (1 fortigate cluster per gc-cap - with flow separation)
3 - workload (non-shared) VPC's peered to their own perimeter (usually prod/stg/dev fortigate separation but we can expand) - I have only see 2 above though
For all 1-3 we can separate using explicit and implicit routing/firewall-rules separation
Usually 2-3 are for both free-form sandbox and specific prod workloads (one team does bucket downloads for example) - where the rest of the workloads are ok with sharing the paas in 1
see prototyping that needs to be moved to kcc
org a- gcp.obrien.services https://console.cloud.google.com/networking/peering/list?orgonly=true&project=ospe-obs-obsprd-obspubper&supportedpurview=project

Existing peering to be updated as per
https://github.com/GoogleCloudPlatform/pubsec-declarative-toolkit/blob/dev/solutions/landing-zone/architecture.md#di-20-separate-vpc-per-cloud-profile-356-workloads
uncomment and KPT render each peer pair in
https://github.com/GoogleCloudPlatform/pubsec-declarative-toolkit/blob/main/solutions/landing-zone/environments/common/network/network-peering.yaml#L15

see https://github.com/GoogleCloudPlatform/pbmm-on-gcp-onboarding/issues/161

Investigate hub spoke as part of partner interconnect enabled transitive peering solution (Around the 25 limit)
#298
https://github.com/GoogleCloudPlatform/pubsec-declarative-toolkit/blob/dev/solutions/landing-zone/architecture.md#di-05-complete-network-design


Add Django Sites Framework compatibility

I want to implement one codebase of helpdesk for different sites, for that, I need to use Sites Framework of Django.
But, there is no foreign key to the Site Model in the Tickets or Queue model.

We already support teams. You can change the team class and set it to a custom class. I suggest using that to query by site.

https://github.com/django-helpdesk/django-helpdesk/blob/stable/docs/teams.rst

Thanks, to be clear, are you asking to add a Sites foreign key to the TIckets and Queue models, @shreyashsrivastava ? Is there any other model that needs this or is that enough?

I'm going to punt this to the next major release (0.5.0) to give some time to explore this and double check any side effects with implementing this (proper migrations, etc)


[BUG] gnn_fraud_detection_pipeline intermittently segfaults on exit

Describe the bug
This appears to happen roughly 1/10 of the time:
====Pipeline Complete====
*** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1660062916" if you are using GNU date ***
PC: @                0x0 (unknown)
*** SIGSEGV (@0x5627d20076f6) received by PID 1879751 (TID 0x7f4c59d05740) from PID<PHONE_NUMBER>2937830134; stack trace: ***
    @     0x7f4c59021197 google::(anonymous namespace)::FailureSignalHandler()
    @     0x7f4c5a04d200 (unknown)
    @     0x7f48194baba9 std::_Hashtable<>::find()
    @     0x7f48194c0cd1 _ZNSt19_Sp_counted_deleterIPN4raft13interruptibleEZNS1_14get_token_implILb1EEESt10shared_ptrIS1_ENSt6thread2idEEUlT_E_SaIvELN9__gnu_cxx12_Lock_policyE2EE10_M_disposeEv
    @     0x7f48194a9650 std::shared_ptr<>::~shared_ptr()
    @     0x7f4c59102f3c (anonymous namespace)::run()
    @     0x7f4c59d44f97 (unknown)
    @     0x7f4c59d4513a exit
    @     0x5622a9c834b1 Py_Exit
    @     0x5622a9c83400 handle_system_exit
    @     0x5622a9c830d3 _PyErr_PrintEx
    @     0x5622a9c8ba4f PyRun_SimpleFileExFlags
    @     0x5622a9c7f0ac Py_RunMain.cold
    @     0x5622a9dc0219 Py_BytesMain
    @     0x7f4c59d2d81d __libc_start_main
    @     0x5622a9dc0125 (unknown)
Segmentation fault

Steps/Code to reproduce bug
cd examples/gnn_fraud_detection_pipeline
python run.py

Expected behavior
Clean exit
Environment overview (please complete the following information)

Environment location: Bare-metal
Method of Morpheus install: from source

I'm no longer able to reproduce the segfault, however while attempting to repro I ran into trouble getting a good version of tensorflow installed that was able to parse the model.


Add docs for v0.16.x 📝

Changes
Adding docs for the recent 0.16.0 release, the 0.16.1 release I'm
currently working on and the release we'll make next week for 0.16.2
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:

[n/a] Includes tests (if functionality changed/added)
[x] Includes docs (if user facing)
[x] Commit messages follow commit message best practices
[x] Release notes block has been filled in or deleted (only if no user facing changes)

See the contribution guide for more details.
Double check this list of stuff that's easy to miss:

If you are adding a new binary/image to the cmd dir, please update
the release Task to build and release this image.

Reviewer Notes
If API changes are included, additive changes must be approved by at least two OWNERS and backwards incompatible changes must be approved by more than 50% of the OWNERS, and they must first be added in a backwards compatible way.
Release Notes
NONE

/kind documentation
Waiting for 0.16.1 and 0.16.2:
/hold

/approve


PR Merge Pipeline Errors

Receiving errors when merging in PRs from forked repositories.

Fixed by PR#23


Missing CID?

is there a place we should be posting broken things we find along the way? here?
Slack Message

This CID link is fixed now: https://help.sumologic.com/?cid=6005


Problem with DecimalWidget with zero value

An error occurs when importing fields that have zero value in xls file.
Original:
class DecimalWidget(NumberWidget):
    """
     Widget for converting decimal fields.
     """
    def clean(self, value):
        if not value:
            return None
        return Decimal(value)

My suggestion for fix:
class DecimalWidget(NumberWidget):
    def clean(self, value):
        return Decimal(value if value else 0)

@mateuspadua which error does it raise?

the error is (1048, "Column 'my_column' cannot be null")
I believe that is because the column can not receive as null value, but in the import file, the value for this column has been set to zero, so I guess that could not give error but to set zero for this column when saving.
Thanks for answer.

Are you sure that field in excel is not empty? It should be "0" in field

Hi, i am sure that field is with "0" value in excel.  Thanks for answer.

Would it be possible for you to add failing test?

Yes, i am going to prepare one and i will send to you. Thanks

How i can send this test to you?

All tests should go to test folder, please see https://github.com/bmihelac/django-import-export/tree/master/tests

Hello guys,
I am not sure if this is related but I have same problem when I import excel column with zero value. This is in my model:
concentration = models.PositiveIntegerField(default=0, blank=False, null=False)
and i get (1048, "Column 'concentration' cannot be null")
The excel cells have zero values. For some reason the values in excel are considered as null. Do you have any idea?
Thank you for the help


update swiftbrowser data folder

update swiftbrowser data folder to include the new tco1279 and tco319 data

closed by PR #95


[General] remove external vendor unknown and placeholder folders

Removal doesn't affect the build, so I assume they are only intended for make board-info.
It is planned to overhaul the buildscript/Makefile, so it should be fine to remove this function for now.

thank god someone did this finally


fix copyright year

Ⅰ. fix copyright year for /pkg/common/label.go
Ⅱ. Does this pull request fix one issue?

fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/lgtm


issue listing procedures in a package:  "No Editor symbols."

I am using Visual Code to view all of the database objects in our Oracle database that I scripted to files. I installed the extension "Language PL/SQL" found here:
https://marketplace.visualstudio.com/items?itemName=xyz.plsql-language
On the extension download website page mentioned above, if you look at the animate GIF, it gives the impression that if you type "@" in the command palette that the extension should list all of the procedures in an Oracle package that I have open, but instead it gives me a message under the popup: "No Editor symbols."
Is it someone possible to either jump to a procedure within a package or better yet, see an outline of the package so, as I scroll through it, I can tell which procedure I am currently in?

I found the issue:
I was using the Oracle function DBMS_METADATA.GET_DDL to get the DDL for a package. If you don't pass a version number into the function, the PACKAGES contain the word "CREATE OR REPLACE ** EDITIONABLE *** PACKAGE . The extension as choking on the word EDITIONABLE. So, I told the GET_DDL function to give me version 11.2 compatible DDL and the extension was worked


Adding indented text for internationals to parents/carers and disabled page

Trello card
https://trello.com/c/Zp3EFYvI/4440-add-international-inset-text-to-if-youre-a-parent-or-carer-if-youre-disabled-and-if-youre-a-veteran-pages
Context
Most international users won't be eligible for the support detailed on the 'if you're a parent or carer' and 'if you're disabled' pages.
We need to highlight this and effectively signpost them on to other relevant support.
Changes proposed in this pull request
Guidance to review

Have discussed the wording with Laura - she's just double checking with Jane to approve. Laura has confirmed we don't need this indented text for the veterans page as the info there can apply to them @philspray @gemmadallmandfe

@EllieNodder My concern with this is around the positioning - being right at the top of the page. Could this be a bit offputting for people who are actually eligible, who may not scroll down? It is a big block of text on mobile in particular
Is there any reason why it couldn't go beneath the intro paras, above the h2 (as we do on steps)? We don't have feedback (as far as I'm aware) that lots of international candidates are applying for this funding
cc @philspray

Have tweaked after feedback from Jayne @gemmadallmandfe @philspray


AOT Compilation Error: import withinviewport from 'withinviewport';

I am getting an error when I try to rollup the project (after AOT compiled), when I use ngx-tour. The error is as below. Maybe I should add something in rollup.config.js but I dont know what exactly.
[!] Error: 'default' is not exported by node_modules\withinviewport\withinviewport.js
https://github.com/rollup/rollup/wiki/Troubleshooting#name-is-not-exported-by-module
node_modules\ngx-tour-md-menu\ngx-tour-md-menu\ngx-tour-md-menu.es5.js (20:7)
18: import { Subject } from 'rxjs/Subject';
19: import { Directionality, MdButtonModule, MdCardModule, MdIconModule, MdMenu, MdMenuModule, MdMenuTrigger, Overlay } from '@angular/material';
20: import withinviewport from 'withinviewport';
           ^
21: var TourState = {};
22: TourState.OFF = 0;

In my ng-package-md-menu.json file I have the following externals declared:
      "@angular/cdk": "_angular_cdk",
      "@angular/material": "_angular_material",
      "withinviewport": "withinviewport"

These are translated into rollup externals/globals by ng-packagr.  I'm definitely out of my depth debugging rollup configs, but hopefully that gives you some direction.


Make previewImagesInEditor false by default

After upgrading to v2.12.0, I noticed that now images are shown in the editor. It was quite surprising to see such a big behavior change for a minor upgrade. I am using an unversioned CDN ("https://unpkg.com/easymde/dist/easymde.min.js"), so I didn't initiate the upgrade manually, and at first I thought there was some kind of bug. I toggled the edit/preview button multiple times, refreshed the page, until I finally investigated and found that the "issue" was caused by a new feature:
https://github.com/Ionaru/easy-markdown-editor/pull/235
While this is a potentially useful feature (thx @ivictbor  !), I really think this ought to be false by default. The first reason is obviously to be backward compatible with the previous behavior. The second reason is that even without backward compatibility concerns, I think showing the image in the editor kinda defeats the primary purpose of Markdown editing, which is to have precise control over the Markdown text to be sent to the server. When images are displayed, it makes editing harder and clunky. We cannot really be sure how many new lines or whitespaces there are before/after the image, and overall we lose some confidence about what's in the Markdown. I think that in most cases, if we want to see images, then a WYSYWYG editor makes more sense than a Markdown editor. It's subjective, obviously, but I feel that I may have not chosen easyMDE if in the demo page I had experienced this behavior, and not seen that it could be disabled.
Anyway, I've already personally turned it off, thanks a lot for making this possible! I just wanted to bring this to your attention, and perhaps start a discussion if others would like to argue for or against having it to false by default.

@dalboris @Ionaru Yep, agree, I think we need to set false (at least for now), I also noted several minor issues, e.g. when image is posted in inline string

Released in 2.12.1


HADOOP-14208. Fix typo in the top page in branch-2.8

Would you target branch-2.8 instead of branch-2.8.1?

OK, I'll try immediately.

Would you close this PR?

I closed it, @aajisaka, Thanks for your remind.


Demonstration run from README is not replicable

Hi @jdutant, great package, I love the idea and the potential use cases.
I tried to do the demo run you have in the README. In particular, I went down the Quarto route by cloning the repo and ran make quarto && open example-quarto/example.html and am not able to replicate the doc.
I am getting a <LaTeX content not imagified> for all the contents supposedly filtered by imagify as can be seen in the attached image below.

I am using Quarto version 1.5.55 and I have tested the output HTML on both Safari and Google Chrome and neither is working.
Best,
Salman

Hi @salfaris , thanks for your interest in the filter (and apologies for a late reply).
The default example runs fine on my system (Quarto 1.6). I'm not sure what problem you face.
LaTeX content not imagified text in output means that Imagify has tried and failed to convert the TeX code. The output log would tell you more. There's a 'debug' option to get more informative output.
A thought: could it be that LaTeX isn't installed on your system outside of Quarto? To convert LaTeX/TikZ to SVG, the filter relies on a LaTeX installation (and its package dvisvgm). Quarto includes a small LaTeX installation to generate PDFs but (as far as I know) the filter can't use it. So, the filter needs LaTeX installed on your system (TexLive or MikTeX, for instance).


Added  solution for LT problem 119 . Pascal's Triangle II

Description
I have added solution for LT problem 119 . Pascal's Triangle II
closes : #747
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / Videos (if applicable)

Checklist

[x] I have performed a self-review of my code.
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[ ] I have commented my code, particularly in hard-to-understand areas.

Additional Context
[Include any additional information or context that might be helpful for reviewers.]
Resources for Guidance
Here are some resources that may be helpful as you contribute to CodeHarborHub:

Code Harbor Hub Community Features
Docusaurus Documentation
React.js Documentation
Markdown Guide
MDX Documentation
Mermaid Documentation

Resources for DSA Features on Our Site:

Tabs: Switch Languages
Line Numbering
Math Equations

@Ayushmaanagarwal1211 update your code

@Ajay-Dhangar  Updated


operationComplete of ChannelFuture is not called

My application calls channel.writeAndFlush to send a HTTP2 request and then get a ChannelFuture, then calls addListener in order to be noticed while the writing procedure complete,  but from log there are some case that the operationComplete is not called, actually at that time the HTTP2 response has been received,
I want to know if there are some cases that operationComplete could not be called after writeAndFlush
Thank you for any advice

it must be always called..
Can you share your code.

On 30 Nov 2016, at 16:18, liuhualh<EMAIL_ADDRESS>wrote:
My application calls channel.writeAndFlush to send a HTTP2 request and then get a ChannelFuture, then calls addListener in order to be noticed while the writing procedure complete, but from log there are some case that the operationComplete is not called, actually at that time the HTTP2 response has been received,
I want to know if there are some cases that operationComplete could not be called after writeAndFlush
Thank you for any advice
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/netty/netty/issues/6073, or mute the thread https://github.com/notifications/unsubscribe-auth/AAa0Qlb8HZ1AZ7KsoyBx95WI68vznHjoks5rDZPKgaJpZM4LATsn.

this sounds like an issue please share a reproducer

These days I have extracted HTTP2 related code from my application, but unfortunately I can not reproduce this case. But I still can reproduce it in our testing application.
Additionally, while trying to reproduce this case , only one time I reproduce this case, at that time I sent some HTTP2 POST request before HTTP2 SETTING negotiation finished, I don know if this the reason for lost of ** operationComplete**
Do you have some other advice for me to find the root cause, Thank you

Have found root reason, this is a bug in my application, sorry for disturbing

@liuhualh thanks for closing the loop :)


2.6-beta eqnarray spacing issues

Reported on meta.math.SE http://meta.math.stackexchange.com/a/21520/26910

This is fixed in the issue1272 branch of my fork of MathJax.

=> Merged

Already in test suite (LaTeXToMathML/environments/eqnarray-*), but these will have to be updated to check for the new MathML.


Add MantiCore

Once the chain launches ill have everyone sign a message with a memo including their peerid. Then include the txhash in the addedPeer field please


Prevent manual input, but allow cell change programmatically

Hi Paul,
Is a way to prevent manual change to a cell in a table, but allow JavaScript to make change?
Thanks!

Never mind. Found solution from a closed thread.
$('#my').jexcel('setValue', 'D2', 10, true);


Add geolocation support for locations

Summary
This PR adds geolocation support for locations as well as themes the full content display.
Metadata

Question
Answer

Did you use a meaningful pull request title?
yes

Did you apply meaningful labels to the pull request?
yes

Did you perform a self review first?
yes

CHANGELOG reflects changes?
yes

Did you perform browser testing?
yes

Risk level
Low

Relevant links
https://thinkoomph.jira.com/browse/RIGA-69

This looks good!
/approved


Did not work with multiple domains; here's a fix

I did not find that the init script worked if I had multiple domains; also I found that I needed to parameterize it to reflect staging/production environments that use different docker-compose files and domains.
Here's what I came up with -- if you like I can make a PR.
#!/bin/bash
die () {
    echo >&2 "$@"
    exit 1
}

[ "$#" -ge 2 ] || die "usage: init-letsencrypt.sh [staging | production] <domain1 domain2 ...> "

domains=(${@:2})
rsa_key_size=4096
data_path="/etc/certbot"
<EMAIL_ADDRESS># Adding a valid address is strongly recommended
staging=0 # Set to 1 if you're testing your setup to avoid hitting request limits

if [ -d "$data_path/conf/live/" ]; then
  read -p "Existing data found. Continue and replace existing certificates? (y/N) " decision
  if [ "$decision" != "Y" ] && [ "$decision" != "y" ]; then
    exit
  fi
fi

if [ ! -e "$data_path/conf/options-ssl-nginx.conf" ] || [ ! -e "$data_path/conf/ssl-dhparams.pem" ]; then
  echo "### Downloading recommended TLS parameters ..."
  mkdir -p "$data_path/conf"
  curl -s https://raw.githubusercontent.com/certbot/certbot/master/certbot-nginx/certbot_nginx/options-ssl-nginx.conf > "$data_path/conf/options-ssl-nginx.conf"
  curl -s https://raw.githubusercontent.com/certbot/certbot/master/certbot/ssl-dhparams.pem > "$data_path/conf/ssl-dhparams.pem"
  echo
fi

for domain in "${domains[@]}"; do
  echo "### Removing old certificate for $domain ..."
  docker-compose -f docker-compose.yml -f docker-compose.$1.yml run --rm --entrypoint "\
    rm -Rf /etc/letsencrypt/live/$domain && \
    rm -Rf /etc/letsencrypt/archive/$domain && \
    rm -Rf /etc/letsencrypt/renewal/$domain.conf" certbot
  echo
done

for domain in "${domains[@]}"; do
  echo "### Creating dummy certificate for $domain ..."

docker-compose -f docker-compose.yml -f docker-compose.$1.yml up --force-recreate -d nginx
echo

for domain in "${domains[@]}"; do
  echo "### Removing dummy certificate for $domain ..."
  docker-compose -f docker-compose.yml -f docker-compose.$1.yml run --rm --entrypoint "\
    rm -Rf /etc/letsencrypt/live/$domain" certbot
  echo
done

echo "### Requesting Let's Encrypt certificates ..."

# Select appropriate email arg
case "$email" in
  "") email_arg="--register-unsafely-without-email" ;;
  *) email_arg="--email $email" ;;
esac

# Enable staging mode if needed
if [ $staging != "0" ]; then staging_arg="--staging"; fi

for domain in "${domains[@]}"; do
  docker-compose -f docker-compose.yml -f docker-compose.$1.yml run --rm --entrypoint "\
    certbot certonly --webroot -w /var/www/certbot \
      $staging_arg \
      $email_arg \
      -d $domain \
      --rsa-key-size $rsa_key_size \
      --agree-tos \
      --force-renewal" certbot
  echo
done

echo "### Reloading nginx ..."
docker-compose -f docker-compose.yml -f docker-compose.$1.yml exec nginx nginx -s reload

I like the idea of calling the script with the domains as parameters. That’s so much more elegant!
Instead of using staging and production as arguments, I would suggest a --staging switch, since productiondoesn't seem to do anything.

Thanks for sharing this. Was looking to add the same. Worked like a charm

The 2nd argument (production | staging) determines which docker-compose.$1 file is used later in the script.  So that works if that's how you organize your docker-composes, otherwise the user has to modify accordingly.  I'll create a PR shortly...

I was just working with this and came to a really close solution, came here to check for any pending PRs or issues.
Nice dude, this will help a lot of people o/

@dchersey I think it would be nicer to be more explicit here. So I suggest adding an -f parameter that allows specifying the actual docker-compose.yml path.

Well, it wasn't so flawless for me. For the first domain everything worked smoothly, but when it running for the second one I got a following error:
Failed authorization procedure. <mydomain> (http-01): urn:ietf:params:acme:error:connection :: The server could not connect to the client to verify the domain :: Fetching http://<mydomain>/.well-known/acme-challenge/VaCGMLcnszmYVc6afGPJbGwUS2SjuKhWABLcCv2HZkM: Connection refused

do a PR <3

I also like this solution!  I apologize if you get an email for some domains I was testing @dchersey i forgot to change the address before I ran it.  However with your updated script I am getting:
req: Can't open "/etc/letsencrypt/live/domain-example.com/privkey.pem" for writing, No such file or directory

for each of the domains, do you know why i get that error?

Same for me. I also got req: Can't open "/etc/letsencrypt/live/domain-example.com/privkey.pem" for writing, No such file or directory

Both the current and this modified init script just hang for me when getting dummy certs..

This worked for me. Thanks! I hope this is merged soon as the code on master is broken for multiple domains

I don't think it will, since the maintainer doesn't seem to focus on this script for the last few months

Hey everyone, thank you for your patience!
The new 2.x branch now makes it much easier to specify multiple domains. You can now call the script like this to initialize it with multiple domains: ./init-letsencrypt.sh -d domain1.org -d domain2.org :tada:


Make it easier to write and read tiled dataset.

Taking this point off of #45.
@mattersoflight suggested:

Introduce a tiles or tiled layout. I picked these words in favor of multiPos or multiFOV. But, I think either of them is a decent choice.

Along with this usage pattern:
https://github.com/czbiohub/iohub/blob/61237f9a9b7a99eb7a1e74ceaa01c0c93204a518/examples/tiles_zarr.py#L1-L41

As far as I can tell, this will require:

Designing a custom hierarchy layout and describe it in a specification that devs and users can refer to.
This has been discussed in #22 and the result is achievable with the current API (see example).
A new class that override the methods (e.g. create_position()) of iohub.ngff.Plate. This means that the objects returned by the main NGFF entry point for standard and this custom layout will have methods of the same name, but take different arguments and mutate datasets differently.

IMO this is not the safest design/usage pattern to manipulate datasets. I'm also not sure if this will bring significant syntactic benefit considering that it only commits a FOV name argument in the get/set calls.

I am thinking of a thin wrapper around hcs layout so that when the user uses tiled layout, they have to only row and col, but not fov. When the acquisition is tiled, the fov parameter doesn't make sense.

When the acquisition is tiled, the fov parameter doesn't make sense.

Then how about a TiledPlate class that inherits from ngff.Plate and adds a couple of extra methods. For example make_tiles(n_rows, n_cols) or create_tile(i_row, i_col), which hidesfov by hardcoding its value to '0'.
My main concern with overriding existing methods is that it's hard to distinguish between multi-well plates and tiled acquisition when reading user-side code that expect different behavior.

@mattersoflight and I just had a chat and realized that tiled acquisitions actually suits well in a chunked array where each acquired frame is a (x_chunksize, y_chunksize) slice of the array. We just need to subclass ngff.ImageArray with something like ngff.TiledImageArray that has some extra convenience methods to create, read, and write tiles.


tag-filtering-expansion

Fixes Issue
closes #662

Changes proposed

Created the Tags directory in the common folder to accommodate tag-related functionality.
Moved filterByTags functionality to common/Tags/filterByTags.jsx

Renamed the tag folder in the explore section to ExploreTags for consistency. This adjustment required renaming relevant files in the ExploreTags folder  such as Tags.jsx to ExploreTags.jsx and TagsElements.jsx to ExploreTagsElements.jsx.

Screenshots

Note to reviewers
The Tags directory serves as a potential hub for future tag-related functions. For instance, I'm considering  implementing a feature to identify top trending tags based on usage.)

Code of Conduct

[x] By submitting this pull request, I confirm I've read and complied with the CoC 🖖

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

You can also join our Discord community.
Feel free to check out other cool repositories of the Thecyberworld.
Join the Thecyberworld GitHub Organisation by raising an issue (you will be sent an invitation).

@Som401 do you need some help with conflicts?

there will be lots of confilicts i think, lets see

there will be lots of confilts, lets see

@ArkadiK94  yes please, its my first time facing that kind of issue

i mistakenly deleted the branch hh


Proxied download doesn't work

And I tired the demo,same problem.

ok i see the problem, the directLink option used in api.json is set to only allow onedrive direct links, if not specified, the proxy could be used to relay any random link what-so-ever. i see that you are using sharepoint, so you can set that option to something like https://nvzwjezr-my.sharepoint.com/.
the reason why the demo site is also not working is that i thought all direct links start with
https://public.dm.files.1drv.com

but today it suddenly starts with
https://public.bn.files.1drv.com

should be fixed some time later.

onedrive international users should be able to use this directly, sharepoint users need to change the regex matching the url in directLinkRegex defined in api.json.

resolved. Thx!


Add Curry as dependency

Can you add curry as dependency in the Podspec?
Argo extracted curry into separate pod some time ago.

Have the same issue, could you please also add Curry as dependency in the Podspec in branch swift 2.3 ?
Many thanks.


Review the restrictions for maximum leaf area, why the potential leaf area is high?

Originally in: https://github.com/danipilze/dssat-csm/issues/98
Based on the report from James:

... we found one variety (Mven 77) with a maximum leaf area of 900 cm2. I personally have never seen a cassava leaf that big and wonder where these data for very large leaves comes from. We discovered these very large leaves when we were trying to track down excessive leaf (and stem) production in MCol 22 in the later stages of growth

I think that probably it is the result of temperature or water stress restrictions in the experiments where it was necessary to put those high potential values to get the actual leaf area reported in the experiments.

Apparently the issue of low values of potential leaf size are related to the leaf area reduction due to water stress. This stress index is the same applied to leaf appearance, is it possible that it should be reduced for leaf area?
Meanwhile, I will leave the model as it is.


Simple enhancement for readibility

I think it is more easy to read the code with changing not not to bool.
https://github.com/huggingface/datatrove/blob/452e69ab17c8e5f25243e0b89a8c6c385e4c9a5a/src/datatrove/tools/check_dataset.py#L77

fix typo (add newlines in docstring)
https://github.com/huggingface/datatrove/blob/452e69ab17c8e5f25243e0b89a8c6c385e4c9a5a/src/datatrove/io.py#L119

LGTM!