Skip to content

envoy: Update to release 1.17.2 #15740

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 20, 2021
Merged

envoy: Update to release 1.17.2 #15740

merged 1 commit into from
Apr 20, 2021

Conversation

jrajahalme
Copy link
Member

[ upstream commit d42cc41 ]

Update Envoy image to release 1.17.2, including the latest security fixes.

Signed-off-by: Jarno Rajahalme jarno@covalent.io

Envoy is updated to release 1.17.2

@jrajahalme jrajahalme added release-note/bug This PR fixes an issue in a previous release of Cilium. kind/backports This PR provides functionality previously merged into master. backport/1.7 labels Apr 16, 2021
@jrajahalme jrajahalme requested a review from a team as a code owner April 16, 2021 05:37
@jrajahalme
Copy link
Member Author

test-backport-1.7

@qmonnet qmonnet mentioned this pull request Apr 16, 2021
Merged
@aanm aanm added the dont-merge/waiting-for-upstream Only merge once upstream library or kernel features have landed label Apr 16, 2021
@jrajahalme
Copy link
Member Author

@aanm None of the test fails in Cilium-Ginkgo-Tests seem related to this PR, is there a way to retest it?

@joestringer
Copy link
Member

joestringer commented Apr 16, 2021 •
edited
Loading

Previous failure link:
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/20094/

These tests failed:
Suite-k8s-1.17.K8sServicesTest External services To Services first endpoint creation
Suite-k8s-1.17.K8sServicesTest External services To Services first policy
Suite-k8s-1.17.K8sServicesTest External services To Services first endpoint creation match service by labels
Suite-k8s-1.17.K8sServicesTest External services To Services first policy, match service by labels
Suite-k8s-1.17.K8sIstioTest Istio Bookinfo Demo Tests bookinfo inter-service connectivity
Suite-k8s-1.17.K8sChaosTest Restart with long lived connections TCP connection is not dropped when cilium restarts
Suite-k8s-1.17.K8sChaosTest Restart with long lived connections L3/L4 policies still work while Cilium is restarted

Why are the 1.17 tests running from the runtime trigger? 🤔

EDIT: Oh, this is normal, that job runs 1.11 + 1.17 + runtime.

@joestringer
Copy link
Member

restart-ginkgo

@joestringer
Copy link
Member

For what it's worth, multiple of these failures seem to match the instability in the v1.8 tree.

@joestringer joestringer force-pushed the pr/v1.7-backport-2021-04-16 branch from 50ee975 to 38f8507 Compare April 16, 2021 23:05
@joestringer
Copy link
Member

joestringer commented Apr 17, 2021 •
edited
Loading

Looks like I accidentally replaced the backport here with the fix for the tree. Need to retrieve the original commit.

EDIT: Fixed now.

@joestringer joestringer force-pushed the pr/v1.7-backport-2021-04-16 branch from 38f8507 to 50ee975 Compare April 17, 2021 02:59
@jrajahalme @aanm
envoy: Update to release 1.17.2
27ef271 
[ upstream commit d42cc41 ]

Update Envoy image to release 1.17.2, including the latest security fixes.

Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
@aanm aanm force-pushed the pr/v1.7-backport-2021-04-16 branch from 50ee975 to 27ef271 Compare April 17, 2021 21:04
@aanm
Copy link
Member

aanm commented Apr 17, 2021

test-backport-1.7

@aanm aanm removed the dont-merge/waiting-for-upstream Only merge once upstream library or kernel features have landed label Apr 17, 2021
@aanm
Copy link
Member

aanm commented Apr 17, 2021 •
edited
Loading

restart-ginkgo (previous failure https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/20098/)

@aanm
Copy link
Member

aanm commented Apr 18, 2021 •
edited
Loading

test-backport-1.7-k8s (previous failure https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/4020)

@aanm aanm merged commit cdb417f into v1.7 Apr 20, 2021
@aanm aanm deleted the pr/v1.7-backport-2021-04-16 branch April 20, 2021 20:27
@joestringer joestringer mentioned this pull request Apr 20, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
kind/backports This PR provides functionality previously merged into master. release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jrajahalme @joestringer @aanm