potiuk edited a comment on pull request #15812: URL: https://github.com/apache/airflow/pull/15812#issuecomment-840596906
I think we need a little more :) The providers are prepared and tested during CI build (We have a special job "Build and test povider packages"). And In fact the job did fail: https://github.com/apache/airflow/pull/15812/checks?check_run_id=2571762641 The job failed because our CI does not install plyvel any more since this is an optional dependency now. But for CI image we run all the tests and check if all providers are fully importable, so we should also add plyvel as depenedncy installed during CI image build. This can be done simply as yet another extra for "airflow" itself - similarly as we have virtualenv, statsd, rabbitmq etc. Simply adding it as one more extra in setup.py and adding it to the "CORE_EXTRAS_REQUIREMENTS" in setup.py should do the job. I believe this is also the reason why level_db tests failed as well and doc build failed (because neither can import plyvel). -- 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. For queries about this service, please contact Infrastructure at: [email protected]
