Merge pull request #1808 from adamreese/fix/missing-name

fix(tiller): add missing name to status response
This commit is contained in:
Adam Reese 2017-01-10 11:57:41 -08:00 committed by GitHub
commit 3e44f87c59
2 changed files with 8 additions and 1 deletions

View File

@ -227,7 +227,11 @@ func (s *ReleaseServer) GetReleaseStatus(c ctx.Context, req *services.GetRelease
}
sc := rel.Info.Status.Code
statusResp := &services.GetReleaseStatusResponse{Info: rel.Info, Namespace: rel.Namespace}
statusResp := &services.GetReleaseStatusResponse{
Name: rel.Name,
Namespace: rel.Namespace,
Info: rel.Info,
}
// Ok, we got the status of the release as we had jotted down, now we need to match the
// manifest we stashed away with reality from the cluster.

View File

@ -1153,6 +1153,9 @@ func TestGetReleaseStatus(t *testing.T) {
t.Errorf("Error getting release content: %s", err)
}
if res.Name != rel.Name {
t.Errorf("Expected name %q, got %q", rel.Name, res.Name)
}
if res.Info.Status.Code != release.Status_DEPLOYED {
t.Errorf("Expected %d, got %d", release.Status_DEPLOYED, res.Info.Status.Code)
}