You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Spring integration CI was still using Maven, diverging from the repository’s Bazel-based build/test path. This updates the spring GitHub Actions workflow to run Spring + Spring AOT integration builds/tests via Bazel targets.
Workflow execution model
Replaced Maven test invocation with Bazel build/test steps scoped to Spring integration targets.
Added Bazel dependency pinning step (@maven//:pin) to keep external resolution aligned with MODULE.bazel.
Runner/tooling setup
Added Bazelisk installation in the workflow job.
Set BAZELISK_SKIP_VERSION_CHECK=1 in job env for consistent CI execution.
Caching strategy
Replaced Maven local-repo cache with Bazel disk/repository cache paths.
Updated cache keys to track Bazel-relevant inputs (MODULE.bazel, .bazelversion, maven_install.json) and Java matrix version.
- name: Build Spring integrations with Bazelrun: >- bazel build --config=ci //spring:client-java-spring-integration //spring-aot:client-java-spring-aot-integration
- name: Test Spring integrations with Bazelrun: >- bazel test --config=ci //spring:tests
This pull-request has been approved by: Copilot Once this PR has been reviewed and has the lgtm label, please ask for approval from brendandburns. For more information see the Code Review Process.
The full list of commands accepted by this bot can be found here.
Details
Needs approval from an approver in each of these files:
@copilot modify the cache key so it is the same as other CI/CD so that it re-uses builds from the main build/test runs.
Updated in 6c67e4b: the Spring workflow cache key and restore-key prefix now match the main Bazel CI format (bazel-${{ runner.os }}-java${{ matrix.java }}-...) so it can reuse the same cache entries.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cncf-cla: yesIndicates the PR's author has signed the CNCF CLA.do-not-merge/work-in-progressIndicates that a PR should not merge because it is a work in progress.size/MDenotes a PR that changes 30-99 lines, ignoring generated files.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Spring integration CI was still using Maven, diverging from the repository’s Bazel-based build/test path. This updates the
springGitHub Actions workflow to run Spring + Spring AOT integration builds/tests via Bazel targets.Workflow execution model
@maven//:pin) to keep external resolution aligned withMODULE.bazel.Runner/tooling setup
BAZELISK_SKIP_VERSION_CHECK=1in job env for consistent CI execution.Caching strategy
MODULE.bazel,.bazelversion,maven_install.json) and Java matrix version.