Skip to content

Commit

Permalink
[ci] Remove Java Flight Recorder from CI and Maven Surefire target, a…
Browse files Browse the repository at this point in the history
  • Loading branch information
adamretter committed Oct 21, 2024
1 parent 0907fc8 commit fc12123
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 3 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/ci-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ on: [push, pull_request]
permissions:
contents: read
env:
MAVEN_OPTS: -XX:StartFlightRecording=maxsize=5g,disk=true,dumponexit=true,settings=default,filename=./ -DtrimStackTrace=false -D'maven.resolver.transport=wagon'
MAVEN_OPTS: -DtrimStackTrace=false -D'maven.resolver.transport=wagon'
DEV_JDK: '8'
jobs:
license:
Expand Down Expand Up @@ -81,6 +81,5 @@ jobs:
name: ${{ runner.os }}-build-logs
retention-days: 5
path: |
**/*.jfr
**/hs_err_pid*.log
**/target/surefire-reports/*
2 changes: 1 addition & 1 deletion exist-parent/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -778,7 +778,7 @@
However it can make it hard to diagnose problems if tests leak state; If you experience
such a problem you may want to set it to `false` whilst debugging -->
<reuseForks>true</reuseForks>
<argLine>@{jacocoArgLine} -XX:+IgnoreUnrecognizedVMOptions --illegal-access=debug --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.ref=ALL-UNNAMED --add-opens=java.base/java.lang.reflect=ALL-UNNAMED -XX:StartFlightRecording=maxsize=5g,disk=true,dumponexit=true,settings=default,filename=${project.build.directory}/ -Dfile.encoding=${project.build.sourceEncoding}</argLine>
<argLine>@{jacocoArgLine} -XX:+IgnoreUnrecognizedVMOptions --illegal-access=debug --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.ref=ALL-UNNAMED --add-opens=java.base/java.lang.reflect=ALL-UNNAMED -Dfile.encoding=${project.build.sourceEncoding}</argLine>
<systemPropertyVariables>
<user.country>UK</user.country>
<user.language>en</user.language>
Expand Down

0 comments on commit fc12123

Please sign in to comment.