Consider container names in Statefulset diffs

This includes a comparison on container names being equal in the decision of whether a Statefulset has been updated.
zalando · Jan 15, 2018 · d8e62b9 · d8e62b9
1 parent 8c9033d
commit d8e62b9
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/pkg/cluster/cluster.go b/pkg/cluster/cluster.go
@@ -311,6 +311,10 @@ func (c *Cluster) compareStatefulSetWith(statefulSet *v1beta1.StatefulSet) *comp
 	} else {
 		for index, container1 := range c.Statefulset.Spec.Template.Spec.Containers {
 			container2 := statefulSet.Spec.Template.Spec.Containers[index]
+			if container1.Name != container2.Name {
+				needsRollUpdate = true
+				reasons = append(reasons, fmt.Sprintf("new statefulset's container %d name doesn't match the current one", index))
+			}
 			if container1.Image != container2.Image {
 				needsRollUpdate = true
 				reasons = append(reasons, fmt.Sprintf("new statefulset's container %d image doesn't match the current one", index))