Change checkIfPodsAreUpdated default to true

We should switch default WaitForControllerPodsRunning behavior to the right one: where we correctly distinguish updated pods from state ones.
wangxy518 · Feb 11, 2020 · ef0c6d8 · ef0c6d8
1 parent e2316e0
commit ef0c6d8
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/clusterloader2/pkg/measurement/common/wait_for_controlled_pods.go b/clusterloader2/pkg/measurement/common/wait_for_controlled_pods.go
@@ -110,8 +110,7 @@ func (w *waitForControlledPodsRunningMeasurement) Execute(config *measurement.Me
 		if err != nil {
 			return nil, err
 		}
-		// TODO(mborsz): Change default to true.
-		w.checkIfPodsAreUpdated, err = util.GetBoolOrDefault(config.Params, "checkIfPodsAreUpdated", false)
+		w.checkIfPodsAreUpdated, err = util.GetBoolOrDefault(config.Params, "checkIfPodsAreUpdated", true)
 		if err != nil {
 			return nil, err
 		}