Avoid displaying alert messages for successfully completed plans

Reference: https://issues.redhat.com/browse/MTV-1730 If plan was completed succesfully, it's irrelevant to display critical errors or warnings alerts in page header. Signed-off-by: Sharon Gratch <[email protected]>
kubev2v · Dec 3, 2024 · ab7a45a · ab7a45a
1 parent c3eeb85
commit ab7a45a
Showing 1 changed file with 31 additions and 21 deletions.
diff --git a/...s/forklift-console-plugin/src/modules/Plans/views/details/components/PlanPageHeadings.tsx b/...s/forklift-console-plugin/src/modules/Plans/views/details/components/PlanPageHeadings.tsx
@@ -79,27 +79,35 @@ export const PlanPageHeadings: React.FC<{ name: string; namespace: string }> = (
     return isPreserveStaticIPs && isMapToPod;
   };
 
-  if (criticalCondition) {
-    alerts.push(
-      <PlanCriticalCondition
-        type={criticalCondition?.type}
-        message={criticalCondition?.message}
-        key={'providerCriticalCondition'}
-      />,
-    );
-  } else if (preserveIpsWithPodMapCondition()) {
-    alerts.push(
-      <PlanWarningCondition
-        type={t('Preserving static IPs of VMs might fail')}
-        message={t(
-          'The plan is set to preserve the static IPs of VMs mapped to a Pod network type. This is not supported and therefore VM IPs can be changed during the migration process.',
-        )}
-        suggestion={t(
-          "For fixing, update the destination network mappings to avoid using the 'Pod Networking' type.",
-        )}
-      />,
-    );
-  }
+  const handleAlerts = () => {
+    // alerts are not relevant to display if plan was completed successfully
+    if (planStatus === 'Succeeded') return;
+
+    if (criticalCondition) {
+      alerts.push(
+        <PlanCriticalCondition
+          type={criticalCondition?.type}
+          message={criticalCondition?.message}
+          key={'providerCriticalCondition'}
+        />,
+      );
+      return;
+    }
+
+    if (preserveIpsWithPodMapCondition()) {
+      alerts.push(
+        <PlanWarningCondition
+          type={t('Preserving static IPs of VMs might fail')}
+          message={t(
+            'The plan is set to preserve the static IPs of VMs mapped to a Pod network type. This is not supported and therefore VM IPs can be changed during the migration process.',
+          )}
+          suggestion={t(
+            "For fixing, update the destination network mappings to avoid using the 'Pod Networking' type.",
+          )}
+        />,
+      );
+    }
+  };
 
   const onClick = () => {
     showModal(
@@ -123,6 +131,8 @@ export const PlanPageHeadings: React.FC<{ name: string; namespace: string }> = (
     </Level>
   );
 
+  handleAlerts();
+
   return (
     <>
       <PageHeadings