fix: add missing step - set app label when creating resources

Signed-off-by: minhthong582000 <[email protected]>

gitops/VERSION

@@ -1 +1 @@
-v0.1.0-alpha.10
+v0.1.0-alpha.11

gitops/internal/controller/controller.go

@@ -234,6 +234,15 @@ func (c *Controller) createResources(ctx context.Context, app *v1alpha1.Applicat
 		return fmt.Errorf("error getting resources with label: %s, %s", label, err)
 	}
 
+	// Set the label for the generated resources
+	label = map[string]string{
+		common.LabelKeyAppInstance: app.Name,
+	}
+	err = c.k8sUtil.SetLabelsForResources(generatedResources, label)
+	if err != nil {
+		return fmt.Errorf("error setting labels for resources: %s", err)
+	}
+
 	// Calculate diff
 	log.Infof("Diffing resources for application %s", app.Name)
 	diff, err := c.k8sUtil.DiffResources(currentResources, generatedResources)

gitops/internal/controller/controller_test.go

@@ -81,6 +81,7 @@ spec:
 				mock.EXPECT().GenerateManifests(gomock.Any()).Return(nil, nil)
 				mock.EXPECT().GetResourceWithLabel(gomock.Any()).Return(nil, nil)
 				mock.EXPECT().DiffResources(gomock.Any(), gomock.Any()).Return(true, nil)
+				mock.EXPECT().SetLabelsForResources(gomock.Any(), gomock.Any()).Return(nil)
 				mock.EXPECT().ApplyResources(gomock.Any()).Return(nil)
 				return mock
 			}(),
@@ -108,6 +109,7 @@ spec:
 				mock := k8sUtilMock.NewMockK8s(ctrl)
 				mock.EXPECT().GenerateManifests(gomock.Any()).Return(nil, nil)
 				mock.EXPECT().GetResourceWithLabel(gomock.Any()).Return(nil, nil)
+				mock.EXPECT().SetLabelsForResources(gomock.Any(), gomock.Any()).Return(nil)
 				mock.EXPECT().DiffResources(gomock.Any(), gomock.Any()).Return(false, nil)
 				return mock
 			}(),