Skip to content

build(ci): add bulkTests profile to grpc-gcp-java - #13905

Merged
lqiu96 merged 1 commit into
mainfrom
build/add-bulk-tests-to-split-modules
Jul 27, 2026
Merged

build(ci): add bulkTests profile to grpc-gcp-java#13905
lqiu96 merged 1 commit into
mainfrom
build/add-bulk-tests-to-split-modules

Conversation

@lqiu96

@lqiu96 lqiu96 commented Jul 27, 2026

Copy link
Copy Markdown
Member

grpc-gcp-java has its own split-units config. BulkTests filter should ignore those unit tests from running on the units CI

[INFO] --- surefire:3.5.2:test (default-test) @ grpc-gcp ---
[INFO] Using auto detected provider org.apache.maven.surefire.junit4.JUnit4Provider
[INFO] -------------------------------------------------------
[INFO]  T E S T S
[INFO] -------------------------------------------------------
[INFO] Running com.google.cloud.grpc.ChannelIdPropagationTest
[INFO] Running com.google.cloud.grpc.GcpManagedChannelOtelMetricsTest
[INFO] Running com.google.cloud.grpc.GcpManagedChannelTest

Logs of it running in units CI.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds a 'bulkTests' profile to skip tests across numerous POM files in various submodules. The reviewer recommends defining this profile in the parent POM instead of duplicating it in every submodule, which would leverage Maven inheritance to reduce boilerplate and simplify maintenance.

Comment thread google-auth-library-java/appengine/pom.xml Outdated
@lqiu96
lqiu96 force-pushed the build/add-bulk-tests-to-split-modules branch from befb374 to 518b9c0 Compare July 27, 2026 18:26
@lqiu96
lqiu96 force-pushed the build/add-bulk-tests-to-split-modules branch from 518b9c0 to e6bc5f6 Compare July 27, 2026 18:30
@lqiu96 lqiu96 changed the title build(ci): add bulkTests profile to submodules of handwritten libraries build(ci): add bulkTests profile to grpc-gcp-java Jul 27, 2026
@lqiu96
lqiu96 requested a review from blakeli0 July 27, 2026 19:04
@lqiu96
lqiu96 marked this pull request as ready for review July 27, 2026 19:04
@lqiu96
lqiu96 requested review from a team as code owners July 27, 2026 19:04
@lqiu96
lqiu96 merged commit 523cb78 into main Jul 27, 2026
218 checks passed
@lqiu96
lqiu96 deleted the build/add-bulk-tests-to-split-modules branch July 27, 2026 19:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants