Skip to content

Commit ce6e8f9

Browse files
committed
Force Snapshot Build is separate workflow
Previously builds against the forced snapshots would prevent releases. This didn't work out in practice because breaking changes could be introduced that were not intended to be picked up until after releases. This change splits the snapshot builds into a separate workflow so that we are still informed when there is a breaking change coming, but it does not block the release. Closes gh-17558
1 parent 66ddfb4 commit ce6e8f9

File tree

2 files changed

+41
-17
lines changed

2 files changed

+41
-17
lines changed

.github/workflows/check-snapshots.yml

Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
name: CI
2+
3+
on:
4+
schedule:
5+
- cron: '0 10 * * *' # Once per day at 10am UTC
6+
workflow_dispatch: # Manual trigger
7+
8+
env:
9+
DEVELOCITY_ACCESS_KEY: ${{ secrets.DEVELOCITY_ACCESS_KEY }}
10+
11+
permissions:
12+
contents: read
13+
14+
jobs:
15+
snapshot-test:
16+
name: Test Against Snapshots
17+
uses: spring-io/spring-security-release-tools/.github/workflows/test.yml@v1
18+
strategy:
19+
matrix:
20+
include:
21+
- java-version: 21-ea
22+
toolchain: 21
23+
- java-version: 17
24+
toolchain: 17
25+
with:
26+
java-version: ${{ matrix.java-version }}
27+
test-args: --refresh-dependencies -PforceMavenRepositories=snapshot,https://oss.sonatype.org/content/repositories/snapshots -PisOverrideVersionCatalog -PtestToolchain=${{ matrix.toolchain }} -PspringFrameworkVersion=7.+ -PreactorVersion=2025.+ -PspringDataVersion=2025.+ --stacktrace
28+
secrets: inherit
29+
send-notification:
30+
name: Send Notification
31+
needs: [ snapshot-test ]
32+
if: ${{ !success() }}
33+
runs-on: ubuntu-latest
34+
steps:
35+
- name: Send Notification
36+
uses: spring-io/spring-security-release-tools/.github/actions/send-notification@v1
37+
with:
38+
webhook-url: ${{ secrets.SPRING_SECURITY_CI_GCHAT_WEBHOOK_URL }}

.github/workflows/continuous-integration-workflow.yml

Lines changed: 3 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -27,38 +27,24 @@ jobs:
2727
java-version: ${{ matrix.jdk }}
2828
distribution: temurin
2929
secrets: inherit
30-
test:
31-
name: Test Against Snapshots
32-
uses: spring-io/spring-security-release-tools/.github/workflows/test.yml@v1
33-
strategy:
34-
matrix:
35-
include:
36-
- java-version: 21-ea
37-
toolchain: 21
38-
- java-version: 17
39-
toolchain: 17
40-
with:
41-
java-version: ${{ matrix.java-version }}
42-
test-args: --refresh-dependencies -PforceMavenRepositories=snapshot,https://oss.sonatype.org/content/repositories/snapshots -PisOverrideVersionCatalog -PtestToolchain=${{ matrix.toolchain }} -PspringFrameworkVersion=7.+ -PreactorVersion=2025.+ -PspringDataVersion=2025.+ --stacktrace
43-
secrets: inherit
4430
deploy-artifacts:
4531
name: Deploy Artifacts
46-
needs: [ build, test]
32+
needs: [ build]
4733
uses: spring-io/spring-security-release-tools/.github/workflows/deploy-artifacts.yml@v1
4834
with:
4935
should-deploy-artifacts: ${{ needs.build.outputs.should-deploy-artifacts }}
5036
default-publish-milestones-central: true
5137
secrets: inherit
5238
deploy-docs:
5339
name: Deploy Docs
54-
needs: [ build, test ]
40+
needs: [ build ]
5541
uses: spring-io/spring-security-release-tools/.github/workflows/deploy-docs.yml@v1
5642
with:
5743
should-deploy-docs: ${{ needs.build.outputs.should-deploy-artifacts }}
5844
secrets: inherit
5945
deploy-schema:
6046
name: Deploy Schema
61-
needs: [ build, test ]
47+
needs: [ build ]
6248
uses: spring-io/spring-security-release-tools/.github/workflows/deploy-schema.yml@v1
6349
with:
6450
should-deploy-schema: ${{ needs.build.outputs.should-deploy-artifacts }}

0 commit comments

Comments
 (0)