potiuk commented on PR #36799: URL: https://github.com/apache/airflow/pull/36799#issuecomment-1894562418
And I even know why it did not fail in PR but started to fail only in SOME PRs and main, "canary" build. The problem is that in PR you do not run "full checks" - only a subset of the tests. And we only build and install modified providers to save time. The problem in this case is that when we do not build all providers - only few - this check only INSTALLS providers but does not VERIFY them - because verification might fail if we only install few providers, because of some dynamic cross-importing between providers. That's why this check only fails in those PRs that - for whatever reason - build and install and VERIFY all providers. However, I think this speed-up is not needed any more. It gave a lot savings when building providers took way more time - and I optimized it a lot since (like 10x) and we can easily build all providers now for all PRs as it takes less than 1 minute now (it used to takes 12 minutes or so). -- 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]
