Excluding J9 as well

dougqh · dougqh · commit b9c5af59f433 · 2024-11-13T17:34:46.000-05:00
diff --git a/dd-smoke-tests/src/main/groovy/datadog/smoketest/AbstractSmokeTest.groovy b/dd-smoke-tests/src/main/groovy/datadog/smoketest/AbstractSmokeTest.groovy
@@ -154,11 +154,11 @@ abstract class AbstractSmokeTest extends ProcessManager {
     ]
 
 
-    // DQH - 13 Nov 2024 - Crashtracking bash script oesn't work on OS X,
+    // DQH - 13 Nov 2024 - Crashtracking bash script doesn't work on OS X,
     // so skipping crash tracking on OS X
-    if (!Platform.isMac()) {
+    if (!Platform.isMac() && !Platform.isJ9()) {
       def errorScript = tmpDir + "/dd_crash_uploader." + getScriptExtension()
-      def onErrorValue = '"' + errorScript + ' %p"'
+      def onErrorValue = errorScript + ' %p'
 
       ret += "-XX:OnError=" + onErrorValue
     }

Original file line number	Diff line number	Diff line change
`@@ -154,11 +154,11 @@ abstract class AbstractSmokeTest extends ProcessManager {`
`154`	`154`	`]`
`155`	`155`
`156`	`156`
`157`		`- // DQH - 13 Nov 2024 - Crashtracking bash script oesn't work on OS X,`
	`157`	`+ // DQH - 13 Nov 2024 - Crashtracking bash script doesn't work on OS X,`
`158`	`158`	`// so skipping crash tracking on OS X`
`159`		`- if (!Platform.isMac()) {`
	`159`	`+ if (!Platform.isMac() && !Platform.isJ9()) {`
`160`	`160`	`def errorScript = tmpDir + "/dd_crash_uploader." + getScriptExtension()`
`161`		`- def onErrorValue = '"' + errorScript + ' %p"'`
	`161`	`+ def onErrorValue = errorScript + ' %p'`
`162`	`162`
`163`	`163`	`ret += "-XX:OnError=" + onErrorValue`
`164`	`164`	`}`