Skip to content

MINOR: [Integration][CI] Remove Python pinning in integration build - #51099

Merged
pitrou merged 1 commit into
apache:mainfrom
pitrou:minor-pins
Sep 8, 2026
Merged

pitrou merged 1 commit into
apache:mainfrom
pitrou:minor-pins

Conversation

@pitrou

@pitrou pitrou commented Aug 31, 2026

Copy link
Copy Markdown
Member

Rationale for this change

Pythonnet now seems to be compatible with Python versions up to 3.15:
https://github.com/pythonnet/pythonnet/blob/d2d27164cdcb62ea022a51ffc3ed78f338f01803/.github/workflows/main.yml#L49

Are these changes tested?

Yes, by corresponding CI job.

Are there any user-facing changes?

No.

@pitrou

pitrou commented Aug 31, 2026

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit conda ubuntuverify*

@github-actions github-actions Bot added the awaiting review Awaiting review label Aug 31, 2026
@github-actions

Copy link
Copy Markdown
Unable to match any tasks for `*ubuntu*verify*`
The Archery job run can be found at: https://github.com/apache/arrow/actions/runs/33399675668

@pitrou

pitrou commented Aug 31, 2026

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit conda verifyubuntu*

@github-actions

This comment was marked as outdated.

@pitrou pitrou changed the title MINOR: [Dev][CI] Remove outdated pins MINOR: [Integration][CI] Remove Python pinning in integration build Aug 31, 2026
@pitrou
pitrou marked this pull request as ready for review August 31, 2026 15:02
@kou

kou commented Aug 31, 2026

Copy link
Copy Markdown
Member

https://github.com/apache/arrow/actions/runs/33406037337/job/99533744366?pr=51099#step:13:346

  #9 26.27   python-3.11.16-h8ab3286_0_cpython    102MB

Hmm. Python 3.11 was still used.

@pitrou

pitrou commented Sep 1, 2026

Copy link
Copy Markdown
Member Author

Yes, weirdly. The constraint solver may be doing a suboptimal choice.

@pitrou

pitrou commented Sep 7, 2026

Copy link
Copy Markdown
Member Author

Ping @raulcd

@raulcd

raulcd commented Sep 8, 2026

Copy link
Copy Markdown
Member

the ci/docker/conda-integration.dockerfile uses ci/docker/conda-cpp.dockerfile as base and there we define Python 3.11, see:

ARG python=3.11

We could bump the base one or override and install a newer one on the conda-integration.dockerfile file.

@pitrou

pitrou commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

@raulcd Oh, you're right. I think we can bump this version.

@pitrou

pitrou commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit conda

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

Revision: 16d7028

Submitted crossbow builds: ursacomputing/crossbow @ actions-d78c302548

Task Status
example-python-minimal-build-fedora-conda GitHub Actions
test-conda-cpp GitHub Actions
test-conda-cpp-valgrind GitHub Actions
test-conda-python-3.11 GitHub Actions
test-conda-python-3.11-dask-latest GitHub Actions
test-conda-python-3.11-dask-upstream_devel GitHub Actions
test-conda-python-3.11-hdfs-2.9.2 GitHub Actions
test-conda-python-3.11-hdfs-3.2.1 GitHub Actions
test-conda-python-3.11-hypothesis GitHub Actions
test-conda-python-3.11-pandas-2.2.2-numpy-2.0.2 GitHub Actions
test-conda-python-3.11-spark-master GitHub Actions
test-conda-python-3.12 GitHub Actions
test-conda-python-3.12-pandas-latest-numpy-latest GitHub Actions
test-conda-python-3.13 GitHub Actions
test-conda-python-3.13-pandas-latest-numpy-2.1.3 GitHub Actions
test-conda-python-3.13-pandas-latest-numpy-latest GitHub Actions
test-conda-python-3.14 GitHub Actions
test-conda-python-3.14-cpython-debug GitHub Actions
test-conda-python-3.14-pandas-nightly-numpy-nightly GitHub Actions
test-conda-python-3.14-pandas-upstream_devel-numpy-nightly GitHub Actions
test-conda-python-emscripten GitHub Actions
verify-rc-source-cpp-linux-conda-latest-amd64 GitHub Actions
verify-rc-source-cpp-macos-conda-amd64 GitHub Actions
verify-rc-source-integration-linux-conda-latest-amd64 GitHub Actions
verify-rc-source-integration-macos-conda-amd64 GitHub Actions
verify-rc-source-python-linux-conda-latest-amd64 GitHub Actions
verify-rc-source-python-macos-conda-amd64 GitHub Actions
verify-rc-source-ruby-linux-conda-latest-amd64 GitHub Actions

@raulcd

raulcd commented Sep 8, 2026

Copy link
Copy Markdown
Member

Seems to use 3.14 now: https://github.com/apache/arrow/actions/runs/34210589556/job/102010384170?pr=51099

  #9 25.09   Package folder                       Size
  #9 25.09 ─────────────────────────────────────────────
  #9 25.09   /opt/conda/pkgs
  #9 25.09 ─────────────────────────────────────────────
  #9 25.09 
  #9 25.09   libffi-3.7.0-h81df57d_1             271kB
  #9 25.09   nodejs-24.19.0-h50021de_1            76MB
  #9 25.09   python-3.14.7-hcd007b5_106_cp314     94MB

@pitrou

pitrou commented Sep 8, 2026

Copy link
Copy Markdown
Member Author

I'll merge then.

Side question: there are many places where an old Python version is hard-coded as default, for example:

I don't think it matters because usually those would be overriden through compose.yaml and the PYTHON environment variable, but do we want to bump them anyway?

@pitrou
pitrou merged commit 7769d54 into apache:main Sep 8, 2026
56 of 57 checks passed
@pitrou pitrou removed the awaiting review Awaiting review label Sep 8, 2026
@pitrou
pitrou deleted the minor-pins branch September 8, 2026 12:43
@raulcd

raulcd commented Sep 8, 2026

Copy link
Copy Markdown
Member

I don't think it matters because usually those would be overriden through compose.yaml and the PYTHON environment variable, but do we want to bump them anyway?

I usually bump all those when an old Python version is EOL'd. I am fine with both bumping them or just keep testing with the oldest Python supported.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants