public_git
/
helmfile
mirror of https://github.com/helmfile/helmfile.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Do delete decrypted secret on sync command (#82) 

Due to not passing around the pointer into channels and through the functions, we were not getting back the generatedValues. This fixes that.
This commit is contained in:
Shane Starcher 2018-04-03 21:22:17 -04:00 committed by KUOKA Yusuke
parent 813fd4bfc7
commit b4f9c1ce79
2 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
state/state.go
View File

@ -123,11 +123,10 @@ func renderTemplateString(s string) (string, error) {
return tplString.String(), nil
}
func (state *HelmState) applyDefaultsTo(spec ReleaseSpec) ReleaseSpec {
func (state *HelmState) applyDefaultsTo(spec *ReleaseSpec) {
if state.Namespace != "" {
spec.Namespace = state.Namespace
}
return spec
}
func (state *HelmState) SyncRepos(helm helmexec.Interface) []error {
@ -156,7 +155,7 @@ func (state *HelmState) SyncRepos(helm helmexec.Interface) []error {
func (state *HelmState) SyncReleases(helm helmexec.Interface, additionalValues []string, workerLimit int) []error {
errs := []error{}
jobQueue := make(chan ReleaseSpec)
jobQueue := make(chan *ReleaseSpec)
doneQueue := make(chan bool)
errQueue := make(chan error)
@ -166,8 +165,8 @@ func (state *HelmState) SyncReleases(helm helmexec.Interface, additionalValues [
for w := 1; w <= workerLimit; w++ {
go func() {
for release := range jobQueue {
releaseWithDefaults := state.applyDefaultsTo(release)
flags, flagsErr := flagsForRelease(helm, state.BaseChartPath, &releaseWithDefaults)
state.applyDefaultsTo(release)
flags, flagsErr := flagsForRelease(helm, state.BaseChartPath, release)
if flagsErr != nil {
errQueue <- flagsErr
doneQueue <- true
@ -199,8 +198,8 @@ func (state *HelmState) SyncReleases(helm helmexec.Interface, additionalValues [
}
go func() {
for _, release := range state.Releases {
jobQueue <- release
for i := 0; i < len(state.Releases); i++ {
jobQueue <- &state.Releases[i]
}
close(jobQueue)
}()

4
state/state_test.go
View File

@ -216,8 +216,8 @@ func TestHelmState_applyDefaultsTo(t *testing.T) {
Repositories: tt.fields.Repositories,
Releases: tt.fields.Releases,
}
if got := state.applyDefaultsTo(tt.args.spec); !reflect.DeepEqual(got, tt.want) {
t.Errorf("HelmState.applyDefaultsTo() = %v, want %v", got, tt.want)
if state.applyDefaultsTo(&tt.args.spec); !reflect.DeepEqual(tt.args.spec, tt.want) {
t.Errorf("HelmState.applyDefaultsTo() = %v, want %v", tt.args.spec, tt.want)
}
})
}