Add SucceededWithErrors for the ApplicationTasksStatus

Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
konveyor · Jul 15, 2024 · 0f04a08 · 0f04a08
1 parent 461a4e2
commit 0f04a08
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 1 deletion.
diff --git a/client/src/app/pages/applications/applications-table/components/column-application-name.tsx b/client/src/app/pages/applications/applications-table/components/column-application-name.tsx
@@ -7,6 +7,7 @@ import {
   TimesCircleIcon,
   InProgressIcon,
   ExclamationCircleIcon,
+  ExclamationTriangleIcon,
   PendingIcon,
 } from "@patternfly/react-icons";
 
@@ -79,6 +80,15 @@ const statusMap: Record<ApplicationTasksStatus, StatusData> = {
       </Icon>
     ),
   },
+  SuccessWithErrors: {
+    popoverVariant: "warning",
+    headerText: "All tasks succeeded, but some errors occurred",
+    icon: () => (
+      <Icon status="warning">
+        <ExclamationTriangleIcon />
+      </Icon>
+    ),
+  },
 };
 
 const linkToTasks = (applicationName: string) => {

diff --git a/client/src/app/pages/applications/applications-table/useDecoratedApplications.ts b/client/src/app/pages/applications/applications-table/useDecoratedApplications.ts
@@ -19,7 +19,8 @@ export type ApplicationTasksStatus =
   | "Queued"
   | "Failed"
   | "Canceled"
-  | "Success";
+  | "Success"
+  | "SuccessWithErrors";
 
 export interface DecoratedApplication extends Application {
   /** reference to the Application being decorated */
@@ -33,6 +34,7 @@ export interface DecoratedApplication extends Application {
     latestHasQueued: boolean;
     latestHasRunning: boolean;
     latestHasSuccess: boolean;
+    latestHasSuccessWithErrors: boolean;
 
     /** The most recently created `kind === "analyzer"` task for the application */
     currentAnalyzer: Task | undefined;
@@ -83,6 +85,8 @@ const chooseApplicationTaskStatus = ({
     ? "Failed"
     : tasks.latestHasCanceled
     ? "Canceled"
+    : tasks.latestHasSuccessWithErrors
+    ? "SuccessWithErrors"
     : "Success";
 };
 
@@ -125,6 +129,9 @@ const decorateApplications = (
         latestHasSuccess: latest.some((task) =>
           TaskStates.Success.includes(task.state ?? "")
         ),
+        latestHasSuccessWithErrors: latest.some((task) =>
+          TaskStates.SuccessWithErrors.includes(task.state ?? "")
+        ),
 
         currentAnalyzer: tasksByKind["analyzer"]?.[0],
       },

diff --git a/client/src/app/queries/tasks.ts b/client/src/app/queries/tasks.ts
@@ -26,6 +26,7 @@ export const TaskStates = {
   Queued: ["Ready", "Postponed", "Pending", "Running"], // "Created", "QuotaBlocked" ??
   Running: ["Running"],
   Success: ["Succeeded", "SucceededWithErrors"],
+  SuccessWithErrors: ["SucceededWithErrors"],
 };
 
 export const TasksQueryKey = "tasks";