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.

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

envoy: Update to release 1.17.1 #14754

Merged
merged 4 commits into from
Mar 5, 2021
Merged

envoy: Update to release 1.17.1 #14754

merged 4 commits into from
Mar 5, 2021

Conversation

jrajahalme
Copy link
Member

@jrajahalme jrajahalme commented Jan 27, 2021 •
edited

Update Envoy to release 1.17.1.

All extensions not needed for Cilium policy enforcement are compiled out, leading to 45% smaller binary size (45.8 MB -> 25.6 MB).

Note for backporting: Only backport the first commit onto the main Dockerfile of the release branches.

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

Envoy is updated to release 1.17.1

@jrajahalme jrajahalme added area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. release-note/minor This PR changes functionality that users may find relevant to operating Cilium. labels Jan 27, 2021
@jrajahalme jrajahalme requested a review from a team as a code owner January 27, 2021 18:02
@jrajahalme jrajahalme requested a review from aanm January 27, 2021 18:02
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Jan 27, 2021
@jrajahalme jrajahalme marked this pull request as draft January 27, 2021 18:03
@jrajahalme
Copy link
Member Author

test-me-please

@jrajahalme jrajahalme force-pushed the envoy-1.17.0 branch 2 times, most recently from e760e9e to 8902130 Compare January 27, 2021 19:13
@jrajahalme
Copy link
Member Author

test-me-please

1 similar comment
@jrajahalme
Copy link
Member Author

test-me-please

@jrajahalme
Copy link
Member Author

test-me-please

@jrajahalme
Copy link
Member Author

4.9 known flake #13071

@jrajahalme
Copy link
Member Author

Net-next known flake #12511

@jrajahalme
Copy link
Member Author

retest-net-next

@jrajahalme
Copy link
Member Author

retest-4.9

@jrajahalme jrajahalme force-pushed the envoy-1.17.0 branch 3 times, most recently from 8c07c8c to 980da67 Compare January 30, 2021 08:49
@jrajahalme
Copy link
Member Author

test-me-please

@jrajahalme
Copy link
Member Author

test-me-please

@jrajahalme jrajahalme force-pushed the envoy-1.17.0 branch 2 times, most recently from bbd9ede to ad9854d Compare February 2, 2021 22:51
@jrajahalme
Copy link
Member Author

test-me-please

@joestringer joestringer moved this from Done to In progress in 1.10.0 Feb 12, 2021
@jrajahalme
Copy link
Member Author

test-me-please

@jrajahalme
Copy link
Member Author

test-me-please

@jrajahalme jrajahalme removed the dont-merge/blocked Another PR must be merged before this one. label Mar 5, 2021
@jrajahalme
Copy link
Member Author

Quarantined TLS L7 CI test passed on local testing:

$ K8S_VERSION=1.19 ginkgo --focus="K8s.*TLS policy" -v -- -cilium.runQuarantined=true -cilium.holdEnvironment=true -cilium.provision=false
...
17:36:14 STEP: Testing L7 Policy with TLS
=== Test Finished at 2021-03-04T17:36:25-08:00====
...
Ginkgo ran 1 suite in 2m20.126306386s
Test Suite Passed

@jrajahalme
Copy link
Member Author

retest-1.20-4.9

@aanm aanm merged commit 29f01fd into cilium:master Mar 5, 2021
1.10.0 automation moved this from In progress to Done Mar 5, 2021
This was referenced Mar 5, 2021
Merged
Merged
Merged
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.8 in 1.8.8 Mar 5, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.9 in 1.9.5 Mar 5, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.7 in 1.7.15 Mar 5, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.8 to Backport done to v1.8 in 1.8.8 Mar 6, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.9 to Backport done to v1.9 in 1.9.5 Mar 6, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.7 to Backport done to v1.7 in 1.7.15 Mar 8, 2021
This was referenced Mar 10, 2021
Merged
Merged
@christarazi christarazi mentioned this pull request Mar 10, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aanm aanm aanm approved these changes

@kkourt kkourt Awaiting requested review from kkourt

Assignees

@kkourt kkourt

Labels
area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. kind/bug This is a bug in the Cilium logic. release-note/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
No open projects
1.10.0
Done
1.7.15
Backport done to v1.7
1.8.8
Backport done to v1.8
1.9.5
Backport done to v1.9
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@jrajahalme @aanm @kkourt @joestringer