Merge branch 'master' into jb/ddprof_1.23.0

Author: jbachorik

components/yaml/build.gradle.kts

@@ -0,0 +1,14 @@
+plugins {
+  id("me.champeau.jmh")
+}
+
+apply(from = "$rootDir/gradle/java.gradle")
+
+jmh {
+  version = "1.28"
+}
+
+// https://repo1.maven.org/maven2/org/yaml/snakeyaml/2.4/snakeyaml-2.4.pom
+dependencies {
+  implementation("org.yaml", "snakeyaml", "2.4")
+}

components/yaml/src/main/java/datadog/yaml/YamlParser.java

@@ -0,0 +1,19 @@
+package datadog.yaml;
+
+import java.io.FileInputStream;
+import java.io.IOException;
+import org.yaml.snakeyaml.Yaml;
+
+public class YamlParser {
+  // Supports clazz == null for default yaml parsing
+  public static <T> T parse(String filePath, Class<T> clazz) throws IOException {
+    Yaml yaml = new Yaml();
+    try (FileInputStream fis = new FileInputStream(filePath)) {
+      if (clazz == null) {
+        return yaml.load(fis);
+      } else {
+        return yaml.loadAs(fis, clazz);
+      }
+    }
+  }
+}

dd-java-agent/agent-bootstrap/src/main/java/datadog/trace/bootstrap/Constants.java

@@ -16,6 +16,7 @@ public final class Constants {
   public static final String[] BOOTSTRAP_PACKAGE_PREFIXES = {
     "datadog.slf4j",
     "datadog.json",
+    "datadog.yaml",
     "datadog.context",
     "datadog.cli",
     "datadog.appsec.api",

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

@@ -186,7 +186,7 @@ jdk.ThreadCPULoad#enabled=true
 jdk.ThreadCPULoad#period=10 s
 jdk.CPUTimeStampCounter#enabled=true
 jdk.CPUTimeStampCounter#period=beginChunk
-jdk.SystemProcess#enabled=true
+jdk.SystemProcess#enabled=false
 jdk.SystemProcess#period=endChunk
 jdk.NetworkUtilization#enabled=true
 jdk.NetworkUtilization#period=5 s

dd-java-agent/build.gradle

@@ -59,8 +59,9 @@ ext.generalShadowJarConfig = {
 
   final String projectName = "${project.name}"
 
-  // Prevents conflict with other OkHttp instances, but don't relocate instrumentation
+  // Prevents conflict with other instances, but doesn't relocate instrumentation
   if (!projectName.equals('instrumentation')) {
+    relocate 'org.yaml.snakeyaml', 'datadog.snakeyaml'
     relocate 'okhttp3', 'datadog.okhttp3'
     relocate 'okio',  'datadog.okio'
   }

dd-java-agent/instrumentation/build.gradle

@@ -59,6 +59,9 @@ subprojects { Project subProj ->
       def name = "java$version.majorVersion"
       jdkCompile = "main_${name}Implementation"
     }
+    configurations.muzzleBootstrap {
+      exclude group: 'org.yaml', module : 'snakeyaml' // we vendor this in the agent jar
+    }
     dependencies {
       // Apply common dependencies for instrumentation.
       implementation project(':dd-trace-api')

dd-java-agent/instrumentation/lettuce-5/build.gradle

@@ -11,6 +11,7 @@ muzzle {
 apply from: "$rootDir/gradle/java.gradle"
 
 addTestSuiteForDir('latestDepTest', 'test')
+addTestSuiteExtendingForDir('latestDepForkedTest', 'latestDepTest', 'test')
 
 dependencies {
   compileOnly group: 'io.lettuce', name: 'lettuce-core', version: '5.0.0.RELEASE'

dd-java-agent/instrumentation/lettuce-5/src/test/groovy/Lettuce5ClientTestBase.groovy

@@ -42,6 +42,13 @@ abstract class Lettuce5ClientTestBase extends VersionedNamingTestBase {
   RedisAsyncCommands<String, ?> asyncCommands
   RedisCommands<String, ?> syncCommands
 
+  @Override
+  boolean useStrictTraceWrites() {
+    // latest seems leaking continuations that terminates later hence the strict trace will discard our spans.
+    !isLatestDepTest
+  }
+
+
   def setup() {
     redisServer.start()
     println "Using redis: $redisServer.redisURI"

dd-java-agent/instrumentation/snakeyaml/build.gradle

@@ -18,7 +18,13 @@ addTestSuiteForDir('latestDepTest', 'test')
 
 dependencies {
   compileOnly group: 'org.yaml', name: 'snakeyaml', version: '1.33'
-  testImplementation group: 'org.yaml', name: 'snakeyaml', version: '1.33'
+
+  testImplementation('org.yaml:snakeyaml') {
+    version {
+      strictly "[1.4, 2.0)"
+      prefer '1.33'
+    }
+  }
 
   latestDepTestImplementation group: 'org.yaml', name: 'snakeyaml', version: '1.+'
 }

dd-java-agent/testing/build.gradle

@@ -38,6 +38,10 @@ excludedClassesCoverage += [
   'datadog.trace.agent.test.TestProfilingContextIntegration.TestQueueTiming'
 ]
 
+configurations.api {
+  exclude group: 'org.yaml', module: 'snakeyaml' // we vendor this in the agent jar
+}
+
 dependencies {
   api libs.bytebuddy
   api libs.bytebuddyagent