Dashboard shows cached pipeline steps in gray (no info about what it …

…means)
opendatahub-io · Jun 12, 2024 · 52934cc · 52934cc
1 parent 74513a0
commit 52934cc
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 2 deletions.
diff --git a/frontend/src/concepts/pipelines/topology/__tests__/parseUtils.spec.ts b/frontend/src/concepts/pipelines/topology/__tests__/parseUtils.spec.ts
@@ -283,7 +283,7 @@ describe('pipeline topology parseUtils', () => {
       { state: RuntimeStateKF.SKIPPED, status: RunStatus.Skipped },
       { state: RuntimeStateKF.SUCCEEDED, status: RunStatus.Succeeded },
       { state: ExecutionStateKF.CANCELED, status: RunStatus.Cancelled },
-      { state: ExecutionStateKF.CACHED, status: RunStatus.Skipped },
+      { state: ExecutionStateKF.CACHED, status: RunStatus.Succeeded },
       { state: ExecutionStateKF.COMPLETE, status: RunStatus.Succeeded },
       { state: ExecutionStateKF.FAILED, status: RunStatus.Failed },
       { state: ExecutionStateKF.RUNNING, status: RunStatus.Running },

diff --git a/frontend/src/concepts/pipelines/topology/parseUtils.ts b/frontend/src/concepts/pipelines/topology/parseUtils.ts
@@ -315,8 +315,8 @@ export const translateStatusForNode = (
       return RunStatus.InProgress;
     case ExecutionStateKF.COMPLETE:
     case RuntimeStateKF.SUCCEEDED:
-      return RunStatus.Succeeded;
     case ExecutionStateKF.CACHED:
+      return RunStatus.Succeeded;
     case RuntimeStateKF.SKIPPED:
       return RunStatus.Skipped;
     case RuntimeStateKF.RUNTIME_STATE_UNSPECIFIED:

diff --git a/frontend/src/concepts/topology/customNodes/StandardTaskNode.tsx b/frontend/src/concepts/topology/customNodes/StandardTaskNode.tsx
@@ -12,7 +12,9 @@ import {
   WithContextMenuProps,
   WithSelectionProps,
 } from '@patternfly/react-topology';
+import { AngleDoubleRightIcon } from '@patternfly/react-icons';
 import { PipelineNodeModelExpanded } from '~/concepts/topology/types';
+import { ExecutionStateKF } from '~/concepts/pipelines/kfTypes';
 
 type StandardTaskNodeProps = {
   element: GraphElement<PipelineNodeModelExpanded>;
@@ -45,6 +47,11 @@ const StandardTaskNode: React.FunctionComponent<StandardTaskNodeProps> = ({
         selected={selected}
         scaleNode={hover && detailsLevel !== ScaleDetailsLevel.high}
         status={data?.runStatus}
+        customStatusIcon={
+          data?.pipelineTask.status?.state === ExecutionStateKF.CACHED ? (
+            <AngleDoubleRightIcon />
+          ) : undefined
+        }
         hideDetailsAtMedium
         hiddenDetailsShownStatuses={[
           RunStatus.Succeeded,