Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

envoy: Bump envoy version to 1.21.5 #20771

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

sayboras
Copy link
Member

@sayboras sayboras commented Aug 3, 2022

envoy: Bump envoy version to 1.21.5

@sayboras sayboras requested review from a team as code owners August 3, 2022 16:11
@sayboras sayboras requested a review from rolinh August 3, 2022 16:11
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 3, 2022
@sayboras sayboras added the release-note/minor This PR changes functionality that users may find relevant to operating Cilium. label Aug 3, 2022
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Aug 3, 2022
@sayboras sayboras added the area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. label Aug 3, 2022
This new image digest is coming from below build. Also, I take this
change to upgrade cilium/proxy in go.mod as well.

https://github.com/cilium/proxy/runs/7656147038?check_suite_focus=true
Signed-off-by: Tam Mach <tam.mach@cilium.io>
@sayboras
Copy link
Member Author

sayboras commented Aug 3, 2022

/test

@sayboras
Copy link
Member Author

sayboras commented Aug 4, 2022

/test-1.24-net-next

Job 'Cilium-PR-K8s-1.24-kernel-net-next' failed:

Click to show.

Test Name

K8sServicesTest Checks device reconfiguration Detects newly added device and reloads datapath

Failure Output

FAIL: Found 1 k8s-app=cilium logs matching list of errors that must be investigated:

If it is a flake and a GitHub issue doesn't already exist to track it, comment /mlh new-flake Cilium-PR-K8s-1.24-kernel-net-next so I can create one.

1 similar comment
@sayboras
Copy link
Member Author

sayboras commented Aug 4, 2022

/test-1.24-net-next

Job 'Cilium-PR-K8s-1.24-kernel-net-next' failed:

Click to show.

Test Name

K8sServicesTest Checks device reconfiguration Detects newly added device and reloads datapath

Failure Output

FAIL: Found 1 k8s-app=cilium logs matching list of errors that must be investigated:

If it is a flake and a GitHub issue doesn't already exist to track it, comment /mlh new-flake Cilium-PR-K8s-1.24-kernel-net-next so I can create one.

@sayboras
Copy link
Member Author

sayboras commented Aug 5, 2022

/test-1.24-net-next

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 5, 2022
@aanm aanm merged commit 3c4efc6 into cilium:master Aug 8, 2022
@sayboras sayboras deleted the tam/upgrade-envoy-1.21.5 branch August 8, 2022 10:11
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.12.1 Aug 8, 2022
@tklauser tklauser added backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. and removed backport-pending/1.12 labels Aug 11, 2022
@tklauser tklauser moved this from Needs backport from master to Backport done to v1.12 in 1.12.1 Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
No open projects
1.12.1
Backport done to v1.12
Development

Successfully merging this pull request may close these issues.

None yet

5 participants