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

fix: helmfile apply and sync should properly delete releases (#555) 

Since #526, `helmfile apply` have been really able to detect deletion of the last release only, and `sync` has been unable to mark releases with `installed: false` for removal.

Fixes #554
This commit is contained in:
KUOKA Yusuke 2019-04-13 21:47:27 +09:00 committed by GitHub
parent f53c66749e
commit 04a9ea1d2a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
state/state.go
View File

@ -206,9 +206,6 @@ type syncPrepareResult struct {
func (st *HelmState) prepareSyncReleases(helm helmexec.Interface, additionalValues []string, concurrency int) ([]syncPrepareResult, []error) { func (st *HelmState) prepareSyncReleases(helm helmexec.Interface, additionalValues []string, concurrency int) ([]syncPrepareResult, []error) {
releases := []*ReleaseSpec{} releases := []*ReleaseSpec{}
for i, _ := range st.Releases { for i, _ := range st.Releases {
if !st.Releases[i].Desired() {
continue
}
releases = append(releases, &st.Releases[i]) releases = append(releases, &st.Releases[i])
} }
@ -297,7 +294,9 @@ func (st *HelmState) DetectReleasesToBeDeleted(helm helmexec.Interface) ([]*Rele
if err != nil { if err != nil {
return nil, err return nil, err
} else if installed { } else if installed {
detected = append(detected, &release) // Otherwise `release` messed up(https://github.com/roboll/helmfile/issues/554)
r := release
detected = append(detected, &r)
} }
} }
} }