8000 Documentation: "cilium config set" restarts by default by joamaki · Pull Request #38114 · cilium/cilium · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Documentation: "cilium config set" restarts by default #38114

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

Merged

Conversation

joamaki
Copy link
Contributor
@joamaki joamaki commented Mar 11, 2025

The documentation incorrectly said that "cilium config set" does not restart cilium-agent pods, while in fact it has done so since April 2021 (cilium/cilium-cli@c4c1dbd)

Reword and add instructions to use "--restart=false" to prevent restart.

The documentation incorrectly said that "cilium config set" does not
restart cilium-agent pods, while in fact it has done so since April 2021.

Reword and add instructions to use "--restart=false" to prevent restart.

Signed-off-by: Jussi Maki <jussi@isovalent.com>
@joamaki joamaki added the release-note/misc This PR makes changes that have no direct user impact. label Mar 11, 2025
@joamaki joamaki requested a review from a team as a code owner March 11, 2025 08:39
@joamaki joamaki requested a review from lambdanis March 11, 2025 08:39
@joestringer joestringer enabled auto-merge March 11, 2025 22:46
@joestringer
Copy link
Member

/test

@joestringer joestringer added needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch needs-backport/1.16 This PR / issue needs backporting to the v1.16 branch needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch labels Mar 11, 2025
@joestringer joestringer added this pull request to the merge queue Mar 11, 2025
Merged via the queue into cilium:main with commit 1243c11 Mar 11, 2025
68 of 69 checks passed
@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 Mar 12, 2025
@YutaroHayakawa YutaroHayakawa mentioned this pull request Mar 18, 2025
13 tasks
@YutaroHayakawa YutaroHayakawa added backport-pending/1.17 The backport for Cilium 1.17.x for this PR is in progress. and removed needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch labels Mar 18, 2025
@YutaroHayakawa YutaroHayakawa mentioned this pull request Mar 18, 2025
3 tasks
@YutaroHayakawa YutaroHayakawa added backport-pending/1.16 The backport for Cilium 1.16.x for this PR is in progress. and removed needs-backport/1.16 This PR / issue needs backporting to the v1.16 branch labels Mar 18, 2025
@YutaroHayakawa YutaroHayakawa mentioned this pull request Mar 18, 2025
4 tasks
@YutaroHayakawa YutaroHayakawa added backport-pending/1.15 The backport for Cilium 1.15.x for this PR is in progress. and removed needs-backport/1.15 This PR / issue needs backporting to the v1.15 branch labels Mar 18, 2025
@github-actions github-actions bot added backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. backport-done/1.17 The backport for Cilium 1.17.x for this PR is done. backport-done/1.15 The backport for Cilium 1.15.x for this PR is done. and removed backport-pending/1.16 The backport for Cilium 1.16.x for this PR is in progress. backport-pending/1.17 The backport for Cilium 1.17.x for this PR is in progress. backport-pending/1.15 The backport for Cilium 1.15.x for this PR is in progress. labels Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-done/1.15 The backport for Cilium 1.15.x for this PR is done. backport-done/1.16 The backport for Cilium 1.16.x for this PR is done. backport-done/1.17 The backport for Cilium 1.17.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.

4 participants
0