Skip to content

Bring back providers compatibility checks #52398

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 Jun 28, 2025

The compatibility checks were removed in #52072 accidentally. This one brings them back:

  • Python 3.10
  • do not add cloudant (it was not working for Python 3.9)

^ 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 backport-to-v3-0-test Mark PR with this label to backport to v3-0-test branch labels Jun 28, 2025
@potiuk potiuk requested a review from eladkal June 28, 2025 08:48
@potiuk potiuk force-pushed the bring-back-providers-compatibility-checks branch 2 times, most recently from 445f766 to 44e6bf9 Compare June 28, 2025 10:17
@potiuk potiuk force-pushed the bring-back-providers-compatibility-checks branch from 44e6bf9 to b6a0ed5 Compare June 28, 2025 10:47
@eladkal
Copy link
Contributor

eladkal commented Jun 28, 2025

I don't think we need to backport to 3.0?
the Python 3.10 PR was not backport too

@potiuk potiuk removed the backport-to-v3-0-test Mark PR with this label to backport to v3-0-test branch label Jun 28, 2025
@potiuk
Copy link
Member Author

potiuk commented Jun 28, 2025

Yeah. No need. Removed it ... I also have hard time figuring out how the compatibility of the links /test should be fixed. Anyone can help with it and provide fixup :) ? Especially if you know more about the context/ti changes between Airflow 2 and 3?

@gopidesupavan
Copy link
Member

Yeah. No need. Removed it ... I also have hard time figuring out how the compatibility of the links /test should be fixed. Anyone can help with it and provide fixup :) ? Especially if you know more about the context/ti changes between Airflow 2 and 3?

Yeah looking it now.

@potiuk potiuk force-pushed the bring-back-providers-compatibility-checks branch from b6a0ed5 to 3cc82b4 Compare June 28, 2025 16:18
@potiuk
Copy link
Member Author

potiuk commented Jun 28, 2025

Yeah. No need. Removed it ... I also have hard time figuring out how the compatibility of the links /test should be fixed. Anyone can help with it and provide fixup :) ? Especially if you know more about the context/ti changes between Airflow 2 and 3?

Yeah looking it now.

OK. I have a fix.

The compatibility checks were removed in apache#52072 accidentally. This
one brings them back:

* Python 3.10
* do not add cloudant (it was not working for Python 3.9)
@potiuk potiuk force-pushed the bring-back-providers-compatibility-checks branch from 3cc82b4 to bb4e943 Compare June 28, 2025 17:21
@potiuk potiuk merged commit 165d5d5 into apache:main Jun 28, 2025
99 checks passed
@potiuk potiuk deleted the bring-back-providers-compatibility-checks branch June 28, 2025 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants