Danny Shemesh 71bb7354e7 Fix: populate .Values regardless of prestate success (#1202) ...

This commit proposes a potential solution for
https://github.com/roboll/helmfile/issues/1201

The gist is that, if prestate rendering fails, for any reason,
we do not populate the .Values in the second pass renderer.

I think that what have been expected in this case is to populate the
.Values irregardless.

pkg/app/two_pass_renderer.go
- Migrated to use finalEnv.GetMergedValues()

pkg/environment/environment.go
- Introduced GetMergedValues, which merges the environment's defaults
and current values, and then casts the keys to string;
This was previously defined in HelmState.Values() - however, as this
method is only concerned with the environment, I think it's more
appropriate for it to sit here.

pkg/state/state_exec_tmpl.go
- Extracted out HelmState.Values() to environment.go, see above

2020-04-17 09:18:01 +09:00

..

environment_test.go

fix: merge environment values by ovewriting with empty values (#1162)

2020-03-29 20:47:23 +09:00

environment.go

Fix: populate .Values regardless of prestate success (#1202)

2020-04-17 09:18:01 +09:00