From 30899e0072eda0a913cf165b8a5567c6444bb013 Mon Sep 17 00:00:00 2001 From: Nate Broyles Date: Tue, 16 Mar 2021 13:34:32 -0400 Subject: [PATCH] Clarify pod update comment --- pkg/controller/controller.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/pkg/controller/controller.go b/pkg/controller/controller.go index 6dee89c..f75988f 100644 --- a/pkg/controller/controller.go +++ b/pkg/controller/controller.go @@ -583,11 +583,11 @@ func (c *M3DBController) handleClusterUpdate(cluster *myspec.M3DBCluster) error } // Using an OnDelete strategy, we have to update nodes if: - // - a statefulset update just happened + // - a statefulset update has happened OR // - we're already in the middle of a rollout - // * because nodes are rolled out in chunks this can happen in many iterations - // Therefore, always check to see if pods need to be updated and return from this loop - // if the statefulset or pods were updated. If a rollout is finished or there has not + // * because nodes are rolled out in chunks, this can happen over many iterations + // Therefore, check to see if pods need to be updated and return from this loop + // if pods were updated. If a rollout is finished or there has not // been a change, this call is a no-op. if onDeleteUpdateStrategy { nodesUpdated, err := c.updateStatefulSetPods(cluster, actual)