-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
GH-38984: [Python][Packaging] Verification of wheels on AlmaLinux 8 are failing due to missing pip #38985
Conversation
@github-actions crossbow submit wheel-manylinux-2-28-cp38-amd64 |
Revision: 0e6b6a8 Submitted crossbow builds: ursacomputing/crossbow @ actions-81d4bb8bcf
|
…ux 8 are failing due to missing pip
…ith almalinux 8.9
@github-actions crossbow submit wheel-manylinux-2-28-cp38-amd64 |
Revision: 98084a5 Submitted crossbow builds: ursacomputing/crossbow @ actions-7ae64bcfcb
|
@github-actions crossbow submit wheel-manylinux--cp38- |
Revision: 98084a5 Submitted crossbow builds: ursacomputing/crossbow @ actions-f7e804a44d
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
It seems that we can remove python38-*
:
arrow/dev/release/setup-rhel-rebuilds.sh
Lines 47 to 48 in d66780d
python38-devel \ | |
python38-pip \ |
Because it seems that recent AlmaLinux 8 installs python3.11-pip-wheel
by dnf -y groupinstall "Development Tools"
:
https://github.com/ursacomputing/crossbow/actions/runs/7016111614/job/19086656171#step:10:677
Can we try removing python38-*
in this PR? Or should we do it in a follow-up PR?
@github-actions crossbow submit wheel-manylinux--cp38- |
Revision: e059d2c Submitted crossbow builds: ursacomputing/crossbow @ actions-c9dbc2c813
|
I've removed them on this PR and have re-run the wheels. Let me know if you are ok with that. Thanks @kou ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Thanks! I'll merge this.
After merging your PR, Conbench analyzed the 6 benchmarking runs that have been run so far on merge-commit 2760faf. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. It also includes information about 4 possible false positives for unstable benchmarks that are known to sometimes produce them. |
…re failing due to missing pip (#38985) ### Rationale for this change Almalinux 8 has been updated from 8.8 to 8.9. When using 8.9 python3 seems to be shipped without pip as the command `python3 -m pip install -U pip` fails to find pip. ### What changes are included in this PR? Use the [ensurepip package](https://docs.python.org/3/library/ensurepip.html) which provides support for bootstrapping the pip installer into an existing Python installation. ### Are these changes tested? Yes via archery. ### Are there any user-facing changes? No * Closes: #38984 Authored-by: Raúl Cumplido <[email protected]> Signed-off-by: Sutou Kouhei <[email protected]>
…ux 8 are failing due to missing pip (apache#38985) ### Rationale for this change Almalinux 8 has been updated from 8.8 to 8.9. When using 8.9 python3 seems to be shipped without pip as the command `python3 -m pip install -U pip` fails to find pip. ### What changes are included in this PR? Use the [ensurepip package](https://docs.python.org/3/library/ensurepip.html) which provides support for bootstrapping the pip installer into an existing Python installation. ### Are these changes tested? Yes via archery. ### Are there any user-facing changes? No * Closes: apache#38984 Authored-by: Raúl Cumplido <[email protected]> Signed-off-by: Sutou Kouhei <[email protected]>
Rationale for this change
Almalinux 8 has been updated from 8.8 to 8.9. When using 8.9 python3 seems to be shipped without pip as the command
python3 -m pip install -U pip
fails to find pip.What changes are included in this PR?
Use the ensurepip package which provides support for bootstrapping the pip installer into an existing Python installation.
Are these changes tested?
Yes via archery.
Are there any user-facing changes?
No