kimyoungi99 commented on PR #62214:
URL: https://github.com/apache/airflow/pull/62214#issuecomment-3936374238

   Thanks for approving the CI! The failing checks (openlineage compat, PROD 
image tests) look unrelated to this PR. Regarding provider tests — our approach 
only adds singleton caching to `create_auth_manager()` without changing the 
initialization flow, so existing fixtures using `purge_cached_app()` should 
work as-is. FAB provider tests pass locally as well.
   
   Also addressed the review feedback — moved `get_auth_manager_cls()` inside 
the lock so the fast path is just a `None` check. Added `purge_cached_app()` in 
`test_upgradedb` to ensure clean state between parametrized cases.


-- 
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]

Reply via email to