potiuk commented on PR #32694: URL: https://github.com/apache/airflow/pull/32694#issuecomment-1641967986
Another check separated out from the #32604 set of changes. This is also related to our recent disucussion on long running CLI commands @o-nikolas and @uranusjr. While implementing separating the providers configuraiton I've found out that it is all-too-easy to accidentally trigger ProvidersManager initialization when you just parse CLI commands (by importing something that runs the initialization at top level). Usually it is as easy as making imports local (see the celery_app imports moved in this PR) - but you have to know you should do that. This change adds a command in providers that will detect such a situation and pytest test using it to fail in case we do such accidental mistake. -- 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]
