CI: Cancel stale merge queue runs + use ubuntu-slim (#4106) · apache/iceberg-python@881665b · GitHub
Skip to content

Commit 881665b

Browse files
rambleraptorkevinjqliuCopilot
authored
CI: Cancel stale merge queue runs + use ubuntu-slim (#4106)
* CI: Cancel stale merge queue runs and move license check to ubuntu-slim 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. * CI: Run lightweight required-path jobs on ubuntu-slim 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> * CI: Cancel stale runs for zizmor and markdown link check 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> --------- Co-authored-by: Kevin Liu <kevin.jq.liu@gmail.com> Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
1 parent c528cf6 commit 881665b

7 files changed

Lines changed: 21 additions & 9 deletions

File tree

‎.github/workflows/asf-allowlist-check.yml‎

Lines changed: 1 addition & 1 deletion

‎.github/workflows/check-md-link.yml‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,10 @@ on:
2727
permissions:
2828
contents: read
2929

30+
concurrency:
31+
group: ${{ github.workflow }}-${{ github.ref }}
32+
cancel-in-progress: true
33+
3034
jobs:
3135
markdown-link-check:
3236
runs-on: ubuntu-latest

‎.github/workflows/license_check.yml‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,13 @@ permissions:
2727

2828
jobs:
2929
rat:
30-
runs-on: ubuntu-latest
30+
runs-on: ubuntu-slim
3131
steps:
3232
- uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
3333
with:
3434
persist-credentials: false
35+
- uses: actions/setup-java@de7274f081f381c8f8158605e0321c36c376e2e6 # v6.0.1
36+
with:
37+
distribution: temurin
38+
java-version: 21
3539
- run: dev/check-license

‎.github/workflows/python-ci-docs.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ permissions:
2929

3030
concurrency:
3131
group: ${{ github.workflow }}-${{ github.ref }}
32-
cancel-in-progress: ${{ github.event_name == 'pull_request' }}
32+
cancel-in-progress: true
3333

3434
jobs:
3535
docs:

‎.github/workflows/python-ci.yml‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ permissions:
2828

2929
concurrency:
3030
group: ${{ github.workflow }}-${{ github.ref }}
31-
cancel-in-progress: ${{ github.event_name == 'pull_request' }}
31+
cancel-in-progress: true
3232

3333
jobs:
3434
lint-and-unit-test:
@@ -91,7 +91,7 @@ jobs:
9191
run: uv run python -m pytest tests/ -m unit --ignore=tests/integration -v -x
9292

9393
cibw-dev-env-smoke-test:
94-
runs-on: ubuntu-latest
94+
runs-on: ubuntu-slim
9595
steps:
9696
- uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
9797
with:
@@ -118,7 +118,7 @@ jobs:
118118
python-ci-required:
119119
if: ${{ always() }}
120120
needs: [lint-and-unit-test, windows-unit-test, cibw-dev-env-smoke-test]
121-
runs-on: ubuntu-latest
121+
runs-on: ubuntu-slim
122122
steps:
123123
- name: Verify Python CI jobs succeeded
124124
env:

‎.github/workflows/python-integration.yml‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ permissions:
2828

2929
concurrency:
3030
group: ${{ github.workflow }}-${{ github.ref }}
31-
cancel-in-progress: ${{ github.event_name == 'pull_request' }}
31+
cancel-in-progress: true
3232

3333
jobs:
3434
integration-test:
@@ -152,7 +152,7 @@ jobs:
152152
include-hidden-files: true
153153

154154
integration-coverage-report:
155-
runs-on: ubuntu-latest
155+
runs-on: ubuntu-slim
156156
needs: [integration-test, integration-test-s3, integration-test-adls, integration-test-gcs]
157157
steps:
158158
- uses: actions/checkout@3d3c42e5aac5ba805825da76410c181273ba90b1 # v7.0.1
@@ -178,7 +178,7 @@ jobs:
178178
python-integration-required:
179179
if: ${{ always() }}
180180
needs: [integration-test, integration-test-s3, integration-test-adls, integration-test-gcs, integration-coverage-report]
181-
runs-on: ubuntu-latest
181+
runs-on: ubuntu-slim
182182
steps:
183183
- name: Verify Python integration jobs succeeded
184184
env:

‎.github/workflows/zizmor.yml‎

Lines changed: 4 additions & 0 deletions

0 commit comments

Comments
 (0)