address review comments

Signed-off-by: Hui Kang <[email protected]>

docs/features/specification.md

@@ -115,8 +115,8 @@ spec:
       # down. Defaults to nil
       scaleDownDelayRevisionLimit: 2
 
-      # Adds a delay before scaling down the p preview replicaset if update is
-      # aborted. Default is 0, meaning preview replicaset  are not scaled down.
+      # Adds a delay before scaling down the preview replicaset if update is
+      # aborted. Default is 0, meaning preview replicaset won't be scaled down.
       AbortScaleDownDelaySeconds: 30
 
       # Anti Affinity configuration between desired and previous ReplicaSet.

rollout/replicaset.go

@@ -122,12 +122,11 @@ func (c *rolloutContext) reconcileNewReplicaSet() (bool, error) {
 
 	abortScaleDownDelaySeconds := time.Duration(defaults.GetAbortScaleDownDelaySecondsOrDefault(c.rollout))
 	if c.isScaleDownOnabort() {
-		c.log.Info("scale down on abort for the newRS")
+		c.log.Infof("Scale down new rs '%s' on abort", c.newRS.Name)
 
-		// if the newRS has scale down annotation and to be scaled down now, set
-		// Update newReplicasCount = int32(0)
+		// if the newRS has scale down annotation, check if it should be scaled down now
 		if scaleDownAtStr, ok := c.newRS.Annotations[v1alpha1.DefaultReplicaSetScaleDownDeadlineAnnotationKey]; ok {
-			c.log.Infof("New rs '%s' has scaledown delay annotation: %s", c.newRS.Name, scaleDownAtStr)
+			c.log.Infof("New rs '%s' has scaledown deadline annotation: %s", c.newRS.Name, scaleDownAtStr)
 			scaleDownAtTime, err := time.Parse(time.RFC3339, scaleDownAtStr)
 			if err != nil {
 				c.log.Warnf("Unable to read scaleDownAt label on rs '%s'", c.newRS.Name)