pelotech / drone-helm3

Plugin for drone to deploy helm charts using helm3
Apache License 2.0
31 stars 36 forks source link

Warn if helm2-specific settings are present #10

Closed ErinCall closed 4 years ago

ErinCall commented 4 years ago

drone-helm has a handful of settings that correspond to helm2 commands/flags that don't exist in helm3. If we see those env vars, it probably means they were left over during an upgrade from drone-helm. drone-helm3 should emit a warning that advises the plugin consumer to remove those config settings. We don't want a user to experience the frustration of "I've set this setting, why isn't it being applied??"

Deprecated env vars:

Remember to look for both the prefixed and non-prefixed forms (e.g. $PURGE and $PLUGIN_PURGE).