Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

release: support retrying rollbacks #252

Merged
merged 2 commits into from
Jan 30, 2020
Merged

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Jan 27, 2020

This commit adds support for retrying rollbacks using .spec.rollback.retry until the configured .spec.rolback.maxRetries count is reached (defaults to 5, setting it to 0 means infinite).

This makes it possible to for example recover from transient errors that were the cause of the rollback, and not a persistent chart or values failure. The reset of the counter happens after a successful
upgrade or a change (e.g. chart change, .spec change).

Fixes #241

@hiddeco hiddeco force-pushed the enhancement/retry-rollback branch 3 times, most recently from e894be6 to cbaf9ac Compare January 27, 2020 16:17
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@hiddeco hiddeco force-pushed the enhancement/retry-rollback branch 2 times, most recently from 6e37143 to 71403ae Compare January 30, 2020 08:46
This commit adds support for retrying rollbacks using
`.spec.rollback.retry` until the configured `.spec.rolback.maxRetries`
count is reached (defaults to 5, setting it to 0 means infinite). The
reset of the counter happens after a successful upgrade or a change
(e.g. chart change, `.spec` change).

This makes it possible to for example recover from transient errors
that were the cause of the rollback, and not a persistent chart or
values failure.
@hiddeco hiddeco marked this pull request as ready for review January 30, 2020 08:47
@hiddeco hiddeco merged commit 9906a23 into master Jan 30, 2020
@hiddeco hiddeco deleted the enhancement/retry-rollback branch January 30, 2020 10:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Retry upgrade after rollback without change to resource
2 participants