Skip to content

Drop need for helm force update for now

DJ Mountney requested to merge remove-need-for-force into master

Followup to !167 (merged)

In the new api syntax, we can avoid the need to use helm upgrade --force for now if we match the matchLabels to what they are already set as under the hood. (As a result of the old labels)

I ran into this issue while testing an upgrade in the gitlab chart, which has a dependency on this one. Using --force did work nicely, so it's not the worst option, but for the gitlab chart we would need to do some additional work to communicate to users that they need to use it for this one update.

My preference would be to instead do the change in this MR for now, and improve the matchLabels in a major version.

Merge request reports