Execution API: Resolve retry callback team via TI history#67998
Execution API: Resolve retry callback team via TI history#67998GayathriSrividya wants to merge 2 commits into
Conversation
|
@GayathriSrividya — I've removed the Automated triage note drafted by an AI-assisted tool — may get things wrong; a real Apache Airflow maintainer takes the next look once it's green. (why automated) Drafted-by: Claude Code (Opus 4.8); reviewed by @potiuk before posting |
4255c7d to
eca0773
Compare
I’ve rebased this onto the latest main commit to branch. It should be back in sync now. |
9a24f09 to
450cb99
Compare
closes #67923
Summary
This PR fixes retry callback team-name resolution in the execution API.
Root cause
Team lookup could miss in retry/history-backed paths when only active task-instance context was considered, causing downstream failures (500 / SQLAlchemy argument errors).
Changes
task_instancetask_instance_historyTests
Added coverage in
airflow-core/tests/unit/api_fastapi/execution_api/test_security.pyfor:UNION ALL)Local validation
uv run --project airflow-core pytest airflow-core/tests/unit/api_fastapi/execution_api/test_security.py -xvspassed.Scope
This change is intentionally scoped to retry callback team-name resolution only.