The GitHub Actions job "Tests (AMD)" on airflow.git/feature/52439-bulk-dag-runs-actions has failed. Run started by GitHub user pierrejeambrun (triggered by pierrejeambrun).
Head commit for run: 7934dd38e31582fb675c79967cf6c753b629e42c / pierrejeambrun <[email protected]> Apply Brent's review feedback from the closed twin PR Pulls in the substantive UI feedback @bbovenzi left on #66554 so this PR lands without re-litigating the same comments. The closed PR was broader (clear / mark / delete) but the structural feedback applies verbatim to delete-only: - Move all DagRuns-related files into ``pages/DagRuns/``, mirroring ``pages/TaskInstances/``. Adds a small ``index.ts`` re-export so ``router.tsx`` keeps the same import path. - Rename ``useBulkDagRuns`` to ``useBulkDeleteDagRuns`` so the hook name matches the single button it serves (one-hook-one-button symmetry — when we add bulk update/clear we'll add sibling hooks). - Stop hand-rolling pending/error state with ``useState<unknown>``; read ``error`` straight off ``useMutation``'s return. - Surface ALL per-entity errors from ``BulkResponse.delete.errors`` instead of just the first; render each as its own ``Alert`` row. - Only invalidate the dag-runs / task-instances queries when at least one entry actually succeeded — a 200 with all-errors should not churn the table. - Keep the dialog open when per-entity errors come back so the user can read what failed; ``reset()`` clears them on close. Report URL: https://github.com/apache/airflow/actions/runs/26104108931 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
