Skip to content

Revert "Fetch deadline callback context via Execution API at runtime (#66608)" - #68909

Merged
kaxil merged 1 commit into
apache:mainfrom
aws-mwaa:revert-66608-workload-token-regression
Jun 23, 2026
Merged

Revert "Fetch deadline callback context via Execution API at runtime (#66608)"#68909
kaxil merged 1 commit into
apache:mainfrom
aws-mwaa:revert-66608-workload-token-regression

Conversation

@seanghaeli

@seanghaeliseanghaeli commented Jun 23, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

Reverts #66608 (commit 8095abb).

Per @ashb's reviews, broadening token:workload to read connections/variables/xcom (so deadline-callback subprocesses could fetch context) is a security concern that we need to align on first.

@boring-cyborgboring-cyborgBot added area:API Airflow's REST/HTTP API area:DAG-processing area:db-migrations PRs with DB migration area:deadline-alerts AIP-86 (former AIP-57) area:Executors-core LocalExecutor & SequentialExecutor area:Scheduler including HA (high availability) scheduler area:task-sdk labels Jun 23, 2026
@seanghaeliseanghaeli mentioned this pull request Jun 23, 2026
Closed
1 task
@kaxil
kaxil merged commit 609656f into apache:mainJun 23, 2026
122 checks passed
This was referenced Jun 23, 2026
Open
Merged
cetingokhan pushed a commit to cetingokhan/airflow that referenced this pull request Jun 24, 2026
@seanghaeliSean Ghaeli@cetingokhan
Revert "Fetch deadline callback context via Execution API at runtime (a…
9c8749b
…pache#66608)" (apache#68909)
This reverts commit 8095abb.
Co-authored-by: Sean Ghaeli <ghaeli@amazon.com>
@seanghaeliseanghaeli mentioned this pull request Jul 13, 2026
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@o-nikolaso-nikolaso-nikolas approved these changes
@ashbashbAwaiting requested review from ashbashb is a code owner
@amoghrajeshamoghrajeshAwaiting requested review from amoghrajeshamoghrajesh is a code owner
@potiukpotiukAwaiting requested review from potiukpotiuk is a code owner
@kaxilkaxilAwaiting requested review from kaxilkaxil is a code owner
@ephraimbuddyephraimbuddyAwaiting requested review from ephraimbuddyephraimbuddy is a code owner
@dstandishdstandishAwaiting requested review from dstandishdstandish is a code owner
@hussein-awalahussein-awalaAwaiting requested review from hussein-awalahussein-awala is a code owner
@bbovenzibbovenziAwaiting requested review from bbovenzibbovenzi is a code owner
@pierrejeambrunpierrejeambrunAwaiting requested review from pierrejeambrunpierrejeambrun is a code owner
@ryanahamiltonryanahamiltonAwaiting requested review from ryanahamiltonryanahamilton is a code owner
@shubhamraj-gitshubhamraj-gitAwaiting requested review from shubhamraj-gitshubhamraj-git is a code owner
@guan404mingguan404mingAwaiting requested review from guan404mingguan404ming is a code owner
@vatsrahul1001vatsrahul1001Awaiting requested review from vatsrahul1001vatsrahul1001 is a code owner
@choo121600choo121600Awaiting requested review from choo121600choo121600 is a code owner
@rawwarrawwarAwaiting requested review from rawwarrawwar is a code owner
@jason810496jason810496Awaiting requested review from jason810496jason810496 is a code owner
@bugraoz93bugraoz93Awaiting requested review from bugraoz93bugraoz93 is a code owner
@jedcunninghamjedcunninghamAwaiting requested review from jedcunninghamjedcunningham is a code owner
@bolkedebruinbolkedebruinAwaiting requested review from bolkedebruinbolkedebruin is a code owner
@XD-DENGXD-DENGAwaiting requested review from XD-DENGXD-DENG is a code owner
@dheerajturagadheerajturagaAwaiting requested review from dheerajturagadheerajturaga is a code owner
+1 more reviewer
@ramitkatariaramitkatariaramitkataria approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area:APIAirflow's REST/HTTP APIarea:DAG-processingarea:db-migrationsPRs with DB migrationarea:deadline-alertsAIP-86 (former AIP-57)area:Executors-coreLocalExecutor & SequentialExecutorarea:Schedulerincluding HA (high availability) schedulerarea:task-sdkarea:translationsarea:Triggererarea:UIRelated to UI/UX. For Frontend Developers.kind:documentationtranslation:default

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@seanghaeli@ramitkataria@o-nikolas@kaxil