Skip to content

Conversation

@raulcd
Copy link
Member

@raulcd raulcd commented Jan 26, 2026

Rationale for this change

This PR removes some duplication by removing ci/docker/python-wheel-windows-test-vs2022-base.dockerfile in order to use a single base image python-wheel-windows-vs2022-base for both building and testing docker containers.

What changes are included in this PR?

  • Remove python-wheel-windows-test-base and the related dockerfile.
  • Add some requirements for test (minio and test_gcs) on base image so it's available everywhere.
  • Remove manual installation of python on docker images or via choco. This will be only done via pymanager which is installed on the base image.

Are these changes tested?

Yes via archery on CI.

Are there any user-facing changes?

No

@raulcd
Copy link
Member Author

raulcd commented Jan 26, 2026

@github-actions crossbow submit wheel-windows-cp314-cp314-amd64

@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Jan 26, 2026
@github-actions
Copy link

Revision: cb51469

Submitted crossbow builds: ursacomputing/crossbow @ actions-ff4f1aab24

Task Status
wheel-windows-cp314-cp314-amd64 GitHub Actions

@raulcd
Copy link
Member Author

raulcd commented Jan 27, 2026

@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64

@github-actions
Copy link

Revision: cb51469

Submitted crossbow builds: ursacomputing/crossbow @ actions-7fa4698ac5

Task Status
wheel-windows-cp314-cp314t-amd64 GitHub Actions

@raulcd raulcd force-pushed the simplify-windows-docker-setup branch from cb51469 to ac76f96 Compare January 27, 2026 10:31
@raulcd
Copy link
Member Author

raulcd commented Jan 27, 2026

@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64

@github-actions
Copy link

Revision: 054b8dc

Submitted crossbow builds: ursacomputing/crossbow @ actions-9f6150165d

Task Status
wheel-windows-cp314-cp314t-amd64 GitHub Actions

@raulcd
Copy link
Member Author

raulcd commented Jan 27, 2026

@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64

@github-actions
Copy link

Revision: a07afbf

Submitted crossbow builds: ursacomputing/crossbow @ actions-ef0cfd85af

Task Status
wheel-windows-cp314-cp314t-amd64 GitHub Actions

@raulcd
Copy link
Member Author

raulcd commented Jan 28, 2026

@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64

@github-actions
Copy link

Revision: e1ab7da

Submitted crossbow builds: ursacomputing/crossbow @ actions-2eb2d5cfd2

Task Status
wheel-windows-cp314-cp314t-amd64 GitHub Actions

@raulcd
Copy link
Member Author

raulcd commented Jan 28, 2026

@github-actions crossbow submit wheel-windows-*

@raulcd raulcd changed the title WIP: [Python] Simplify Windows wheel build ecosystem WIP: [Python] Remove test base docker image and use single windows base image for both build and test containers Jan 28, 2026
@github-actions
Copy link

Revision: 6665098

Submitted crossbow builds: ursacomputing/crossbow @ actions-d8042cc7a1

Task Status
wheel-windows-cp310-cp310-amd64 GitHub Actions
wheel-windows-cp311-cp311-amd64 GitHub Actions
wheel-windows-cp312-cp312-amd64 GitHub Actions
wheel-windows-cp313-cp313-amd64 GitHub Actions
wheel-windows-cp313-cp313t-amd64 GitHub Actions
wheel-windows-cp314-cp314-amd64 GitHub Actions
wheel-windows-cp314-cp314t-amd64 GitHub Actions

@raulcd
Copy link
Member Author

raulcd commented Jan 28, 2026

@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64

@github-actions
Copy link

Revision: 2aa3a1f

Submitted crossbow builds: ursacomputing/crossbow @ actions-22c758b35f

Task Status
wheel-windows-cp314-cp314t-amd64 GitHub Actions

@raulcd
Copy link
Member Author

raulcd commented Jan 28, 2026

@github-actions crossbow submit wheel-windows-cp314-cp314t-amd64

