[BACKPORT] make Breeze installation and reinstallation support both uv and pipx … #43626
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
potiuk
requested review from
amoghrajesh,
eladkal,
ephraimbuddy,
hussein-awala and
utkarsharma2
hussein-awala
approved these changes
Nov 4, 2024
potiuk
force-pushed
the
breeze-reinstallation-with-uv
branch
from
9640174 to
5d58c49
Compare
Member
Author
|
I've added a few more things in CI/selectuve to sync with latest version in main. I could have cherry-picked individual PRs but at this stage it's easier to just "make it works" in v2-10-test as there are significant non-cherry-pickable changes (new UI, new providers support etc. - all of which are not usable /harmful in v2-10-test. |
potiuk
changed the title
Member
Author
|
Backport of #43607 |
potiuk
force-pushed
the
breeze-reinstallation-with-uv
branch
from
5d58c49 to
437bda8
Compare
Member
Author
|
ok. looks like we have now full support for |
potiuk
force-pushed
the
breeze-reinstallation-with-uv
branch
from
437bda8 to
6628fa2
Compare
…pache#43607) So far `breeze` fully supported only `pipx` installation. For example it would reinstall itself automatically with pipx if you attempted to run it from another workspace/checked out repository of Airflow, and it only provided instructions for pipx. With this PR: * the `uv tool` is preferred way to install breeze * the `pipx` is PSF-governance managed alternative * breeze will reinstall itself using uv if it has been installed with uv before when it is run from a different workspace or different airflow repo checked out in another folder Also documentation is updated to make `uv` the recommended tool and describing how to install it - with `pipx` provided as an alternative. Warning is printed in case pre-commit-uv is not installed with the pre-commit (pre-commit-uv significantly speeds up installation of the venvs by pre-commit). This warning also provides instructions how to install it. (cherry picked from commit ddc5670)
potiuk
force-pushed
the
breeze-reinstallation-with-uv
branch
from
6628fa2 to
9b7412f
Compare
utkarsharma2
added
the
changelog:skip
utkarsharma2
pushed a commit
that referenced
this pull request
Dec 4, 2024
…43607) (#43626) So far `breeze` fully supported only `pipx` installation. For example it would reinstall itself automatically with pipx if you attempted to run it from another workspace/checked out repository of Airflow, and it only provided instructions for pipx. With this PR: * the `uv tool` is preferred way to install breeze * the `pipx` is PSF-governance managed alternative * breeze will reinstall itself using uv if it has been installed with uv before when it is run from a different workspace or different airflow repo checked out in another folder Also documentation is updated to make `uv` the recommended tool and describing how to install it - with `pipx` provided as an alternative. Warning is printed in case pre-commit-uv is not installed with the pre-commit (pre-commit-uv significantly speeds up installation of the venvs by pre-commit). This warning also provides instructions how to install it. (cherry picked from commit ddc5670)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
…(#43607)
So far
breezefully supported onlypipxinstallation. For example it would reinstall itself automatically with pipx if you attempted to run it from another workspace/checked out repository of Airflow, and it only provided instructions for pipx.With this PR:
uv toolis preferred way to install breezepipxis PSF-governance managed alternativeAlso documentation is updated to make
uvthe recommended tool and describing how to install it - withpipxprovided as an alternative.Warning is printed in case pre-commit-uv is not installed with the pre-commit (pre-commit-uv significantly speeds up installation of the venvs by pre-commit). This warning also provides instructions how to install it.
(cherry picked from commit ddc5670)
^ 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.rstor{issue_number}.significant.rst, in newsfragments.