potiuk commented on PR #37320: URL: https://github.com/apache/airflow/pull/37320#issuecomment-1939347144
All right - I finally get that one green and does exactly what I wanted it to do. With this one: 1) Pydantic is now completely optional dependency for Airlfow. It was a bit of a mistake to put it there as "hard" dependency, since we are not really using it (yet) and even if we will it will be a new way of running Airflow. 2) Airflow works and all the tests are pasising in CI in three modes: * With Pydantic 2 installed * With Pydantic 1 installed * Without Pydantic at all We have new CI Jobs that test just that - they will replace default (Pydantic2) installation with Pydantic 1 or remove it (and all the provider packages that depend on it). I managed to run it. in a separate `complete set of tests` run - I've managed to fix all the side-effects from various test cases that caused some other test failures, so that should be good to go (and future proof as well). The set of changes finally is not huge and they are rather localized - so I really look forward to be able to merge it and cherry-pick for 2.8.2, because that would solve a number of pain of our users who still have some old dependencies that use Pydantic 1 and oculd not use Airflow 2.8 because of that. -- 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]
