Commit 881153cc authored by Adam Reese's avatar Adam Reese Committed by GitHub

Merge pull request #1812 from adamreese/fix/1807-purge-error

fix(tiller): don't update purged releases
parents c8af660b 6600fcb0
...@@ -981,13 +981,16 @@ func (s *ReleaseServer) UninstallRelease(c ctx.Context, req *services.UninstallR ...@@ -981,13 +981,16 @@ func (s *ReleaseServer) UninstallRelease(c ctx.Context, req *services.UninstallR
} }
} }
rel.Info.Status.Code = release.Status_DELETED
if req.Purge { if req.Purge {
if err := s.purgeReleases(rels...); err != nil { err := s.purgeReleases(rels...)
if err != nil {
log.Printf("uninstall: Failed to purge the release: %s", err) log.Printf("uninstall: Failed to purge the release: %s", err)
} }
return res, err
} }
rel.Info.Status.Code = release.Status_DELETED
if err := s.env.Releases.Update(rel); err != nil { if err := s.env.Releases.Update(rel); err != nil {
log.Printf("uninstall: Failed to store updated release: %s", err) log.Printf("uninstall: Failed to store updated release: %s", err)
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment