Skip to content

Commit 4359bd1

Browse files
astubbsclaude
andcommitted
ci: fix PIT minion crash — use jvmArgs not argLine for heap
Minion crashed with UNKNOWN_ERROR because -DargLine="-Xmx1g" doesn't pass JVM flags to PIT minions (argLine is a Surefire property that PIT reads for classpath/agent args, not JVM flags). Split into: -DjvmArgs=-Xmx1g (JVM heap for minions) -DargLine= (strips Jacoco agent) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 9c6155a commit 4359bd1

1 file changed

Lines changed: 6 additions & 4 deletions

File tree

.github/workflows/maven.yml

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -224,9 +224,10 @@ jobs:
224224
restore-keys: |
225225
pit-history-${{ github.base_ref }}-
226226
pit-history-
227-
# -DargLine="-Xmx1g" gives minion JVMs 1GB heap (default is too small,
228-
# causes MEMORY_ERROR) and strips Jacoco (PIT has
229-
# its own coverage).
227+
# -DjvmArgs=-Xmx1g gives minion JVMs 1GB heap (default too small,
228+
# causes MEMORY_ERROR / UNKNOWN_ERROR crashes).
229+
# -DargLine= strips Jacoco's javaagent from minions (PIT has
230+
# its own coverage). Separate from jvmArgs.
230231
# -DtimeoutConstant/Factor generous per-mutation timeout for CI hardware
231232
# -Dthreads=1 I/O-bound tests; parallelism adds contention not speed
232233
# -DwithHistory incremental: skips re-mutating unchanged code.
@@ -240,7 +241,8 @@ jobs:
240241
-Dlicense.skip \
241242
-DtargetClasses="io.confluent.parallelconsumer.internal.*" \
242243
-DtargetTests="io.confluent.parallelconsumer.*" \
243-
-DargLine="-Xmx1g" \
244+
-DjvmArgs=-Xmx1g \
245+
-DargLine= \
244246
-DtimeoutConstant=30000 -DtimeoutFactor=3.0 \
245247
-Dthreads=1 \
246248
-DwithHistory \

0 commit comments

Comments
 (0)