From 87f7e652ab06cd8f41b108835726abca471ebd1f Mon Sep 17 00:00:00 2001 From: Parship Chowdhury Date: Mon, 28 Jul 2025 08:17:58 +0530 Subject: [PATCH] Fix workload webhook filtering: patch label bug and add Kruise StatefulSet support (#299) * Fix workload type label patching bug and add Kruise StatefulSet support Signed-off-by: Parship Chowdhury * added objectSelector to filter workloads by rollout label Signed-off-by: Parship Chowdhury * reverted the changes in manifest Signed-off-by: Parship Chowdhury --------- Signed-off-by: Parship Chowdhury --- pkg/controller/rollout/rollout_status.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/pkg/controller/rollout/rollout_status.go b/pkg/controller/rollout/rollout_status.go index 066014f..389f59d 100755 --- a/pkg/controller/rollout/rollout_status.go +++ b/pkg/controller/rollout/rollout_status.go @@ -298,10 +298,12 @@ func (r *RolloutReconciler) patchWorkloadRolloutWebhookLabel(rollout *v1beta1.Ro workloadType = util.DeploymentType case util.ControllerKindSts.Kind: workloadType = util.StatefulSetType + case util.ControllerKruiseKindSts.Kind, util.ControllerKruiseOldKindSts.Kind: + workloadType = util.StatefulSetType case util.ControllerKruiseKindDS.Kind: workloadType = util.DaemonSetType } - if workload.Annotations[util.WorkloadTypeLabel] == "" && workloadType != "" { + if workload.Labels[util.WorkloadTypeLabel] == "" && workloadType != "" { workloadGVK := schema.FromAPIVersionAndKind(workload.APIVersion, workload.Kind) obj := util.GetEmptyWorkloadObject(workloadGVK) obj.SetNamespace(workload.Namespace)