Commit 0e7781c4 authored by Adam Reese's avatar Adam Reese

fix(tiller): add missing name to status response

parent d830bc19
...@@ -227,7 +227,11 @@ func (s *ReleaseServer) GetReleaseStatus(c ctx.Context, req *services.GetRelease ...@@ -227,7 +227,11 @@ func (s *ReleaseServer) GetReleaseStatus(c ctx.Context, req *services.GetRelease
} }
sc := rel.Info.Status.Code 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 // 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. // manifest we stashed away with reality from the cluster.
......
...@@ -1153,6 +1153,9 @@ func TestGetReleaseStatus(t *testing.T) { ...@@ -1153,6 +1153,9 @@ func TestGetReleaseStatus(t *testing.T) {
t.Errorf("Error getting release content: %s", err) 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 { if res.Info.Status.Code != release.Status_DEPLOYED {
t.Errorf("Expected %d, got %d", release.Status_DEPLOYED, res.Info.Status.Code) t.Errorf("Expected %d, got %d", release.Status_DEPLOYED, res.Info.Status.Code)
} }
......
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