Allow soft-rollout with crashtracking auto-config only if ddprof-java is enabled

Author: jbachorik

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

@@ -40,7 +40,6 @@
 import datadog.trace.api.profiling.ProfilingEnablement;
 import datadog.trace.api.scopemanager.ScopeListener;
 import datadog.trace.bootstrap.benchmark.StaticEventLogger;
-import datadog.trace.bootstrap.config.provider.ConfigProvider;
 import datadog.trace.bootstrap.config.provider.StableConfigSource;
 import datadog.trace.bootstrap.instrumentation.api.AgentTracer;
 import datadog.trace.bootstrap.instrumentation.api.AgentTracer.TracerAPI;
@@ -1052,10 +1051,22 @@ private static void initializeCrashTrackingDefault() {
   }
 
   private static boolean isCrashTrackingAutoconfigEnabled() {
-    return !ConfigProvider.getInstance()
-        .getBoolean(
-            CrashTrackingConfig.CRASH_TRACKING_DISABLE_AUTOCONFIG,
-            CrashTrackingConfig.CRASH_TRACKING_DISABLE_AUTOCONFIG_DEFAULT);
+    String enabledVal = ddGetProperty("dd." + CrashTrackingConfig.CRASH_TRACKING_ENABLE_AUTOCONFIG);
+    boolean enabled = CrashTrackingConfig.CRASH_TRACKING_ENABLE_AUTOCONFIG_DEFAULT;
+    if (enabledVal != null) {
+      enabled = Boolean.parseBoolean(enabledVal);
+    } else {
+      // If the property is not set, then we check if ddprof-java profiling is enabled,
+      // because crash tracking autoconfig can safely be enabled by default when
+      // ddprof-java profiling is enabled
+      String ddprofEnabledVal = ddGetProperty("dd.profiling.ddprof.enabled");
+      // the property default is 'true', so if it is not set then we assume it is enabled
+      if (ddprofEnabledVal == null || Boolean.parseBoolean(ddprofEnabledVal)) {
+        // ddprof library supported only on Linux
+        enabled = OperatingSystem.isLinux();
+      }
+    }
+    return enabled;
   }
 
   private static void initializeCrashTracking(boolean delayed, boolean checkNative) {

dd-trace-api/src/main/java/datadog/trace/api/config/CrashTrackingConfig.java

@@ -27,9 +27,9 @@ public final class CrashTrackingConfig {
   public static final String CRASH_TRACKING_START_EARLY = "crashtracking.debug.start-force-first";
   public static final boolean CRASH_TRACKING_START_EARLY_DEFAULT = false;
 
-  public static final String CRASH_TRACKING_DISABLE_AUTOCONFIG =
-      "crashtracking.debug.autoconfig.disable";
-  public static final boolean CRASH_TRACKING_DISABLE_AUTOCONFIG_DEFAULT = false;
+  public static final String CRASH_TRACKING_ENABLE_AUTOCONFIG =
+      "crashtracking.debug.autoconfig.enable";
+  public static final boolean CRASH_TRACKING_ENABLE_AUTOCONFIG_DEFAULT = false;
 
   private CrashTrackingConfig() {}
 }