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

install: Add --rollback option #566

Merged
merged 1 commit into from
Oct 6, 2021

Conversation

jrajahalme
Copy link
Member

@jrajahalme jrajahalme commented Oct 4, 2021

Add an option to control rollback on failed install. This allows
install failure debugging e.g. via artifacts collected by the CI by
passing '--rollback=false' option to 'cilium install'. With rollback
the resources created by a failed install are deleted so they can not
be inspected after the fact.

Signed-off-by: Jarno Rajahalme [email protected]

@jrajahalme jrajahalme added the area/CI Continuous Integration testing issue or flake label Oct 4, 2021
@jrajahalme jrajahalme requested a review from a team as a code owner October 4, 2021 15:10
@jrajahalme jrajahalme temporarily deployed to ci October 4, 2021 15:10 Inactive
internal/cli/cmd/install.go Outdated Show resolved Hide resolved
@jrajahalme jrajahalme force-pushed the pr/jrajahalme/install-skip-rollback branch from cf77e3d to 5418c15 Compare October 4, 2021 15:21
@jrajahalme jrajahalme temporarily deployed to ci October 4, 2021 15:21 Inactive
@jrajahalme jrajahalme changed the title install: Add --skip-rollback option install: Add --rollback option Oct 4, 2021
Add an option to control rollback on failed install. This allows
install failure debugging e.g. via artifacts collected by the CI by
passing '--rollback=false' option to 'cilium install'. With rollback
the resources created by a failed install are deleted so they can not
be inspected after the fact.

Signed-off-by: Jarno Rajahalme <[email protected]>
@jrajahalme jrajahalme force-pushed the pr/jrajahalme/install-skip-rollback branch from 5418c15 to 6fd7ca9 Compare October 6, 2021 09:37
@jrajahalme jrajahalme temporarily deployed to ci October 6, 2021 09:37 Inactive
@tklauser tklauser merged commit 15a544b into master Oct 6, 2021
@tklauser tklauser deleted the pr/jrajahalme/install-skip-rollback branch October 6, 2021 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants