Skip to content

Add JVM diagnostics to CI test workflow and surefire config - #111

Merged
bernardladenthin merged 1 commit into
mainfrom
claude/intelligent-cray-9tfnxv
Jun 17, 2026
Merged

Add JVM diagnostics to CI test workflow and surefire config#111
bernardladenthin merged 1 commit into
mainfrom
claude/intelligent-cray-9tfnxv

Conversation

@bernardladenthin

Copy link
Copy Markdown
Owner

Summary

  • Enhanced CI test workflow with memory monitoring before/after test execution and artifact collection for crash dumps and surefire reports on test failure
  • Added JVM diagnostic flags to maven-surefire-plugin configuration: 2 GB heap cap, heap dump on OutOfMemoryError, and per-PID error logs
  • Updated maven-surefire-plugin to use -e flag for enhanced error reporting
  • Added reference to CI test diagnostics policy in CLAUDE.md

Test plan

  • CI is green on this branch
  • Existing tests pass with new surefire JVM flags

Related issues / PRs

Checklist

  • I have read CONTRIBUTING.md and CODE_OF_CONDUCT.md
  • My commits follow Conventional Commits
  • No security-sensitive changes (if there are, I have notified the maintainer privately per SECURITY.md)

https://claude.ai/code/session_01JBzF5wtCjRu5t4FMzphryM

…ss-repo sync)

New surefire <argLine>@{argLine} -Xmx2g -XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=. -XX:ErrorFile=hs_err_pid%p.log (the @{argLine} prefix preserves the JaCoCo agent). CI test job: 'free -h' before/after, 'mvn -e', and on-failure crash+surefire dump upload. Implements workspace/policies/ci-test-diagnostics.md.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JBzF5wtCjRu5t4FMzphryM
@bernardladenthin
bernardladenthin merged commit a958e8e into main Jun 17, 2026
10 of 12 checks passed
@bernardladenthin
bernardladenthin deleted the claude/intelligent-cray-9tfnxv branch June 17, 2026 13:01
@sonarqubecloud

Copy link
Copy Markdown

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