potiuk commented on PR #41067: URL: https://github.com/apache/airflow/pull/41067#issuecomment-2254221364
This PR will have quite a number of failing tests still - some of them because they should just be disabled - on a module level in most cases (with adding `pytest.mark.skuip-if-database-isolation` to `pytest.mark.db-test`. Some of them because there are still some missing pieces to fix in the internal API. The "special" `DatabaseIsolation` test that replaces "InProgressDisabled" runs the special "Database Isolation" mode that allows all our test fixtures and generally "test code" to access the database, while any code that is being tested from "airflow" should get special TracingSession assigned that will fail if the "airflow" code will attempt to use the database directly rather than via internal API. I will make an extra pass today - once it completes - and disable all the "big" modules that should be disabled from testing, and maybe fix some obvious cases. But anyone (especially @jscheffl @vincbeck and @dstandish ) are most welcome to sign-up to fix some of those in parallel (please comment here when you pick something - I will be merging my fixes individually as separate PRs and will keep on rebasing that PR until we will get it hopefully green before 2.10rc1. -- 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]
