fix: deployment render should only mutate workloads created by yurtap… (

openyurtio#1974)

* fix: deployment render should only mutate workloads created by yurtappset v1alpha1

pkg/yurtmanager/webhook/deploymentrender/v1alpha1/deploymentrender_default.go

@@ -73,6 +73,9 @@ func (webhook *DeploymentRenderHandler) Default(ctx context.Context, obj runtime
 	var instance client.Object
 	switch app.Kind {
 	case "YurtAppSet":
+		if app.APIVersion != v1alpha1.SchemeGroupVersion.String() {
+			return nil
+		}
 		instance = &v1alpha1.YurtAppSet{}
 	case "YurtAppDaemon":
 		instance = &v1alpha1.YurtAppDaemon{}

pkg/yurtmanager/webhook/deploymentrender/v1alpha1/deploymentrender_webhook_test.go

@@ -118,6 +118,45 @@ var defaultAppDaemon = &v1alpha1.YurtAppDaemon{
 	},
 }
 
+var deploymentByYasv1beta1 = &appsv1.Deployment{
+	ObjectMeta: metav1.ObjectMeta{
+		Name:      "test1",
+		Namespace: "default",
+		OwnerReferences: []metav1.OwnerReference{{
+			APIVersion: "apps.openyurt.io/v1beta1",
+			Kind:       "YurtAppSet",
+			Name:       "yurtappset-patch",
+		}},
+		Labels: map[string]string{
+			"apps.openyurt.io/pool-name": "nodepool-test",
+		},
+	},
+	Status: appsv1.DeploymentStatus{},
+	Spec: appsv1.DeploymentSpec{
+		Replicas: &replica,
+		Selector: &metav1.LabelSelector{
+			MatchLabels: map[string]string{
+				"app": "test",
+			},
+		},
+		Template: corev1.PodTemplateSpec{
+			ObjectMeta: metav1.ObjectMeta{
+				Labels: map[string]string{
+					"app": "test",
+				},
+			},
+			Spec: corev1.PodSpec{
+				Containers: []corev1.Container{
+					{
+						Name:  "nginx",
+						Image: "nginx",
+					},
+				},
+			},
+		},
+	},
+}
+
 var defaultDeployment = &appsv1.Deployment{
 	ObjectMeta: metav1.ObjectMeta{
 		Name:      "test1",
@@ -328,6 +367,9 @@ func TestDeploymentRenderHandler_Default(t *testing.T) {
 			if err := webhook.Default(context.TODO(), daemonDeployment); err != nil {
 				t.Fatal(err)
 			}
+			if err := webhook.Default(context.TODO(), deploymentByYasv1beta1); err != nil {
+				t.Fatal(err)
+			}
 		})
 	}
 }