Skip to content

[v2-10-test] Workaround Python 3.8 alibabacloud issue #49163

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Apr 12, 2025

Alibabacloud 1.0 has a confusing setup.py where it changes dependencies dynamically based on sys.version.minor and it confuses installers - uv installs appsheduler 3.11 for Python 3.8 where pip expects < 3.11

We upgrade both pip and uv to latest versions and add manual protection to avoid this from happening.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@boring-cyborg boring-cyborg bot added area:dev-tools area:production-image Production image improvements and fixes labels Apr 12, 2025
@potiuk potiuk force-pushed the fix-python-3-8-build-failing-airflow-2-10 branch 5 times, most recently from 4c97415 to 3eeec4a Compare April 12, 2025 20:42
Alibabacloud 1.0 has a confusing setup.py where it changes dependencies
dynamically based on sys.version.minor and it confuses installers -
uv installs appsheduler 3.11 for Python 3.8 where pip expects < 3.11

We upgrade both `pip` and `uv` to latest versions and add manual
protection to avoid this from happening.

Also celery integration tests are not valid with the last celery
version. The test was supposed to be run with latest provider
and it should not be part of core integration tests.
@potiuk potiuk force-pushed the fix-python-3-8-build-failing-airflow-2-10 branch from 3eeec4a to 91a2b72 Compare April 12, 2025 20:52
Copy link
Member

@jason810496 jason810496 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! A small question about deleting test_celery_integration.

@potiuk potiuk merged commit 4e1b7e2 into apache:v2-10-test Apr 13, 2025
110 checks passed
@potiuk potiuk deleted the fix-python-3-8-build-failing-airflow-2-10 branch April 13, 2025 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:dev-tools area:production-image Production image improvements and fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants