search

blackducksoftware / synopsys-operator

Archived and moved this to https://github.com/blackducksoftware/synopsysctl
Apache License 2.0
10 stars 14 forks source link

fix helm upgrade with new version of charts #1304

Closed msenmurugan closed 4 years ago

msenmurugan commented 4 years ago

resolves CLOUD-697

As per Helm, during upgrade to new version, we need to use reset-values flags, so that it will set the new values from the new chart and during update/override flags of an existing version, we need use reuse-values.

I have fixed this issue by always get the set values and then use reset values for both upgrading and updating scenario

mphammer commented 4 years ago

/lgtm

operator-r2d2 commented 4 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mphammer, msenmurugan, yashbhutwala

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/blackducksoftware/synopsys-operator/blob/develop/OWNERS)~~ [mphammer,msenmurugan,yashbhutwala] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment