Skip to content

Commit cb2f9f8

Browse files
Merge pull request #18549 from mfojtik/oc-status-02
Automatic merge from submit-queue. cli: fix status for standalone deployment Fixing: https://bugzilla.redhat.com/show_bug.cgi?id=1540560
2 parents 8581458 + a6cb524 commit cb2f9f8

File tree

2 files changed

+23
-13
lines changed

2 files changed

+23
-13
lines changed

pkg/oc/cli/describe/projectstatus.go

+11-1
Original file line numberDiff line numberDiff line change
@@ -200,6 +200,9 @@ func (d *ProjectStatusDescriber) Describe(namespace, name string) (string, error
200200
standaloneDCs, coveredByDCs := graphview.AllDeploymentConfigPipelines(g, coveredNodes)
201201
coveredNodes.Insert(coveredByDCs.List()...)
202202

203+
standaloneDeployments, coveredByDeployments := graphview.AllDeployments(g, coveredNodes)
204+
coveredNodes.Insert(coveredByDeployments.List()...)
205+
203206
standaloneRCs, coveredByRCs := graphview.AllReplicationControllers(g, coveredNodes)
204207
coveredNodes.Insert(coveredByRCs.List()...)
205208

@@ -300,6 +303,12 @@ func (d *ProjectStatusDescriber) Describe(namespace, name string) (string, error
300303
return graphview.MaxRecentContainerRestartsForRC(g, rc)
301304
})...)
302305
}
306+
for _, standaloneDeployment := range standaloneDeployments {
307+
fmt.Fprintln(out)
308+
printLines(out, indent, 0, describeDeploymentInServiceGroup(f, standaloneDeployment, func(rs *kubegraph.ReplicaSetNode) int32 {
309+
return graphview.MaxRecentContainerRestartsForRS(g, rs)
310+
})...)
311+
}
303312

304313
for _, standaloneImage := range standaloneImages {
305314
fmt.Fprintln(out)
@@ -312,6 +321,7 @@ func (d *ProjectStatusDescriber) Describe(namespace, name string) (string, error
312321
fmt.Fprintln(out)
313322
printLines(out, indent, 0, describeRCInServiceGroup(f, standaloneRC.RC)...)
314323
}
324+
315325
for _, standaloneRS := range standaloneRSs {
316326
fmt.Fprintln(out)
317327
printLines(out, indent, 0, describeRSInServiceGroup(f, standaloneRS.RS)...)
@@ -550,7 +560,7 @@ func (f namespacedFormatter) ResourceName(obj interface{}) string {
550560
case *kubegraph.ReplicationControllerNode:
551561
return namespaceNameWithType("rc", t.ReplicationController.Name, t.ReplicationController.Namespace, f.currentNamespace, f.hideNamespace)
552562
case *kubegraph.ReplicaSetNode:
553-
return namespaceNameWithType("rc", t.ReplicaSet.Name, t.ReplicaSet.Namespace, f.currentNamespace, f.hideNamespace)
563+
return namespaceNameWithType("rs", t.ReplicaSet.Name, t.ReplicaSet.Namespace, f.currentNamespace, f.hideNamespace)
554564
case *kubegraph.HorizontalPodAutoscalerNode:
555565
return namespaceNameWithType("hpa", t.HorizontalPodAutoscaler.Name, t.HorizontalPodAutoscaler.Namespace, f.currentNamespace, f.hideNamespace)
556566
case *kubegraph.StatefulSetNode:

pkg/oc/graph/kubegraph/nodes/nodes.go

+12-12
Original file line numberDiff line numberDiff line change
@@ -117,33 +117,33 @@ func EnsureReplicationControllerNode(g osgraph.MutableUniqueGraph, rc *kapi.Repl
117117

118118
// EnsureReplicaSetNode adds a graph node for the ReplicaSet if it does not already exist.
119119
func EnsureReplicaSetNode(g osgraph.MutableUniqueGraph, rs *extensions.ReplicaSet) *ReplicaSetNode {
120-
rcNodeName := ReplicaSetNodeName(rs)
121-
rcNode := osgraph.EnsureUnique(g,
122-
rcNodeName,
120+
rsNodeName := ReplicaSetNodeName(rs)
121+
rsNode := osgraph.EnsureUnique(g,
122+
rsNodeName,
123123
func(node osgraph.Node) graph.Node {
124124
return &ReplicaSetNode{node, rs, true}
125125
},
126126
).(*ReplicaSetNode)
127127

128-
rcSpecNode := EnsureReplicaSetSpecNode(g, &rs.Spec, rs.Namespace, rcNodeName)
129-
g.AddEdge(rcNode, rcSpecNode, osgraph.ContainsEdgeKind)
128+
rcSpecNode := EnsureReplicaSetSpecNode(g, &rs.Spec, rs.Namespace, rsNodeName)
129+
g.AddEdge(rsNode, rcSpecNode, osgraph.ContainsEdgeKind)
130130

131-
return rcNode
131+
return rsNode
132132
}
133133

134134
func EnsureReplicaSetSpecNode(g osgraph.MutableUniqueGraph, rsSpec *extensions.ReplicaSetSpec, namespace string, ownerName osgraph.UniqueName) *ReplicaSetSpecNode {
135-
rcSpecName := ReplicaSetSpecNodeName(rsSpec, ownerName)
136-
rcSpecNode := osgraph.EnsureUnique(g,
137-
rcSpecName,
135+
rsSpecName := ReplicaSetSpecNodeName(rsSpec, ownerName)
136+
rsSpecNode := osgraph.EnsureUnique(g,
137+
rsSpecName,
138138
func(node osgraph.Node) graph.Node {
139139
return &ReplicaSetSpecNode{node, rsSpec, namespace, ownerName}
140140
},
141141
).(*ReplicaSetSpecNode)
142142

143-
ptSpecNode := EnsurePodTemplateSpecNode(g, &rsSpec.Template, namespace, rcSpecName)
144-
g.AddEdge(rcSpecNode, ptSpecNode, osgraph.ContainsEdgeKind)
143+
ptSpecNode := EnsurePodTemplateSpecNode(g, &rsSpec.Template, namespace, rsSpecName)
144+
g.AddEdge(rsSpecNode, ptSpecNode, osgraph.ContainsEdgeKind)
145145

146-
return rcSpecNode
146+
return rsSpecNode
147147
}
148148

149149
func FindOrCreateSyntheticReplicationControllerNode(g osgraph.MutableUniqueGraph, rc *kapi.ReplicationController) *ReplicationControllerNode {

0 commit comments

Comments
 (0)