The GitHub Actions job "Tests" on airflow.git has succeeded. Run started by GitHub user potiuk (triggered by potiuk).
Head commit for run: 4c4217014bcea6f7d011513986a7dc2f71168cec / Jarek Potiuk <[email protected]> Fix installing older airflow versions in Breeze with openssl fix The dnspython and Flask Application Builder have a weird transitive dependency to openssl. It does not require it - it is an optional dependnency of email validator that uses dnspython and the dnspython uses pyopenssl when installed. This is all fine. But when opendns is used with older FAB < 4.1.4) and NEWER opensl (>=23.0), just creating the FAB applicatio in flask causes an exception than there is a missing parameter in openssl. Since openssl is NOT declared as required dependency in neither FAB nor dnspython, installing airflow with constraints while having a newer version of pyopenssl does not downgrade pyopenssl it is just mentioned in a conflicting message resulting from conflicts with other packages already installed (such as cryptography). This caused a problem when installin older airflow version in breeze with --use-airflow-version switch. Also - during checking this issue, it turned out that we were not - by default - using the constraints for older version when installing it in breeze (you could do that by explicitly specifying the constraints). This was because when you used anoter way to specify versio (commit hash, tag) it is quite complex to figure out which constraints to use. However we can automatically derive constraints when you specify version, or branch, which are the most common scenarios. This PR fixes the `--use-airflow-version` case for Airflow 2.4 and below by: * automatically deriving the right constraints when version or branch are used in `--use-airflow-version` * adding pyopenssl specifically as dependency when --use-airflow-version switch is used. * also a small quality of life improvment - when airflow db migrate fails (because old versio of airflow does not support it) - we inform the user that it happened and that we are attempting to run the legacy airflow db init instead. Report URL: https://github.com/apache/airflow/actions/runs/5784592520 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
