-
Notifications
You must be signed in to change notification settings - Fork 4k
GH-49057: [Python][Packaging] Remove test base docker image and use single windows base image for both build and test containers #48996
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
base: main
Are you sure you want to change the base?
Conversation
|
@github-actions crossbow submit wheel-windows-cp314-cp314-amd64 |
|
Revision: cb51469 Submitted crossbow builds: ursacomputing/crossbow @ actions-ff4f1aab24
|
|
@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64 |
|
Revision: cb51469 Submitted crossbow builds: ursacomputing/crossbow @ actions-7fa4698ac5
|
cb51469 to
ac76f96
Compare
|
@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64 |
|
Revision: 054b8dc Submitted crossbow builds: ursacomputing/crossbow @ actions-9f6150165d
|
|
@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64 |
|
Revision: a07afbf Submitted crossbow builds: ursacomputing/crossbow @ actions-ef0cfd85af
|
|
@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64 |
|
Revision: e1ab7da Submitted crossbow builds: ursacomputing/crossbow @ actions-2eb2d5cfd2
|
|
@github-actions crossbow submit wheel-windows-* |
|
Revision: 6665098 Submitted crossbow builds: ursacomputing/crossbow @ actions-d8042cc7a1 |
|
@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64 |
|
Revision: 2aa3a1f Submitted crossbow builds: ursacomputing/crossbow @ actions-22c758b35f
|
|
@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64 |
|
Revision: 1ca2938 Submitted crossbow builds: ursacomputing/crossbow @ actions-a84a844d69
|
|
@github-actions crossbow submit wheel-windows-* |
|
Revision: 1ca2938 Submitted crossbow builds: ursacomputing/crossbow @ actions-bb12f9cd6a |
|
|
|
This PR is only to merge installing build tools, pymanager and other dependencies in a single base to be used by build and test images so we don't have to manage installing pymanager or python via choco on a bunch of different places. |
|
|
||
| # Install the GCS testbench using a well-known Python version. | ||
| # NOTE: cannot use pipx's `--fetch-missing-python` because of | ||
| # https://github.com/pypa/pipx/issues/1521, therefore download Python ourselves. |
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.
It seems the pipx issue was solved, should we reconsider this?
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.
Or we can use pymanager instead of choco, I don't know. (but this might be fine anyway)
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.
We still would require to install python manually (either via choco or pymanager) to pip install pipx, right? As the base image won't have any python by default. I don't understand what the --fetch-missing-python problem was trying to solve on that specific case.
I am fine trying to install Python here with pymanager instead of choco.
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.
Ah, I think the image had some Python installed before. If choco works fine then we can probably keep it for now...
|
@github-actions crossbow submit wheel-windows-cp313-cp313-amd64 |
|
Revision: 73ca678 Submitted crossbow builds: ursacomputing/crossbow @ actions-17f326e4c3
|
|
@github-actions crossbow submit wheel-windows-cp313-cp313-amd64 |
|
Revision: 8bbf7d5 Submitted crossbow builds: ursacomputing/crossbow @ actions-437184cc2f
|
|
@github-actions crossbow submit wheel-windows-cp313-cp313-amd64 |
|
Revision: 8e8cd87 Submitted crossbow builds: ursacomputing/crossbow @ actions-f107a3ffcb
|
8e8cd87 to
f4d33c2
Compare
|
@github-actions crossbow submit wheel-windows-cp313-cp313-amd64 |
|
Revision: eac026f Submitted crossbow builds: ursacomputing/crossbow @ actions-19c432ac62
|
|
@github-actions crossbow submit wheel-windows-cp313-cp313-amd64 |
|
Revision: 98410dd Submitted crossbow builds: ursacomputing/crossbow @ actions-7cf85758e1
|
Rationale for this change
This PR removes some duplication by removing
ci/docker/python-wheel-windows-test-vs2022-base.dockerfilein order to use a single base imagepython-wheel-windows-vs2022-basefor both building and testing docker containers.What changes are included in this PR?
python-wheel-windows-test-baseand the related dockerfile.Are these changes tested?
Yes via archery on CI.
Are there any user-facing changes?
No