diff options
| author | Yuriy Andamasov <yuriy@vyos.io> | 2026-07-12 16:42:21 +0300 |
|---|---|---|
| committer | Yuriy Andamasov <yuriy@vyos.io> | 2026-07-12 16:42:21 +0300 |
| commit | 01a7a7b939af3eba31e776dc07e0e7bee3c47874 (patch) | |
| tree | 00667f3bfc2e0d15af535a1e7f4edde640c238cd | |
| parent | d353f306c7fb85b76221b2f953abe6b0680e7df3 (diff) | |
| download | vyos-documentation-01a7a7b939af3eba31e776dc07e0e7bee3c47874.tar.gz vyos-documentation-01a7a7b939af3eba31e776dc07e0e7bee3c47874.zip | |
docs-infra: run workers tests on PRs (gap let broken deps land)
apex-deploy.yml's test job only triggered on push to rolling, so no
PR-level check ran the workers/ vitest suite โ that's how the vitest
3.2/pool-workers 0.18.4 peer conflict landed on rolling undetected
(two dependabot PRs merged clean since nothing exercised `npm ci` in
workers/ pre-merge).
Add a `pull_request` trigger scoped to `workers/**` so the test job
runs pre-merge. Gate deploy-canary and deploy-production with
`if: github.event_name != 'pull_request'` so PR runs test only โ
no canary/production deploys fire on a PR (deploy-production would
also auto-skip via `needs: deploy-canary` once canary skips, but the
explicit `if:` keeps intent visible). Concurrency group now branches
per-event so PR runs queue/cancel independently of the rolling deploy
group instead of colliding with it.
Validated with actionlint + a YAML parse check.
๐ค Generated by [robots](https://vyos.io)
| -rw-r--r-- | .github/workflows/apex-deploy.yml | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/.github/workflows/apex-deploy.yml b/.github/workflows/apex-deploy.yml index 1528221b..efa42fd7 100644 --- a/.github/workflows/apex-deploy.yml +++ b/.github/workflows/apex-deploy.yml @@ -4,10 +4,14 @@ on: push: branches: [rolling] paths: ["workers/**"] + pull_request: + paths: ["workers/**"] workflow_dispatch: {} concurrency: - group: apex-deploy-${{ github.ref_name }} + # PR runs group per head ref so pushes to the same PR queue/cancel independently + # of the rolling deploy group. + group: apex-deploy-${{ github.event_name == 'pull_request' && github.head_ref || github.ref_name }} # false (not true): a mid-job cancel can leave the apex/preview worker pair # half-updated; queuing subsequent runs is safe. cancel-in-progress: false @@ -26,6 +30,7 @@ jobs: deploy-canary: needs: test + if: github.event_name != 'pull_request' runs-on: ubuntu-latest steps: - uses: actions/checkout@v4 @@ -58,6 +63,7 @@ jobs: deploy-production: needs: deploy-canary + if: github.event_name != 'pull_request' runs-on: ubuntu-latest environment: docs-production # requires reviewer approval (ยง7.2) steps: |