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

[v1.8] daemon: store Cilium's configuration in a file #16092

Merged
merged 1 commit into from
May 12, 2021

Conversation

aanm
Copy link
Member

@aanm aanm commented May 11, 2021

Once this PR is merged, you can update the PR labels via:

$ for pr in 16017; do contrib/backporting/set-labels.py $pr done 1.9; done

@aanm aanm requested a review from a team as a code owner May 11, 2021 19:42
@maintainer-s-little-helper maintainer-s-little-helper bot added the kind/backports This PR provides functionality previously merged into master. label May 11, 2021
@aanm aanm force-pushed the pr/store-config-locally/v1.8 branch 2 times, most recently from 2f8464d to fd588da Compare May 11, 2021 19:44
@aanm
Copy link
Member Author

aanm commented May 11, 2021

test-backport-1.8

[ upstream commit 2e0603b ]

To make it easier debug or share configurations across users and
developers, Cilium will store its viper.Config as well as agent's
configuration in files under `/var/run/cilium`. It will store up to
the last 3 previous configurations.

Another use case for it will be to check the previous configuration run
by Cilium in case certain steps need to be executed, for example, to
clean up state left from a previous run that will not be cleaned up by a
new set of flags.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/store-config-locally/v1.8 branch from fd588da to 30cd625 Compare May 12, 2021 16:30
@aanm
Copy link
Member Author

aanm commented May 12, 2021

test-backport-1.8

@aanm
Copy link
Member Author

aanm commented May 12, 2021

hit #16040

@joestringer joestringer merged commit 7a3990c into cilium:v1.8 May 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/backports This PR provides functionality previously merged into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants