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

docs: Add matrix version between envoy and cilium #25109

Merged
merged 1 commit into from
Apr 28, 2023

Conversation

sayboras
Copy link
Member

This is to add a small doc for the version matrix between Cilium and Cilium envoy versions, which is useful with the upcoming work to move the envoy proxy out of Cilium agent container.

@sayboras sayboras requested review from a team as code owners April 25, 2023 12:55
@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 Apr 25, 2023
@sayboras sayboras added the release-note/misc This PR makes changes that have no direct user impact. label Apr 25, 2023
@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 Apr 25, 2023
@sayboras sayboras added area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. needs-backport/1.11 needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels Apr 25, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.13.3 Apr 25, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.12.10 Apr 25, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from main in 1.11.17 Apr 25, 2023
@sayboras
Copy link
Member Author

This is only for documentation update, so full CI is not required.

Copy link
Contributor

@zacharysarah zacharysarah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sayboras Nice work, just a couple of nits. ✨ I'm approving with the understanding that changes need to happen before merge.

Documentation/security/network/proxy/envoy.rst Outdated Show resolved Hide resolved
Documentation/security/network/proxy/envoy.rst Outdated Show resolved Hide resolved
This is to add a small docs for version matrix between Cilium and Cilium
envoy versions, which is useful with the upcoming work to move envoy
proxy out of Cilium agent container.

Co-authored-by: ZSC <zacharysarah@users.noreply.github.com>
Signed-off-by: Tam Mach <sayboras@yahoo.com>
@sayboras sayboras added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Apr 28, 2023
@sayboras
Copy link
Member Author

The changes are related to docs only, full CI is not required. Reviews are in, marking this ready to merge.

@pchaigno pchaigno merged commit 11e1bcc into cilium:main Apr 28, 2023
35 checks passed
@sayboras sayboras deleted the tam/envoy-version-docs branch April 29, 2023 00:50
@joamaki joamaki mentioned this pull request May 2, 2023
8 tasks
@joamaki joamaki added backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. and removed needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels May 2, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.13 in 1.13.3 May 2, 2023
@joamaki joamaki mentioned this pull request May 3, 2023
8 tasks
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.12 in 1.12.10 May 3, 2023
@joamaki joamaki mentioned this pull request May 3, 2023
8 tasks
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from main to Backport pending to v1.11 in 1.11.17 May 3, 2023
@julianwiedmann julianwiedmann added backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. and removed backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. labels May 9, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.13 to Backport done to v1.13 in 1.13.3 May 9, 2023
@jrajahalme jrajahalme added backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. and removed backport-pending/1.11 labels May 12, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.11 to Backport done to v1.11 in 1.11.17 May 12, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.11 to Backport done to v1.11 in 1.11.17 May 12, 2023
@julianwiedmann julianwiedmann added backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. and removed backport-pending/1.12 labels Jul 10, 2023
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.12 to Backport done to v1.12 in 1.12.10 Jul 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. backport-done/1.11 The backport for Cilium 1.11.x for this PR is done. backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. backport-done/1.13 The backport for Cilium 1.13.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/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.11.17
Backport done to v1.11
1.12.10
Backport done to v1.12
1.13.3
Backport done to v1.13
Development

Successfully merging this pull request may close these issues.

None yet

6 participants