Skip to content

Commit ba7123a

Browse files
authored
Remove process information from JFR recording (#8661)
* JFR default settings - remove process information * JFR default settings - Adjust smoke test Extend smoke test to check that process information is not collected
1 parent 6d284ce commit ba7123a

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

dd-java-agent/agent-profiling/profiling-controller-jfr/src/main/resources/jfr/dd.jfp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -186,7 +186,7 @@ jdk.ThreadCPULoad#enabled=true
186186
jdk.ThreadCPULoad#period=10 s
187187
jdk.CPUTimeStampCounter#enabled=true
188188
jdk.CPUTimeStampCounter#period=beginChunk
189-
jdk.SystemProcess#enabled=true
189+
jdk.SystemProcess#enabled=false
190190
jdk.SystemProcess#period=endChunk
191191
jdk.NetworkUtilization#enabled=true
192192
jdk.NetworkUtilization#period=5 s

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

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -565,6 +565,11 @@ private void assertRecordingEvents(
565565
final IItemCollection events,
566566
final boolean expectEndpointEvents,
567567
final boolean asyncProfilerEnabled) {
568+
// Process events should not be collected
569+
assertFalse(
570+
events.apply(ItemFilters.type("jdk.SystemProcess")).hasItems(),
571+
"jdk.SystemProcess events should not be collected");
572+
568573
assertTrue(
569574
events
570575
.apply(

0 commit comments

Comments
 (0)