Skip to content

CI: Cancel stale merge queue runs + use ubuntu-slim - #4106

Merged
kevinjqliu merged 3 commits into
apache:mainfrom
rambleraptor:ci-merge-queue-runners
Oct 20, 2026
Merged

kevinjqliu merged 3 commits into
apache:mainfrom
rambleraptor:ci-merge-queue-runners

Conversation

@rambleraptor

Copy link
Copy Markdown
Collaborator

Rationale for this change

We're having trouble merging in #4100. The ultimate issue is a lack of ubuntu-latest runners, but there's a couple easy wins we can get to help out.

  1. cancel-in-progress only works on PRs, which makes sense in a pre-merge queue world. We should open it up everywhere, so that merge queue runs aren't stalled forever.
  2. We should try to move runners over to ubuntu-slim when possible. That image has very little baked in, so it's really hard for us to do in practice. The license checker is the easiest win, so let's go ahead and give that a try.

Are these changes tested?

We expect CI to still work + we're hoping that this will help merging in #4100 on the edges.

Are there any user-facing changes?

@Fokko

Fokko commented Oct 19, 2026

Copy link
Copy Markdown
Contributor

@rambleraptor Can you pull in latest main?

A PR re-added to the merge queue reuses the same queue branch, so its
runs waited behind the previous attempt's still-queued runs in the same
concurrency group. Cancel in-progress runs for merge_group as well.

Run the license check on ubuntu-slim to reduce demand for ubuntu-latest
runners. That image has no Java, so install it explicitly.
@rambleraptor
rambleraptor force-pushed the ci-merge-queue-runners branch from fecfa1e to 6a3d1ed Compare October 19, 2026 18:36
@rambleraptor

Copy link
Copy Markdown
Collaborator Author

@Fokko just pulled in! With #4100 now merged in, this PR should be green.

@Fokko Fokko left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @rambleraptor, appreciate it 👍

@rambleraptor rambleraptor mentioned this pull request Oct 19, 2026
9 tasks
@Fokko
Fokko added this pull request to the merge queue Oct 19, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Oct 19, 2026
kevinjqliu and others added 2 commits October 19, 2026 16:48
The required gate jobs, ASF allowlist check, integration coverage report,
and cibw dev env smoke test only need shell and Python, so ubuntu-slim
runners suffice and free ubuntu-latest capacity for PR and merge queue runs.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Both workflows run on ubuntu-latest, so superseded queued runs compete
with required CI for scarce runners.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@kevinjqliu

Copy link
Copy Markdown
Contributor

pushed a few more similar changes to use slim. thanks!

@kevinjqliu
kevinjqliu enabled auto-merge October 19, 2026 23:49
@kevinjqliu
kevinjqliu added this pull request to the merge queue Oct 20, 2026
Merged via the queue into apache:main with commit 881665b Oct 20, 2026
21 checks passed
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