Erin Call b6ba856c31 Pass CleanupOnFail to the Upgrade Step [#65] ...

I don't love the mismatch between the helm.Config field (CleanupOnFail)
and the setting name (cleanup_failed_upgrade). I do think the setting
name should contain "upgrade" since it's specific to the upgrade command,
but if I make the config field CleanupFailedUpgrade, it becomes the new
longest field name, and gofmt creates a bunch of churn. Is that a good
enough reason...?

2020-01-07 12:56:51 -08:00

3.9 KiB

Raw Permalink Blame History

View Raw