fix bug: allow optional filed max unavilable in ads, and set default …

…value as 1 Signed-off-by: ABNER-1 <abner199709@gmail.com> Signed-off-by: yuanyuxing <yuanyuxing@bilibili.com>
openkruise · Jun 27, 2022 · 45c8ce2 · 45c8ce2
1 parent 345c292
commit 45c8ce2
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/pkg/controller/daemonset/daemonset_util.go b/pkg/controller/daemonset/daemonset_util.go
@@ -254,7 +254,10 @@ func unavailableCount(ds *appsv1alpha1.DaemonSet, numberToSchedule int) (int, er
 	}
 	r := ds.Spec.UpdateStrategy.RollingUpdate
 	if r == nil {
-		return 0, nil
+		return 1, nil
+	}
+	if r.MaxUnavailable == nil {
+		return 1, nil
 	}
 	return intstrutil.GetScaledValueFromIntOrPercent(r.MaxUnavailable, numberToSchedule, true)
 }