search

oracle-cne / ocne

The Oracle Cloud Native Environment CLI
Universal Permissive License v1.0
1 stars 0 forks source link

Add option to "ocne app update" to reset the overrides to the ones built into the chart #16

Closed mgianatagh closed 4 weeks ago

mgianatagh commented 1 month ago

Summary The ocne application update command reuses the helm override values that were used for the current deployment. Sometimes this is not desired.

For example, if the existing deployment of ingress-nginx-controller used the helm overrides below.

controller:
  image:
    repository: olcne/ingress-nginx-controller
    tag: "v1.9.5"

An update of this component to v1.9.6 would require the same overrides (which are already set in the values.yaml of the chart):

ocne application update --release ingress-controller --namespace ingress-nginx --version 1.9.6 --values - <<EOF
controller:
  image:
    repository: olcne/ingress-nginx-controller
    tag: "v1.9.6"

It is desirable to instead perform the update and ignore the previous overrides, and let them come from the values.yaml of the chart.

Proposal

Add the option --reset-values to reset the values to the ones built into the chart. Any overrides specified with the update command are treated as a fresh new set of overrides.

For the example provided above, the update would be done by:

ocne application update --release ingress-controller --namespace ingress-nginx --version 1.9.6 --reset-values