From 0fe6d2549b494c9c44598aeb3009bbdc6faa2eac Mon Sep 17 00:00:00 2001 From: ehl-jf Date: Wed, 17 Jun 2026 10:36:04 +0200 Subject: [PATCH] JGC-499 - Remove concurrency from reusable test suites When invoked via workflow_call, each suite's `concurrency` group resolves using the caller's context (github.workflow = "Build Gate"), so all suites and the parent run share one identical group. With cancel-in-progress: true they cancelled each other before starting, leaving only frogbot (which has no concurrency block) to run and making build-gate-success fail on cancelled needs. The orchestrator's own concurrency already supersedes older runs per PR/branch, so the per-suite blocks are removed. --- .github/workflows/accessTests.yml | 4 ---- .github/workflows/artifactoryTests.yml | 4 ---- .github/workflows/conanTests.yml | 3 --- .github/workflows/distributionTests.yml | 4 ---- .github/workflows/dockerTests.yml | 4 ---- .github/workflows/evidenceTests.yml | 5 ----- .github/workflows/ghostFrogTests.yml | 4 ---- .github/workflows/goTests.yml | 4 ---- .github/workflows/gradleTests.yml | 4 ---- .github/workflows/helmTests.yml | 5 ----- .github/workflows/huggingfaceTests.yml | 5 ----- .github/workflows/lifecycleTests.yml | 4 ---- .github/workflows/mavenTests.yml | 4 ---- .github/workflows/nixTests.yml | 3 --- .github/workflows/npmTests.yml | 4 ---- .github/workflows/nugetTests.yml | 5 ----- .github/workflows/oidcTests.yml | 5 ----- .github/workflows/pluginsTests.yml | 4 ---- .github/workflows/pnpmTests.yml | 4 ---- .github/workflows/podmanTests.yml | 4 ---- .github/workflows/poetryTests.yml | 5 ----- .github/workflows/pythonTests.yml | 4 ---- .github/workflows/scriptTests.yml | 4 ---- .github/workflows/transferTests.yml | 4 ---- .github/workflows/uvTests.yml | 5 ----- 25 files changed, 105 deletions(-) diff --git a/.github/workflows/accessTests.yml b/.github/workflows/accessTests.yml index 62f9ec488..d7bad4c3f 100644 --- a/.github/workflows/accessTests.yml +++ b/.github/workflows/accessTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true permissions: id-token: write contents: read diff --git a/.github/workflows/artifactoryTests.yml b/.github/workflows/artifactoryTests.yml index fcb9ad60b..b562ffa23 100644 --- a/.github/workflows/artifactoryTests.yml +++ b/.github/workflows/artifactoryTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Artifactory-Tests: name: ${{ matrix.suite }} ${{ matrix.os.name }} diff --git a/.github/workflows/conanTests.yml b/.github/workflows/conanTests.yml index 452d0b7b7..2c4c0bf55 100644 --- a/.github/workflows/conanTests.yml +++ b/.github/workflows/conanTests.yml @@ -14,9 +14,6 @@ on: required: false default: "" -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Conan-Tests: name: Conan tests (${{ matrix.os.name }}) diff --git a/.github/workflows/distributionTests.yml b/.github/workflows/distributionTests.yml index 64f335bc1..52d4bd059 100644 --- a/.github/workflows/distributionTests.yml +++ b/.github/workflows/distributionTests.yml @@ -3,10 +3,6 @@ on: workflow_call: workflow_dispatch: -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Distribution-Tests: name: Distribution tests (${{ matrix.os }}) diff --git a/.github/workflows/dockerTests.yml b/.github/workflows/dockerTests.yml index 761589f4f..b3a76d4a2 100644 --- a/.github/workflows/dockerTests.yml +++ b/.github/workflows/dockerTests.yml @@ -3,10 +3,6 @@ on: workflow_call: workflow_dispatch: -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Docker-tests: name: Docker tests (${{ matrix.os.name }}, containerd-snapshotter=${{ !matrix.disable-containerd-snapshotter }}) diff --git a/.github/workflows/evidenceTests.yml b/.github/workflows/evidenceTests.yml index 3fca11723..c694a4f25 100644 --- a/.github/workflows/evidenceTests.yml +++ b/.github/workflows/evidenceTests.yml @@ -3,10 +3,6 @@ on: workflow_call: workflow_dispatch: -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Evidence-Tests: name: Evidence tests (${{ matrix.os }}) @@ -44,4 +40,3 @@ jobs: - name: Run Evidence tests run: go test -v github.com/jfrog/jfrog-cli --timeout 0 --test.evidence --jfrog.url=${{ secrets.EPLUS_PLATFORM_URL }} --jfrog.adminToken=${{ secrets.EPLUS_ADMIN_TOKEN }} --jfrog.evidenceToken=${{ secrets.EVIDENCE_USER_TOKEN }} --jfrog.projectKey=${{ secrets.EVIDENCE_PROJECT_KEY }} --jfrog.projectToken=${{ secrets.EVIDENCE_PROJECT_TOKEN }} --ci.runId=${{ runner.os }}-evidence - diff --git a/.github/workflows/ghostFrogTests.yml b/.github/workflows/ghostFrogTests.yml index 607a87274..a00a6362e 100644 --- a/.github/workflows/ghostFrogTests.yml +++ b/.github/workflows/ghostFrogTests.yml @@ -3,10 +3,6 @@ on: workflow_call: workflow_dispatch: -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Ghost-Frog-Tests: name: Ghost Frog tests (${{ matrix.os.name }}) diff --git a/.github/workflows/goTests.yml b/.github/workflows/goTests.yml index 5bcb445dd..356adc90c 100644 --- a/.github/workflows/goTests.yml +++ b/.github/workflows/goTests.yml @@ -6,10 +6,6 @@ on: workflow_call: workflow_dispatch: -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: GO-tests: # Go modules doesn't allow passing credentials to a private registry using an HTTP URL. Therefore, the Go tests run against a remote Artifactory server. diff --git a/.github/workflows/gradleTests.yml b/.github/workflows/gradleTests.yml index 5d8d32aeb..60abff632 100644 --- a/.github/workflows/gradleTests.yml +++ b/.github/workflows/gradleTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Gradle-Tests: name: ${{ matrix.os.name }}-gradle-${{ matrix.gradle-version }} diff --git a/.github/workflows/helmTests.yml b/.github/workflows/helmTests.yml index 3801232ed..9b4ad2234 100644 --- a/.github/workflows/helmTests.yml +++ b/.github/workflows/helmTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true permissions: id-token: write contents: read @@ -102,4 +98,3 @@ jobs: go test -v github.com/jfrog/jfrog-cli --timeout 0 --test.helm --jfrog.url=${{ env.JFROG_TESTS_URL || 'http://127.0.0.1:8082' }} --jfrog.adminToken=${{ env.JFROG_TESTS_LOCAL_ACCESS_TOKEN }} - diff --git a/.github/workflows/huggingfaceTests.yml b/.github/workflows/huggingfaceTests.yml index 84c7dd8aa..7f8456a66 100644 --- a/.github/workflows/huggingfaceTests.yml +++ b/.github/workflows/huggingfaceTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true permissions: id-token: write contents: read @@ -107,4 +103,3 @@ jobs: go test -v github.com/jfrog/jfrog-cli --timeout 0 --test.huggingface --jfrog.url=${{ env.JFROG_TESTS_URL || 'http://127.0.0.1:8082' }} --jfrog.adminToken=${{ env.JFROG_TESTS_LOCAL_ACCESS_TOKEN }} - diff --git a/.github/workflows/lifecycleTests.yml b/.github/workflows/lifecycleTests.yml index 68b934a9e..db938ed13 100644 --- a/.github/workflows/lifecycleTests.yml +++ b/.github/workflows/lifecycleTests.yml @@ -16,10 +16,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Lifecycle-Tests: name: Lifecycle tests (${{ matrix.os.name }}) diff --git a/.github/workflows/mavenTests.yml b/.github/workflows/mavenTests.yml index f6114e9e2..a061feff6 100644 --- a/.github/workflows/mavenTests.yml +++ b/.github/workflows/mavenTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Maven-Tests: name: Maven tests (${{ matrix.os.name }}) diff --git a/.github/workflows/nixTests.yml b/.github/workflows/nixTests.yml index 78e91af47..a01fda073 100644 --- a/.github/workflows/nixTests.yml +++ b/.github/workflows/nixTests.yml @@ -14,9 +14,6 @@ on: required: false default: "" -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Nix-Tests: name: Nix tests (${{ matrix.os.name }}) diff --git a/.github/workflows/npmTests.yml b/.github/workflows/npmTests.yml index 025816028..3cc787b51 100644 --- a/.github/workflows/npmTests.yml +++ b/.github/workflows/npmTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: npm-Tests: name: npm tests (${{ matrix.os.name }}) diff --git a/.github/workflows/nugetTests.yml b/.github/workflows/nugetTests.yml index 3fcff9e37..dde9a7572 100644 --- a/.github/workflows/nugetTests.yml +++ b/.github/workflows/nugetTests.yml @@ -15,11 +15,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true - jobs: NuGet-Tests: name: NuGet tests (${{ matrix.os.name }}) diff --git a/.github/workflows/oidcTests.yml b/.github/workflows/oidcTests.yml index bf48ffe15..04d9d6034 100644 --- a/.github/workflows/oidcTests.yml +++ b/.github/workflows/oidcTests.yml @@ -5,11 +5,6 @@ on: workflow_call: workflow_dispatch: -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true - env: JFROG_CLI_LOG_LEVEL: DEBUG diff --git a/.github/workflows/pluginsTests.yml b/.github/workflows/pluginsTests.yml index 93e356ffb..ce91fbe3d 100644 --- a/.github/workflows/pluginsTests.yml +++ b/.github/workflows/pluginsTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Plugins-Tests: name: Plugins tests (${{ matrix.os.name }}) diff --git a/.github/workflows/pnpmTests.yml b/.github/workflows/pnpmTests.yml index b8989ccbe..15dd32dd8 100644 --- a/.github/workflows/pnpmTests.yml +++ b/.github/workflows/pnpmTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true - jobs: pnpm-Tests: name: "pnpm 10 tests (${{ matrix.os.name }})" diff --git a/.github/workflows/podmanTests.yml b/.github/workflows/podmanTests.yml index 9ed6b479b..1375576f8 100644 --- a/.github/workflows/podmanTests.yml +++ b/.github/workflows/podmanTests.yml @@ -3,10 +3,6 @@ on: workflow_call: workflow_dispatch: -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Podman-tests: name: Podman tests (${{ matrix.os.name }}) diff --git a/.github/workflows/poetryTests.yml b/.github/workflows/poetryTests.yml index 01ad86394..0ac609a9a 100644 --- a/.github/workflows/poetryTests.yml +++ b/.github/workflows/poetryTests.yml @@ -19,11 +19,6 @@ on: required: false default: "1.8.5" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true - jobs: Poetry-Tests: name: poetry ${{ matrix.os.name }} diff --git a/.github/workflows/pythonTests.yml b/.github/workflows/pythonTests.yml index 6741df604..eab027e9c 100644 --- a/.github/workflows/pythonTests.yml +++ b/.github/workflows/pythonTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Python-Tests: name: ${{ matrix.suite }} ${{ matrix.os.name }} diff --git a/.github/workflows/scriptTests.yml b/.github/workflows/scriptTests.yml index 186b4752c..5970488e3 100644 --- a/.github/workflows/scriptTests.yml +++ b/.github/workflows/scriptTests.yml @@ -3,10 +3,6 @@ on: workflow_call: workflow_dispatch: -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Scripts-tests: name: Script tests (${{ matrix.os.name }}-${{ matrix.os.version }}) diff --git a/.github/workflows/transferTests.yml b/.github/workflows/transferTests.yml index 7bf68bbf1..6594a0fc9 100644 --- a/.github/workflows/transferTests.yml +++ b/.github/workflows/transferTests.yml @@ -14,10 +14,6 @@ on: required: false default: "" -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true jobs: Transfer-Artifactory-7-Tests: name: Transfer tests (${{ matrix.os.name }}) diff --git a/.github/workflows/uvTests.yml b/.github/workflows/uvTests.yml index bbb876115..2b22ba185 100644 --- a/.github/workflows/uvTests.yml +++ b/.github/workflows/uvTests.yml @@ -3,11 +3,6 @@ on: workflow_call: workflow_dispatch: -# Ensures that only the latest commit is running for each PR at a time. -concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.ref }} - cancel-in-progress: true - jobs: UV-Tests: name: uv ${{ matrix.os.name }} py${{ matrix.python-version }} uv${{ matrix.uv-version }}