@github-actions
Copy link

Revision: 1ca2938

Submitted crossbow builds: ursacomputing/crossbow @ actions-a84a844d69

Task Status
wheel-windows-cp314-cp314t-amd64 GitHub Actions

@raulcd
Copy link
Member Author

raulcd commented Jan 29, 2026

@github-actions crossbow submit wheel-windows-*

@github-actions
Copy link

Revision: 1ca2938

Submitted crossbow builds: ursacomputing/crossbow @ actions-bb12f9cd6a

Task Status
wheel-windows-cp310-cp310-amd64 GitHub Actions
wheel-windows-cp311-cp311-amd64 GitHub Actions
wheel-windows-cp312-cp312-amd64 GitHub Actions
wheel-windows-cp313-cp313-amd64 GitHub Actions
wheel-windows-cp313-cp313t-amd64 GitHub Actions
wheel-windows-cp314-cp314-amd64 GitHub Actions
wheel-windows-cp314-cp314t-amd64 GitHub Actions

@raulcd raulcd changed the title WIP: [Python] Remove test base docker image and use single windows base image for both build and test containers GH-49057: [Python][Packaging] Remove test base docker image and use single windows base image for both build and test containers Jan 29, 2026
@github-actions
Copy link

⚠️ GitHub issue #49057 has been automatically assigned in GitHub to PR creator.

@raulcd raulcd marked this pull request as ready for review January 29, 2026 09:50
@raulcd raulcd requested review from AlenkaF and pitrou January 29, 2026 09:50
@raulcd
Copy link
Member Author

raulcd commented Jan 29, 2026

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.
Copy link
Member

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?

Copy link
Member

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)

Copy link
Member Author

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.

Copy link
Member

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 github-actions bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Jan 29, 2026
@raulcd
Copy link
Member Author

raulcd commented Jan 29, 2026

@github-actions crossbow submit wheel-windows-cp313-cp313-amd64

@github-actions github-actions bot added awaiting change review Awaiting change review and removed awaiting changes Awaiting changes labels Jan 29, 2026
@github-actions
Copy link

Revision: 73ca678

Submitted crossbow builds: ursacomputing/crossbow @ actions-17f326e4c3

Task Status
wheel-windows-cp313-cp313-amd64 GitHub Actions

@raulcd
Copy link
Member Author

raulcd commented Jan 29, 2026

@github-actions crossbow submit wheel-windows-cp313-cp313-amd64

@github-actions
Copy link

Revision: 8bbf7d5

Submitted crossbow builds: ursacomputing/crossbow @ actions-437184cc2f

Task Status
wheel-windows-cp313-cp313-amd64 GitHub Actions

@raulcd
Copy link
Member Author

raulcd commented Jan 29, 2026

@github-actions crossbow submit wheel-windows-cp313-cp313-amd64

@github-actions
Copy link

Revision: 8e8cd87

Submitted crossbow builds: ursacomputing/crossbow @ actions-f107a3ffcb

Task Status
wheel-windows-cp313-cp313-amd64 GitHub Actions

@raulcd raulcd force-pushed the simplify-windows-docker-setup branch from 8e8cd87 to f4d33c2 Compare January 30, 2026 09:48
@raulcd
Copy link
Member Author

raulcd commented Jan 30, 2026

@github-actions crossbow submit wheel-windows-cp313-cp313-amd64

@github-actions
Copy link

Revision: eac026f

Submitted crossbow builds: ursacomputing/crossbow @ actions-19c432ac62

Task Status
wheel-windows-cp313-cp313-amd64 GitHub Actions

@raulcd
Copy link
Member Author

raulcd commented Jan 30, 2026

@github-actions crossbow submit wheel-windows-cp313-cp313-amd64

@github-actions
Copy link

Revision: 98410dd

Submitted crossbow builds: ursacomputing/crossbow @ actions-7cf85758e1

Task Status
wheel-windows-cp313-cp313-amd64 GitHub Actions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting change review Awaiting change review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants