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: 40eef0f5f901e2d98d559e405887651688d31634 / pierrejeambrun <[email protected]> Address inline review: mirror task-instance bulk delete patterns Backend: - Inline ``_resolve_dag_id``, ``_result_key`` and ``_fetch_dag_runs`` — each had exactly one caller. Memory rule added so future PRs avoid premature helpers. - Drop the deletable-state restriction. Bulk task-instance delete has no such restriction; bulk Dag-run delete shouldn't either. - Emit one 404 error per missing entity when ``action_on_non_existence`` is ``fail`` instead of collating them into a single error, and stop early-returning so matched runs still get deleted. The invariant ``len(success) + len(errors) == len(requested entities)`` now holds. - Distinguish the two ways a wildcard can leak into ``dag_id`` (path vs body) in the 400 message, mirroring ``BulkTaskInstanceService._categorize_entities``. UI: - Mirror ``useBulkTaskInstances`` exactly: bring back ``useState`` for the error, the shared ``handleActionResult`` helper, single-error surfacing via ``ErrorAlert``, and the ``bulkAction(requestBody)`` shape so the consumer constructs the full ``BulkBody``. Brent's prior review of the closed twin PR pushed past this pattern, but until TI is updated we want both hooks symmetrical — a follow-up can improve both at once. - Inline the affected-runs column array into ``BulkDeleteDagRunsButton`` and delete the standalone ``bulkDagRunsColumns.tsx`` file (single caller). Report URL: https://github.com/apache/airflow/actions/runs/26153955612 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
