Fix condition error of ManifestApplied.

Signed-off-by: xuezhaojun <[email protected]>
open-cluster-management-io · Jun 3, 2024 · e85a301 · e85a301
1 parent e24923d
commit e85a301
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 9 deletions.
diff --git a/pkg/addonmanager/controllers/agentdeploy/controller.go b/pkg/addonmanager/controllers/agentdeploy/controller.go
@@ -391,15 +391,6 @@ func (c *addonDeployController) buildDeployManifestWorksFunc(addonWorkBuilder *a
 			})
 			return nil, nil, err
 		}
-		if len(objects) == 0 {
-			meta.SetStatusCondition(&addon.Status.Conditions, metav1.Condition{
-				Type:    appliedType,
-				Status:  metav1.ConditionTrue,
-				Reason:  addonapiv1alpha1.AddonManifestAppliedReasonManifestsApplied,
-				Message: "no manifest need to apply",
-			})
-			return nil, nil, nil
-		}
 
 		// this is to retrieve the intended mode of the addon.
 		var mode string
@@ -425,6 +416,14 @@ func (c *addonDeployController) buildDeployManifestWorksFunc(addonWorkBuilder *a
 			})
 			return nil, nil, err
 		}
+		if len(appliedWorks) == 0 {
+			meta.SetStatusCondition(&addon.Status.Conditions, metav1.Condition{
+				Type:    appliedType,
+				Status:  metav1.ConditionTrue,
+				Reason:  addonapiv1alpha1.AddonManifestAppliedReasonManifestsApplied,
+				Message: "no manifest need to apply",
+			})
+		}
 		return appliedWorks, deleteWorks, nil
 	}
 }

diff --git a/pkg/addonmanager/controllers/agentdeploy/hosted_sync_test.go b/pkg/addonmanager/controllers/agentdeploy/hosted_sync_test.go
@@ -237,6 +237,20 @@ func TestHostingReconcile(t *testing.T) {
 				if meta.IsStatusConditionFalse(addOn.Status.Conditions, addonapiv1alpha1.ManagedClusterAddOnHostingManifestApplied) {
 					t.Errorf("Condition Reason is not correct: %v", addOn.Status.Conditions)
 				}
+
+				manifestAppliyedCondition := meta.FindStatusCondition(addOn.Status.Conditions, addonapiv1alpha1.ManagedClusterAddOnManifestApplied)
+				if manifestAppliyedCondition == nil {
+					t.Fatal("manifestapplied condition should not be nil")
+				}
+				if manifestAppliyedCondition.Reason != addonapiv1alpha1.AddonManifestAppliedReasonManifestsApplied {
+					t.Errorf("Condition Reason is not correct: %v", manifestAppliyedCondition.Reason)
+				}
+				if manifestAppliyedCondition.Message != "no manifest need to apply" {
+					t.Errorf("Condition Message is not correct: %v", manifestAppliyedCondition.Message)
+				}
+				if manifestAppliyedCondition.Status != metav1.ConditionTrue {
+					t.Errorf("Condition Status is not correct: %v", manifestAppliyedCondition.Status)
+				}
 			},
 		},
 		{

diff --git a/test/integration/cloudevents/agent_hosting_deploy_test.go b/test/integration/cloudevents/agent_hosting_deploy_test.go
@@ -229,6 +229,20 @@ var _ = ginkgo.Describe("Agent deploy", func() {
 			if !meta.IsStatusConditionTrue(addon.Status.Conditions, addonapiv1alpha1.ManagedClusterAddOnHostingManifestApplied) {
 				return fmt.Errorf("Unexpected addon applied condition, %v", addon.Status.Conditions)
 			}
+
+			manifestAppliyedCondition := meta.FindStatusCondition(addon.Status.Conditions, addonapiv1alpha1.ManagedClusterAddOnManifestApplied)
+			if manifestAppliyedCondition == nil {
+				return fmt.Errorf("%s Condition is not found", addonapiv1alpha1.ManagedClusterAddOnManifestApplied)
+			}
+			if manifestAppliyedCondition.Reason != addonapiv1alpha1.AddonManifestAppliedReasonManifestsApplied {
+				return fmt.Errorf("Condition Reason is not correct: %v", manifestAppliyedCondition.Reason)
+			}
+			if manifestAppliyedCondition.Message != "no manifest need to apply" {
+				return fmt.Errorf("Condition Message is not correct: %v", manifestAppliyedCondition.Message)
+			}
+			if manifestAppliyedCondition.Status != metav1.ConditionTrue {
+				return fmt.Errorf("Condition Status is not correct: %v", manifestAppliyedCondition.Status)
+			}
 			return nil
 		}, eventuallyTimeout, eventuallyInterval).ShouldNot(gomega.HaveOccurred())