anishgirianish commented on PR #66141:
URL: https://github.com/apache/airflow/pull/66141#issuecomment-4349903600
Hey! Confirmed the regression is real and on me, generate_token flipped to
workload scope at the base class but supervise_callback never got the paired
swap. Thanks for catching it.
Quick thought before you go deeper on the PR: this is now the third place
we'd be hand-rolling the workload→execution swap (tasks /run, your callback
/run, and #62343's connection-test PATCH). Wonder if it's worth lifting it into
a generic POST /auth/exchange + a transparent swap in the SDK client, so
resource endpoints stay execution-only and future workload types inherit the
fix automatically. Happy to chat / draft if it sounds reasonable.
Would love to hear @ashb @amoghrajesh @kaxil thoughts on this as well
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
