Fix condition error of ManifestApplied.

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

Hide file tree

Showing 2 changed files with 11 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,9 @@ func TestHostingReconcile(t *testing.T) {
 				if meta.IsStatusConditionFalse(addOn.Status.Conditions, addonapiv1alpha1.ManagedClusterAddOnHostingManifestApplied) {
 					t.Errorf("Condition Reason is not correct: %v", addOn.Status.Conditions)
 				}
+				if meta.IsStatusConditionFalse(addOn.Status.Conditions, addonapiv1alpha1.ManagedClusterAddOnManifestApplied) {
+					t.Errorf("Condition Reason is not correct: %v", addOn.Status.Conditions)
+				}
 			},
 		},
 		{