Skip to content

Commit d1000c8

Browse files
authored
fix(ci): Add cancellation logic where missing (#26327)
## Description <!---Describe your changes in detail--> ## Motivation and Context <!---Why is this change required? What problem does it solve?--> <!---If it fixes an open issue, please link to the issue here.--> ## Impact <!---Describe any public API or user-facing feature change or any performance impact--> ## Test Plan <!---Please fill in how you tested your change--> ## Contributor checklist - [ ] Please make sure your submission complies with our [contributing guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md), in particular [code style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style) and [commit standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards). - [ ] PR description addresses the issue accurately and concisely. If the change is non-trivial, a GitHub Issue is referenced. - [ ] Documented new properties (with its default value), SQL syntax, functions, or other functionality. - [ ] If release notes are required, they follow the [release notes guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines). - [ ] Adequate tests were added if applicable. - [ ] CI passed. - [ ] If adding new dependencies, verified they have an [OpenSSF Scorecard](https://securityscorecards.dev/#the-checks) score of 5.0 or higher (or obtained explicit TSC approval for lower scores). ## Release Notes Please follow [release notes guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines) and fill in the release notes below. ``` == NO RELEASE NOTE == ```
1 parent cfb8cea commit d1000c8

File tree

3 files changed

+15
-0
lines changed

3 files changed

+15
-0
lines changed

.github/workflows/arrow-flight-tests.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,9 @@ jobs:
6161
runs-on: ubuntu-22.04
6262
container:
6363
image: prestodb/presto-native-dependency:0.292-20250204112033-cf8ba84
64+
concurrency:
65+
group: ${{ github.workflow }}-prestocpp-linux-build-for-test-${{ github.event.pull_request.number }}
66+
cancel-in-progress: true
6467
env:
6568
CCACHE_DIR: "${{ github.workspace }}/ccache"
6669
DEPENDENCY_DIR: "${{ github.workspace }}/adapter-deps/download"

.github/workflows/owasp-dependency-check.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,9 @@ on:
1414
jobs:
1515
dependency-check:
1616
runs-on: ubuntu-latest
17+
concurrency:
18+
group: ${{ github.workflow }}-owasp-dependency-check-${{ github.event.pull_request.number }}
19+
cancel-in-progress: true
1720
env:
1821
CVSS_THRESHOLD: ${{ github.event.inputs.cvss-threshold || '7.0' }}
1922
OWASP_VERSION: 12.1.3

.github/workflows/prestocpp-linux-build-and-unit-test.yml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -339,6 +339,9 @@ jobs:
339339
matrix:
340340
storage-format: [PARQUET, DWRF]
341341
enable-sidecar: ["true", "false"]
342+
concurrency:
343+
group: ${{ github.workflow }}-prestocpp-linux-presto-on-spark-e2e-tests-${{ matrix.storage-format }}-${{ matrix.enable-sidecar }}-${{ github.event.pull_request.number }}
344+
cancel-in-progress: true
342345
container:
343346
image: prestodb/presto-native-dependency:0.296-20251021214746-7eb2686
344347
env:
@@ -434,6 +437,9 @@ jobs:
434437
runs-on: ubuntu-22.04
435438
container:
436439
image: prestodb/presto-native-dependency:0.296-20251021214746-7eb2686
440+
concurrency:
441+
group: ${{ github.workflow }}-prestocpp-linux-presto-sidecar-tests-${{ github.event.pull_request.number }}
442+
cancel-in-progress: true
437443
env:
438444
MAVEN_OPTS: -Xmx4G -XX:+ExitOnOutOfMemoryError
439445
MAVEN_FAST_INSTALL: -B -V --quiet -T 1C -DskipTests -Dair.check.skip-all -Dmaven.javadoc.skip=true
@@ -523,6 +529,9 @@ jobs:
523529
runs-on: ubuntu-22.04
524530
container:
525531
image: prestodb/presto-native-dependency:0.296-20251021214746-7eb2686
532+
concurrency:
533+
group: ${{ github.workflow }}-prestocpp-linux-presto-plan-checker-router-plugin-tests-${{ github.event.pull_request.number }}
534+
cancel-in-progress: true
526535
env:
527536
MAVEN_OPTS: -Xmx4G -XX:+ExitOnOutOfMemoryError
528537
MAVEN_FAST_INSTALL: -B -V --quiet -T 1C -DskipTests -Dair.check.skip-all -Dmaven.javadoc.skip=true

0 commit comments

Comments
 (0)