frontend: Job: Add sort function for job durations

This change adds a sort function for job duration times. Previously, sorting by duration would show the list of jobs out of order, but this should no longer be the case. Fixes: #2564 Signed-off-by: Evangelos Skopelitis <[email protected]>
headlamp-k8s · Nov 13, 2024 · 7da4469 · 7da4469
1 parent d24fc9f
commit 7da4469
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/frontend/src/components/job/List.tsx b/frontend/src/components/job/List.tsx
@@ -111,14 +111,13 @@ export function JobsListRenderer(props: JobsListRendererProps) {
           id: 'duration',
           label: t('translation|Duration'),
           getValue: job => {
-            const startTime = job.status?.startTime;
-            const completionTime = job.status?.completionTime;
-            if (!!startTime && !!completionTime) {
-              const duration = new Date(completionTime).getTime() - new Date(startTime).getTime();
+            const duration = job.getDuration();
+            if (duration > 0) {
               return formatDuration(duration, { format: 'mini' });
             }
             return '-';
           },
+          sort: (job1, job2) => job1.getDuration() - job2.getDuration(),
           gridTemplate: 0.6,
         },
         {