set default advanced deployment strategy (#176)

Signed-off-by: mingzhou.swx <[email protected]> Co-authored-by: mingzhou.swx <[email protected]>
openkruise · Dec 19, 2023 · 8620408 · 8620408
1 parent e19a89c
commit 8620408
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/pkg/controller/batchrelease/control/partitionstyle/deployment/control.go b/pkg/controller/batchrelease/control/partitionstyle/deployment/control.go
@@ -95,6 +95,7 @@ func (rc *realController) Initialize(release *v1beta1.BatchRelease) error {
 		RollingStyle:  v1alpha1.PartitionRollingStyle,
 		RollingUpdate: rollingUpdate,
 	}
+	v1alpha1.SetDefaultDeploymentStrategy(&strategy)
 
 	d := rc.object.DeepCopy()
 	patchData := patch.NewDeploymentPatch()

diff --git a/pkg/webhook/workload/mutating/workload_update_handler.go b/pkg/webhook/workload/mutating/workload_update_handler.go
@@ -285,6 +285,7 @@ func (h *WorkloadHandler) handleDeployment(newObj, oldObj *apps.Deployment) (boo
 				modified = true
 				strategy.Paused = true
 			}
+			appsv1alpha1.SetDefaultDeploymentStrategy(&strategy)
 			setDeploymentStrategyAnnotation(strategy, newObj)
 		default:
 			// Do not allow to modify strategy as Recreate during rolling