Skip to content

Add CI profile for optimized continuous integration builds #2957

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 3 commits into from

Conversation

velo
Copy link
Member

@velo velo commented Jul 20, 2025

Summary

Add a new Maven profile specifically designed for CI environments to improve build stability and test output visibility.

Changes

  • New Maven profile ci: Configures surefire and failsafe plugins for CI-optimized execution
  • Single-threaded test execution: Uses forkCount=1 and reuseForks=false to prevent resource contention
  • Enhanced test output: Sets redirectTestOutputToFile=false for better log visibility in CI
  • GitHub Actions integration: Updated workflow to use -T1 (single-threaded builds) and -Pci profile

Test plan

  • Verify CI profile can be activated locally: mvn test -Pci
  • Confirm GitHub Actions workflow uses new configuration
  • Test output should be visible directly in CI logs
  • Single-threaded execution should reduce timing-related test failures

🤖 Generated with Claude Code

velo and others added 3 commits July 20, 2025 10:58
- Create new Maven profile 'ci' with single-threaded test execution
- Configure redirectTestOutputToFile=false for better CI log visibility
- Set forkCount=1 and reuseForks=false to avoid resource contention
- Update GitHub Actions to use -T1 (single-threaded) and -Pci profile

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Remove forkCount and reuseForks settings that were interfering with
the existing toolchain setup and causing Java version mismatches.
Keep only redirectTestOutputToFile=false for CI log visibility.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Add forkCount=1 to both surefire and failsafe plugins in the CI profile
to ensure tests run in single-threaded mode while maintaining toolchain
compatibility by not setting reuseForks=false.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Signed-off-by: Marvin Froeder <[email protected]>
@velo velo force-pushed the fix/vertx-http2-client-reconnect-test-timeout branch from f16c1ec to 602f7ab Compare July 20, 2025 14:24
@velo velo closed this Jul 21, 2025
@velo velo deleted the fix/vertx-http2-client-reconnect-test-timeout branch July 21, 2025 08:52
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.

1 participant