Skip to content

Commit 8cd5fa8

Browse files
randomandersonsarahchen6
authored andcommitted
Various Gitlab tweaks and test fixes (#8847)
* skip should be for entire class * bump memory to combat oom issues * bump smoketests and forked tests memory and defaults
1 parent 3e38785 commit 8cd5fa8

File tree

5 files changed

+10
-5
lines changed

5 files changed

+10
-5
lines changed

dd-java-agent/instrumentation/exception-profiling/src/test/java/datadog/trace/bootstrap/instrumentation/jfr/exceptions/ExceptionHistogramTest.java

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
import jdk.jfr.FlightRecorder;
2222
import jdk.jfr.Recording;
2323
import org.junit.jupiter.api.AfterEach;
24+
import org.junit.jupiter.api.BeforeAll;
2425
import org.junit.jupiter.api.BeforeEach;
2526
import org.junit.jupiter.api.Test;
2627
import org.openjdk.jmc.common.item.Aggregators;
@@ -60,9 +61,13 @@ public boolean equals(final Object obj) {
6061
private Recording snapshot;
6162
private ExceptionHistogram instance;
6263

64+
@BeforeAll
65+
public static void precheck() {
66+
assumeFalse(Platform.isJ9());
67+
}
68+
6369
@BeforeEach
6470
public void setup() {
65-
assumeFalse(Platform.isJ9());
6671
recording = new Recording();
6772
recording.enable("datadog.ExceptionCount");
6873
recording.start();

dd-smoke-tests/debugger-integration-tests/src/test/java/datadog/smoketest/ProcessBuilderHelper.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ public static ProcessBuilder createProcessBuilder(
3232
Arrays.asList(
3333
javaPath(),
3434
// "-agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=localhost:5006",
35-
"-Xmx" + System.getProperty("datadog.forkedMaxHeapSize", "512M"),
35+
"-Xmx" + System.getProperty("datadog.forkedMaxHeapSize", "1024M"),
3636
"-Xms" + System.getProperty("datadog.forkedMinHeapSize", "64M"),
3737
"-javaagent:" + agentShadowJar(),
3838
"-XX:ErrorFile=/tmp/hs_err_pid%p.log",

dd-smoke-tests/profiling-integration-tests/src/test/java/datadog/smoketest/JFRBasedProfilingIntegrationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -748,7 +748,7 @@ private static ProcessBuilder createProcessBuilder(
748748
final List<String> command =
749749
Arrays.asList(
750750
javaPath(),
751-
"-Xmx" + System.getProperty("datadog.forkedMaxHeapSize", "512M"),
751+
"-Xmx" + System.getProperty("datadog.forkedMaxHeapSize", "1024M"),
752752
"-Xms" + System.getProperty("datadog.forkedMinHeapSize", "64M"),
753753
"-javaagent:" + agentShadowJar(),
754754
"-XX:ErrorFile=/tmp/hs_err_pid%p.log",

dd-smoke-tests/profiling-integration-tests/src/test/java/datadog/smoketest/SmokeTestUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ static ProcessBuilder createProcessBuilder(
2828
new ArrayList<>(
2929
Arrays.asList(
3030
javaPath(),
31-
"-Xmx" + System.getProperty("datadog.forkedMaxHeapSize", "512M"),
31+
"-Xmx" + System.getProperty("datadog.forkedMaxHeapSize", "1024M"),
3232
"-Xms" + System.getProperty("datadog.forkedMinHeapSize", "64M"),
3333
"-javaagent:" + agentShadowJar(),
3434
"-XX:ErrorFile=/tmp/hs_err_pid%p.log",

utils/test-utils/src/main/java/datadog/trace/test/util/ForkedTestUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
public class ForkedTestUtils {
44
public static String getMaxMemoryArgumentForFork() {
5-
return "-Xmx" + System.getProperty("datadog.forkedMaxHeapSize", "512M");
5+
return "-Xmx" + System.getProperty("datadog.forkedMaxHeapSize", "1024M");
66
}
77

88
public static String getMinMemoryArgumentForFork() {

0 commit comments

Comments
 (0)