List the names of the failing benchmark subworkflows (#2144)

Fixes OPS-3888.

<img width="573" height="914" alt="Screenshot 2026-03-17 at 14 58 50"
src="https://github.com/user-attachments/assets/c2b1c510-7e7a-47af-ab98-7fa533e8978b"
/>

Author: cezudas

packages/react-ui/src/app/features/benchmark/components/benchmark-wizard.tsx

@@ -87,6 +87,7 @@ export const BenchmarkWizard = ({
   const {
     runPhase,
     runningProgress,
+    failedWorkflows,
     isRunPending,
     handleRunBenchmark,
     handleResetRun,
@@ -174,6 +175,7 @@ export const BenchmarkWizard = ({
                     result={benchmarkCreateResult}
                     runPhase={runPhase}
                     runningProgress={runningProgress ?? undefined}
+                    failedWorkflows={failedWorkflows}
                   />
                 )}
               </WizardStep>

packages/react-ui/src/app/features/benchmark/tests/use-benchmark-run.test.ts

@@ -136,6 +136,14 @@ describe('useBenchmarkRun', () => {
 
       expect(result.current.isRunPending).toBe(false);
     });
+
+    it('should start with empty failedWorkflows', () => {
+      const { result } = renderHook(() =>
+        useBenchmarkRun(buildBenchmarkCreationResult()),
+      );
+
+      expect(result.current.failedWorkflows).toEqual([]);
+    });
   });
 
   describe('handleRunBenchmark', () => {
@@ -255,6 +263,7 @@ describe('useBenchmarkRun', () => {
       });
 
       expect(result.current.runPhase).toBe('succeeded');
+      expect(result.current.failedWorkflows).toEqual([]);
     });
 
     it('should transition to succeeded_with_failures when some sub-workflows failed', async () => {
@@ -293,6 +302,9 @@ describe('useBenchmarkRun', () => {
       });
 
       expect(result.current.runPhase).toBe('succeeded_with_failures');
+      expect(result.current.failedWorkflows).toEqual([
+        { displayName: 'Sub', runId: 'run-2' },
+      ]);
     });
 
     it('should transition to failed when status is FAILED', async () => {

packages/react-ui/src/app/features/benchmark/use-benchmark-run.ts

@@ -15,9 +15,15 @@ import { useEffect, useMemo, useState } from 'react';
 
 import { benchmarkApi } from './benchmark-api';
 
+export type FailedWorkflow = {
+  displayName: string;
+  runId?: string;
+};
+
 export type UseBenchmarkRunResult = {
   runPhase: BenchmarkRunPhase;
   runningProgress: { completed: number; total: number } | null;
+  failedWorkflows: FailedWorkflow[];
   isRunPending: boolean;
   handleRunBenchmark: () => Promise<void>;
   handleResetRun: () => void;
@@ -43,11 +49,25 @@ function mapStatusToRunPhase(
   }
 }
 
+function getFailedSubWorkflows(
+  data: BenchmarkStatusResponse,
+): FailedWorkflow[] {
+  return data.workflows
+    .filter(
+      (w) =>
+        !w.isOrchestrator &&
+        !w.isCleanup &&
+        w.runStatus === BenchmarkStatus.FAILED,
+    )
+    .map((w) => ({ displayName: w.displayName, runId: w.runId }));
+}
+
 export const useBenchmarkRun = (
   benchmarkCreateResult: BenchmarkCreationResult | null,
 ): UseBenchmarkRunResult => {
   const [runPhase, setRunPhase] = useState<BenchmarkRunPhase>('idle');
   const [lastRunId, setLastRunId] = useState<string | undefined>();
+  const [failedWorkflows, setFailedWorkflows] = useState<FailedWorkflow[]>([]);
   const [runCount, setRunCount] = useState(0);
 
   const benchmarkId = benchmarkCreateResult?.benchmarkId ?? null;
@@ -80,6 +100,9 @@ export const useBenchmarkRun = (
       setLastRunId(statusData.lastRunId);
     }
     const newPhase = mapStatusToRunPhase(statusData);
+    if (newPhase === 'succeeded_with_failures') {
+      setFailedWorkflows(getFailedSubWorkflows(statusData));
+    }
     if (newPhase !== null) {
       setRunPhase(newPhase);
     }
@@ -122,6 +145,7 @@ export const useBenchmarkRun = (
   const handleResetRun = () => {
     setRunPhase('idle');
     setLastRunId(undefined);
+    setFailedWorkflows([]);
   };
 
   const runningProgress = useMemo(() => {
@@ -148,6 +172,7 @@ export const useBenchmarkRun = (
   return {
     runPhase,
     runningProgress,
+    failedWorkflows,
     isRunPending,
     handleRunBenchmark,
     handleResetRun,

packages/ui-components/src/components/benchmark/benchmark-analytics-phase.tsx

@@ -8,7 +8,7 @@ export const BenchmarkAnalyticsPhase = ({
   message: string;
   provider: string;
 }) => (
-  <div className="flex flex-col gap-3 py-2">
+  <div className="flex flex-col gap-1 py-2">
     <p className="text-sm dark:text-muted-foreground">{message}</p>
     <Link
       className="text-sm text-primary-200"

packages/ui-components/src/components/benchmark/benchmark-ready-step.tsx

@@ -1,24 +1,63 @@
 import { BenchmarkRunPhase } from '@/lib/types';
 import { BenchmarkCreationResult } from '@openops/shared';
 import { t } from 'i18next';
+import { Link } from 'react-router-dom';
 import { StepDescription } from '../../ui/wizard/wizard-step';
 import { BenchmarkAnalyticsPhase } from './benchmark-analytics-phase';
 import { BenchmarkFailedPhase } from './benchmark-failed-phase';
 import { BenchmarkRunningPhase } from './benchmark-running-phase';
 import { BenchmarkWorkflowList } from './benchmark-workflow-list';
 
+export type FailedWorkflow = {
+  displayName: string;
+  runId?: string;
+};
+
+const FailedWorkflowsList = ({
+  workflows,
+}: {
+  workflows: FailedWorkflow[];
+}) => (
+  <div className="flex flex-col gap-1 py-2">
+    <p className="text-sm dark:text-muted-foreground font-medium">
+      {t('The following workflows failed:')}
+    </p>
+    <ul className="list-disc pl-5 text-sm dark:text-muted-foreground">
+      {workflows.map((workflow) => (
+        <li key={workflow.displayName}>
+          {workflow.runId ? (
+            <Link
+              to={`/runs/${workflow.runId}`}
+              className="text-primary hover:underline"
+            >
+              {workflow.displayName}
+            </Link>
+          ) : (
+            workflow.displayName
+          )}
+        </li>
+      ))}
+    </ul>
+    <p className="text-sm dark:text-muted-foreground mt-1">
+      {t('Click on a workflow name to review failure details.')}
+    </p>
+  </div>
+);
+
 interface BenchmarkReadyStepProps {
   providerName: string;
   result: BenchmarkCreationResult;
   runPhase: BenchmarkRunPhase;
   runningProgress?: { completed: number; total: number };
+  failedWorkflows?: FailedWorkflow[];
 }
 
 export const BenchmarkReadyStep = ({
   providerName,
   result,
   runPhase,
   runningProgress,
+  failedWorkflows,
 }: BenchmarkReadyStepProps) => {
   const subWorkflows = result.workflows.filter(
     (w) => !w.isOrchestrator && !w.isCleanup,
@@ -52,12 +91,17 @@ export const BenchmarkReadyStep = ({
       {runPhase === 'failed' && <BenchmarkFailedPhase />}
 
       {runPhase === 'succeeded_with_failures' && (
-        <BenchmarkAnalyticsPhase
-          provider={providerName}
-          message={t(
-            "You can review your Benchmark Report here (it's not final since there are some failed workflows)",
+        <>
+          <BenchmarkAnalyticsPhase
+            provider={providerName}
+            message={t(
+              'You can access the Benchmark Report here. Note: results only cover services from workflows that ran successfully.',
+            )}
+          />
+          {failedWorkflows && failedWorkflows.length > 0 && (
+            <FailedWorkflowsList workflows={failedWorkflows} />
           )}
-        />
+        </>
       )}
 
       {runPhase === 'succeeded' && (

packages/ui-components/src/stories/benchmark/benchmark-ready-step.stories.tsx

@@ -67,6 +67,18 @@ export const RunningWithProgress: ReadyStepStory = {
 };
 export const Failed: ReadyStepStory = { args: { runPhase: 'failed' } };
 export const SucceededWithFailures: ReadyStepStory = {
-  args: { runPhase: 'succeeded_with_failures' },
+  args: {
+    runPhase: 'succeeded_with_failures',
+    failedWorkflows: [
+      {
+        displayName: 'EC2 Reserved Instances Analysis',
+        runId: 'run-001',
+      },
+      {
+        displayName: 'S3 Storage Optimization',
+        runId: 'run-002',
+      },
+    ],
+  },
 };
 export const Succeeded: ReadyStepStory = { args: { runPhase: 'succeeded' } };