Skip to content

Automator: update envoy@ in istio/proxy@master#5455

Closed
istio-testing wants to merge 1 commit intoistio:masterfrom
istio-testing:none-master-update_envoy_dep-1c4d1f14
Closed

Automator: update envoy@ in istio/proxy@master#5455
istio-testing wants to merge 1 commit intoistio:masterfrom
istio-testing:none-master-update_envoy_dep-1c4d1f14

Conversation

@istio-testing
Copy link
Copy Markdown
Collaborator

Generated by Automator - 2024-04-10T13:12:19+00:00

@istio-testing istio-testing requested a review from a team April 10, 2024 13:12
@istio-testing istio-testing added auto-merge Set this label on a PR to auto-merge it on success of presubmit tests size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Apr 10, 2024
@ericvn
Copy link
Copy Markdown

ericvn commented Apr 10, 2024

/test test-tsan

@ericvn
Copy link
Copy Markdown

ericvn commented Apr 10, 2024

/test release-test

@ericvn
Copy link
Copy Markdown

ericvn commented Apr 10, 2024

/test release-test-arm64

@istio-testing
Copy link
Copy Markdown
Collaborator Author

@istio-testing: 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
release-test_proxy fa73817 link true /test release-test
release-test-arm64_proxy fa73817 link true /test release-test-arm64
Details

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.

@ericvn
Copy link
Copy Markdown

ericvn commented Apr 10, 2024

Caused by envoyproxy/envoy#30850?

+ go run tools/extension-check/main.go --ignore-extensions tools/extension-check/wellknown-extensions --envoy-extensions-build-config /tmp/tmp.vKTsY4BG3y/envoy.bzl --proxy-extensions-build-config /tmp/tmp.vKTsY4BG3y/proxy.bzl
ignore extensions:  29
missing extension: envoy.filters.http.credential_injector: //source/extensions/filters/http/credential_injector:config
missing extension: envoy.http.injected_credentials.generic: //source/extensions/http/injected_credentials/generic:config
exit status 1
+ echo 'failed to check extension build config'

@ericvn
Copy link
Copy Markdown

ericvn commented Apr 10, 2024

I did create #5458 which should supersede this with a extension change.

@ericvn
Copy link
Copy Markdown

ericvn commented Apr 10, 2024

Superseded by #5458

@ericvn ericvn closed this Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge Set this label on a PR to auto-merge it on success of presubmit tests size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants