diff --git a/pkg/tiller/release_update.go b/pkg/tiller/release_update.go index cb8b577925d7a7f24e0905fa0de694fbe21ba4c6..6f5d37331222c8613e97f3fa81c73ad3e33f8dcb 100644 --- a/pkg/tiller/release_update.go +++ b/pkg/tiller/release_update.go @@ -80,7 +80,7 @@ func (s *ReleaseServer) prepareUpdate(req *services.UpdateReleaseRequest) (*rele return nil, nil, err } - // If new values were not supplied in the upgrade, re-use the existing values. + // determine if values will be reused if err := s.reuseValues(req, currentRelease); err != nil { return nil, nil, err } diff --git a/pkg/tiller/release_update_test.go b/pkg/tiller/release_update_test.go index 0189201d163bd2ebf627daeedf95834ec7678578..a1b9a4bff1b33d31a39e16aa4b31c44e46dbd106 100644 --- a/pkg/tiller/release_update_test.go +++ b/pkg/tiller/release_update_test.go @@ -260,7 +260,7 @@ func TestUpdateRelease_ReuseValues(t *testing.T) { t.Errorf("Expected chart values to be %q, got %q", expect, res.Release.Chart.Values.Raw) } // This should have the newly-passed overrides and any other computed values. `name: value` comes from release Config via releaseStub() - expect = "name: value\nname2: val2" + expect = "name: value\nname2: val2\n" if res.Release.Config != nil && res.Release.Config.Raw != expect { t.Errorf("Expected request config to be %q, got %q", expect, res.Release.Config.Raw) }