Skip to content

CI: trim build matrix on macOS/Windows and add workflow concurrency#98

Open
vharseko wants to merge 2 commits into
OpenIdentityPlatform:masterfrom
vharseko:features/ci-build-matrix-concurrency
Open

CI: trim build matrix on macOS/Windows and add workflow concurrency#98
vharseko wants to merge 2 commits into
OpenIdentityPlatform:masterfrom
vharseko:features/ci-build-matrix-concurrency

Conversation

@vharseko

@vharseko vharseko commented Jul 9, 2026

Copy link
Copy Markdown
Member

Analogous to OpenAM #1064 and #1065.

Trim build matrix (like OpenAM #1064)

build.yml ran a full 5×3 matrix (JDK 11/17/21/25/26 × Ubuntu/macOS/Windows = 15 jobs). macOS and Windows runners are the slowest and scarcest in CI and rarely surface OS-specific issues beyond the JDK edges, so:

  • Ubuntu keeps the full JDK matrix (11, 17, 21, 25, 26).
  • macOS and Windows now smoke-test only the minimum (11) and maximum (26) JDKs via include.
  • Total jobs: 15 → 9.

Workflow concurrency (like OpenAM #1065)

Added concurrency groups keyed on ${{ github.workflow }}-${{ github.ref }}:

  • build.yml: cancel-in-progress: true — a new push/PR cancels a superseded in-flight build.
  • release.yml / deploy.yml: cancel-in-progress: false — overlapping runs queue instead of being interrupted, protecting Maven Central publishing / GitHub releases.

codeql.yml is intentionally left untouched.

- build.yml: run the full JDK matrix (11, 17, 21, 25, 26) only on Ubuntu;
  macOS and Windows now smoke-test just the min (11) and max (26) JDKs,
  cutting the matrix from 15 jobs to 9.
- Add concurrency groups keyed on workflow+ref: cancel-in-progress for
  build.yml, and cancel-in-progress: false for release.yml and deploy.yml
  so publishing/release runs queue instead of being interrupted.

Analogous to OpenAM #1064 and #1065.
@vharseko vharseko requested a review from maximthomas July 9, 2026 13:06
@vharseko vharseko added the ci CI, build & workflow changes label Jul 9, 2026
Comment thread .github/workflows/deploy.yml Outdated
Co-authored-by: Maxim Thomas <maxim.thomas@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI, build & workflow changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants