Display pipeline run volume name correctly (opendatahub-io#3027)

red-hat-data-services · Jul 23, 2024 · c6336ff · c6336ff
1 parent d1324f6
commit c6336ff
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/frontend/src/concepts/pipelines/kfTypes.ts b/frontend/src/concepts/pipelines/kfTypes.ts
@@ -293,6 +293,7 @@ export type PipelineExecutorKF = {
   };
   pvcMount?: {
     mountPath: string;
+    constant?: string;
     taskOutputParameter?: {
       outputParameterKey: string;
       producerTask: string;

diff --git a/frontend/src/concepts/pipelines/topology/__tests__/parseUtils.spec.ts b/frontend/src/concepts/pipelines/topology/__tests__/parseUtils.spec.ts
@@ -598,6 +598,10 @@ describe('pipeline topology parseUtils', () => {
                         producerTask: 'test-task-1',
                       },
                     },
+                    {
+                      mountPath: 'path-2',
+                      constant: 'test-constant-value',
+                    },
                   ],
                 },
               },
@@ -606,7 +610,10 @@ describe('pipeline topology parseUtils', () => {
         },
       };
       const result = parseVolumeMounts(testPlatformSpec, testExecutorLabel);
-      expect(result).toEqual([{ mountPath: 'path-1', name: 'test-task-1' }]);
+      expect(result).toEqual([
+        { mountPath: 'path-1', name: 'test-task-1' },
+        { mountPath: 'path-2', name: 'test-constant-value' },
+      ]);
     });
   });
 });

diff --git a/frontend/src/concepts/pipelines/topology/parseUtils.ts b/frontend/src/concepts/pipelines/topology/parseUtils.ts
@@ -340,7 +340,7 @@ export const parseVolumeMounts = (
   }
   return executor.pvcMount.map((pvc) => ({
     mountPath: pvc.mountPath,
-    name: pvc.taskOutputParameter?.producerTask ?? '',
+    name: pvc.taskOutputParameter?.producerTask ?? pvc.constant ?? '',
   }));
 };