eks/irsa: fix DeploymentTook field update

Signed-off-by: Gyuho Lee <leegyuho@amazon.com>
aws · Feb 12, 2020 · a9ca6f0 · a9ca6f0
1 parent 24d2754
commit a9ca6f0
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/eks/irsa/irsa.go b/eks/irsa/irsa.go
@@ -1120,9 +1120,6 @@ func (ts *tester) waitDeployment() error {
 			}
 		}
 		if available && dresp.Status.ReadyReplicas >= ts.cfg.EKSConfig.AddOnIRSA.DeploymentReplicas {
-			ts.cfg.EKSConfig.AddOnIRSA.DeploymentTook = time.Since(ts.deploymentCreated)
-			ts.cfg.EKSConfig.AddOnIRSA.DeploymentTookString = ts.cfg.EKSConfig.AddOnIRSA.DeploymentTook.String()
-			ts.cfg.EKSConfig.Sync()
 			ready = true
 			break
 		}
@@ -1163,6 +1160,9 @@ func (ts *tester) waitOutputLogs() error {
 			zap.Int("current", cnt),
 		)
 		if cnt >= expects {
+			ts.cfg.EKSConfig.AddOnIRSA.DeploymentTook = time.Since(ts.deploymentCreated)
+			ts.cfg.EKSConfig.AddOnIRSA.DeploymentTookString = ts.cfg.EKSConfig.AddOnIRSA.DeploymentTook.String()
+			ts.cfg.EKSConfig.Sync()
 			break
 		}
 	}