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 more details for the Cluster Mesh key rotation #28145

Merged

Conversation

margamanterola
Copy link
Member

Rather than having this information hidden in a parenthesis in the troubleshooting section, let's have a full paragraph in the key rotation section that explains what needs to be done when using Cluster Mesh.

@margamanterola margamanterola added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. area/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. release-note/misc This PR makes changes that have no direct user impact. labels Sep 13, 2023
@margamanterola margamanterola requested review from a team as code owners September 13, 2023 12:48
Copy link
Member

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

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

Thank you!

@margamanterola margamanterola force-pushed the pr/marga/update-ipsec-rotation-docs branch from 8f8aaea to d19d3d3 Compare September 13, 2023 13:01
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.

@margamanterola Valuable PR! ✨ Some small edits for clarity, otherwise LGTM

Documentation/security/network/encryption-ipsec.rst Outdated Show resolved Hide resolved
@margamanterola margamanterola force-pushed the pr/marga/update-ipsec-rotation-docs branch from d19d3d3 to f7a9dc4 Compare September 14, 2023 07:15
Rather than having this information hidden in a parenthesis in the
troubleshooting section, let's have a full paragraph in the key rotation
section that explains what needs to be done when using Cluster Mesh.

Signed-off-by: Marga Manterola <marga@isovalent.com>
@margamanterola margamanterola force-pushed the pr/marga/update-ipsec-rotation-docs branch from f7a9dc4 to b099856 Compare September 14, 2023 07:16
@margamanterola margamanterola added needs-backport/1.12 needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Sep 14, 2023
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.

LGTM

@margamanterola
Copy link
Member Author

This is a documentation-only check, so no need to waste energy running conformance tests. Setting ready-to-merge.

@margamanterola margamanterola added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Sep 15, 2023
@julianwiedmann julianwiedmann merged commit 2b4f6df into cilium:main Sep 15, 2023
41 checks passed
@margamanterola margamanterola deleted the pr/marga/update-ipsec-rotation-docs branch September 15, 2023 08:05
@doniacld doniacld mentioned this pull request Sep 22, 2023
10 tasks
@doniacld doniacld 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 Sep 22, 2023
@giorio94 giorio94 mentioned this pull request Sep 26, 2023
22 tasks
@giorio94 giorio94 added backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. and removed needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Sep 26, 2023
@giorio94 giorio94 mentioned this pull request Sep 26, 2023
12 tasks
@aanm aanm added backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. backport-done/1.12 The backport for Cilium 1.12.x for this PR is done. and removed backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. backport-pending/1.12 labels Sep 29, 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/encryption Impacts encryption support such as IPSec, WireGuard, or kTLS. 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. backport-done/1.14 The backport for Cilium 1.14.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
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants