Skip to content

MINOR: [CI] Set max-parallel=20#49228

Merged
kou merged 1 commit intoapache:mainfrom
rok:set_max-parallel_for_python_workflow
Feb 12, 2026
Merged

MINOR: [CI] Set max-parallel=20#49228
kou merged 1 commit intoapache:mainfrom
rok:set_max-parallel_for_python_workflow

Conversation

@rok
Copy link
Member

@rok rok commented Feb 11, 2026

Rationale for this change

We want to comply with ASF policy

What changes are included in this PR?

Sets max-parallel=20 in .github/workflows/python.yml

Are these changes tested?

No.

Are there any user-facing changes?

Only to CI users.

@rok rok marked this pull request as ready for review February 11, 2026 15:09
@github-actions github-actions bot added the awaiting committer review Awaiting committer review label Feb 11, 2026
Copy link
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but I am wondering why this has been flagged here as a problem but not on other workfows like the CPP one:

strategy:
fail-fast: false
matrix:
include:
- arch: amd64
clang-tools: 18
image: conda-cpp
llvm: 18
runs-on: ubuntu-latest
simd-level: AVX2
title: AMD64 Conda C++ AVX2
ubuntu: 22.04
- arch: amd64
clang-tools: 18
image: ubuntu-cpp-sanitizer
llvm: 18
runs-on: ubuntu-latest
title: AMD64 Ubuntu 24.04 C++ ASAN UBSAN
ubuntu: 24.04
- arch: arm64v8
clang-tools: 18
image: ubuntu-cpp
llvm: 18
runs-on: ubuntu-24.04-arm
title: ARM64 Ubuntu 22.04 C++
ubuntu: 22.04

Seems slightly arbitrary. @assignUser any idea?

@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Feb 11, 2026
@assignUser
Copy link
Member

@raulcd I assume it's the way the check script calculates the parallelism not properly handeling include: but I don't know

Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kou kou merged commit a444c87 into apache:main Feb 12, 2026
19 of 20 checks passed
@kou kou removed the awaiting changes Awaiting changes label Feb 12, 2026
@github-actions github-actions bot added the awaiting merge Awaiting merge label Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting merge Awaiting merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants