shahar1 commented on PR #61738: URL: https://github.com/apache/airflow/pull/61738#issuecomment-3879810253
> Thanks for the detailed feedback. > > I agree the current approach is blunt, and that an expiry-aware or exec-specific solution would be better long-term. Given the active breakage with exec-based auth, I prioritized correctness first to unblock users. > > I’ll fix the failing static checks shortly. > > Once CI is green, I’m happy to either: > > * add a TODO here and follow up with a refined PR, or > * update this PR to scope caching only to non-exec auth. > > Please let me know which direction you’d prefer. If you could scope caching only to non-exec auth I think it would be the optimal compromise for now. Later you could implement a long-term solution. Please be aware that I'm starting to cut the release in few minutes for all providers altogether, so I'm not sure that it will make it to the upcoming release - but if the other maintainers find this PR critical to merge during the voting period, we could postpone k8s release, and do it ad-hoc (on demand) afertwards. Feel free to comment in the dev list thread, and in the issue when the voting starts. -- 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]
