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

release: introduce reuse-values functionality #359

Merged
merged 1 commit into from
Apr 4, 2020

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Apr 4, 2020

This commit is a fix of a regression bug, or faulty description of a
feature.

In a previous version of the Helm Operator the ResetValues option
was introduced to make Helm reset the values to the chart defaults
during an upgrade. In a later version this behaviour changed, and
values were reset by default (as expected given a HelmRelease
resource should be the sole truth).

This commit adds the possibility to reuse values from previous
releases by explicitly setting .spec.resetValues to false. This
will make an upgrade made by the operator behave as if the
--reuse-values flag would be set on an helm upgrade command.

Removing this defintion from the HelmRelease, or setting
.resetValues to true will make the operator change back to its
default behaviour during subsquent upgrades made for the release.

Fixes #352

This commit is a fix of a regression bug, or faulty description of a
feature.

In a previous version of the Helm Operator the `ResetValues` option
was introduced to make Helm reset the values to the chart defaults
during an upgrade. In a later version this behaviour changed, and
values were reset by default (as expected given a HelmRelease
resource should be the sole truth).

This commit adds the possibility to reuse values from previous
releases by explicitly setting `.spec.resetValues` to `false`. This
will make an upgrade made by the operator behave as if the
`--reuse-values` flag would be set on an `helm upgrade` command.

Removing this defintion from the HelmRelease, or setting
`.resetValues` to true will make the operator change back to its
default behaviour during subsquent upgrades made for the release.
@hiddeco hiddeco requested a review from stefanprodan April 4, 2020 09:57
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 merged commit 71ae411 into master Apr 4, 2020
@hiddeco hiddeco deleted the enhancement/reuse-values branch April 4, 2020 10:33
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.

resetValues doesn
2 participants