fix: `helmfile delete` should not stop on uninstalled release(s) (#509)
Resolves #481
This commit is contained in:
		
							parent
							
								
									d3c5417177
								
							
						
					
					
						commit
						870d33f418
					
				| 
						 | 
					@ -708,7 +708,14 @@ func (st *HelmState) DeleteReleases(helm helmexec.Interface, purge bool) []error
 | 
				
			||||||
		if purge {
 | 
							if purge {
 | 
				
			||||||
			flags = append(flags, "--purge")
 | 
								flags = append(flags, "--purge")
 | 
				
			||||||
		}
 | 
							}
 | 
				
			||||||
 | 
							installed, err := isReleaseInstalled(helm, release)
 | 
				
			||||||
 | 
							if err != nil {
 | 
				
			||||||
 | 
								return err
 | 
				
			||||||
 | 
							}
 | 
				
			||||||
 | 
							if installed {
 | 
				
			||||||
			return helm.DeleteRelease(release.Name, flags...)
 | 
								return helm.DeleteRelease(release.Name, flags...)
 | 
				
			||||||
 | 
							}
 | 
				
			||||||
 | 
							return nil
 | 
				
			||||||
	})
 | 
						})
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -80,6 +80,9 @@ info "Deleting release"
 | 
				
			||||||
${helmfile} -f ${dir}/happypath.yaml delete
 | 
					${helmfile} -f ${dir}/happypath.yaml delete
 | 
				
			||||||
${helm} status --namespace=${test_ns} httpbin &> /dev/null && fail "release should not exist anymore after a delete"
 | 
					${helm} status --namespace=${test_ns} httpbin &> /dev/null && fail "release should not exist anymore after a delete"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					info "Ensuring \"helmfile delete\" doesn't fail when no releases installed"
 | 
				
			||||||
 | 
					${helmfile} -f ${dir}/happypath.yaml delete || fail "\"helmfile delete\" shouldn't fail when there are no installed releases"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
test_pass "happypath"
 | 
					test_pass "happypath"
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue