Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Feb 13, 2024

When building reproducible packages with Python 3.8 they are ... not reproducible. The tarfile produces slightly different output and packages are not binary identical.

This change forces anyone preparing reproducible package to have breeze installed using 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 newsfragments.

@potiuk potiuk added this to the Airflow Helm Chart 1.12.1 milestone Feb 13, 2024
@potiuk potiuk force-pushed the fix-teething-problems-with-reproducible-packages branch 3 times, most recently from dda9e2b to 464b2e3 Compare February 13, 2024 21:51
@potiuk potiuk added the non committer build When this label is set, the build is treated as non-committer build label Feb 13, 2024
@potiuk potiuk force-pushed the fix-teething-problems-with-reproducible-packages branch from 464b2e3 to 5b4023f Compare February 13, 2024 22:47
@potiuk potiuk modified the milestones: Airflow Helm Chart 1.12.1, Airflow 2.8.2 Feb 13, 2024
@potiuk potiuk force-pushed the fix-teething-problems-with-reproducible-packages branch from aa91302 to 1c575ed Compare February 13, 2024 23:50
@potiuk
Copy link
Member Author

potiuk commented Feb 13, 2024

Turned out pipx reinstall is the right way to reinstall, and you cannot combine --python with --force :)

When building reproducible packages with Python 3.8 they are ...
not reproducible. The tarfile produces slightly different output
and packages are not binary identical.

This change forces anyone preparing reproducible package to have
breeze installed using Python 3.9+
@potiuk potiuk force-pushed the fix-teething-problems-with-reproducible-packages branch from 1c575ed to 8cf5216 Compare February 13, 2024 23:52
@potiuk
Copy link
Member Author

potiuk commented Feb 13, 2024

And yes, also removed the print

@potiuk potiuk merged commit 4ae096e into apache:main Feb 14, 2024
@potiuk potiuk deleted the fix-teething-problems-with-reproducible-packages branch February 14, 2024 01:18
@potiuk potiuk added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Feb 14, 2024
potiuk added a commit that referenced this pull request Feb 14, 2024
When building reproducible packages with Python 3.8 they are ...
not reproducible. The tarfile produces slightly different output
and packages are not binary identical.

This change forces anyone preparing reproducible package to have
breeze installed using Python 3.9+

(cherry picked from commit 4ae096e)
potiuk added a commit that referenced this pull request Feb 14, 2024
The #37401 forced Python 3.9 for all release commands that build
reproducible packages. Thie PR sets Python 3.9 in the PROD image
builds that run in release branches so that they also work.
potiuk added a commit that referenced this pull request Feb 14, 2024
The #37401 forced Python 3.9 for all release commands that build
reproducible packages. Thie PR sets Python 3.9 in the PROD image
builds that run in release branches so that they also work.
potiuk added a commit that referenced this pull request Feb 14, 2024
The #37401 forced Python 3.9 for all release commands that build
reproducible packages. Thie PR sets Python 3.9 in the PROD image
builds that run in release branches so that they also work.
ephraimbuddy pushed a commit that referenced this pull request Feb 22, 2024
When building reproducible packages with Python 3.8 they are ...
not reproducible. The tarfile produces slightly different output
and packages are not binary identical.

This change forces anyone preparing reproducible package to have
breeze installed using Python 3.9+

(cherry picked from commit 4ae096e)
ephraimbuddy pushed a commit that referenced this pull request Feb 22, 2024
The #37401 forced Python 3.9 for all release commands that build
reproducible packages. Thie PR sets Python 3.9 in the PROD image
builds that run in release branches so that they also work.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) non committer build When this label is set, the build is treated as non-committer build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants