Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Feb 4, 2025

When PROD image is build using sources, it should use source requireents rather than pip requirements - they might be conflicting if the main providers are different than released prividers.


^ 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 newsfragments.

When PROD image is build using sources, it should use source
requireents rather than pip requirements - they might be conflicting
if the main providers are different than released prividers.
Copy link
Contributor

@ferruzzi ferruzzi left a comment

Choose a reason for hiding this comment

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

Seems to make sense 👍

@potiuk potiuk merged commit bc27355 into apache:main Feb 4, 2025
92 checks passed
@potiuk potiuk deleted the fix-conflicting-constraints-with-prod-build branch February 4, 2025 22:10
insomnes pushed a commit to insomnes/airflow that referenced this pull request Feb 6, 2025
When PROD image is build using sources, it should use source
requireents rather than pip requirements - they might be conflicting
if the main providers are different than released prividers.
niklasr22 pushed a commit to niklasr22/airflow that referenced this pull request Feb 8, 2025
When PROD image is build using sources, it should use source
requireents rather than pip requirements - they might be conflicting
if the main providers are different than released prividers.
ambika-garg pushed a commit to ambika-garg/airflow that referenced this pull request Feb 17, 2025
When PROD image is build using sources, it should use source
requireents rather than pip requirements - they might be conflicting
if the main providers are different than released prividers.
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.

3 participants