Disable tiered JIT workaround by default (#3579)

* Disable tiered-jit workaround by default

* Enable the tiered-jit workaround in CI

* Update to link to PR

tracer/build/_build/docker/smoke.dockerfile

@@ -36,6 +36,9 @@ ENV DD_APPSEC_ENABLED=1
 ENV DD_TRACE_DEBUG=1
 ENV DD_PROFILING_LOG_DIR=/var/log/datadog/dotnet
 
+# see https://github.com/DataDog/dd-trace-dotnet/pull/3579
+ENV DD_INTERNAL_WORKAROUND_77973_ENABLED=1
+
 ENV ASPNETCORE_URLS=http://localhost:5000
 
 # Copy the app across

tracer/build/_build/docker/smoke.dotnet-tool.dockerfile

@@ -32,6 +32,9 @@ ENV DD_TRACE_DEBUG=1
 ENV ASPNETCORE_URLS=http://localhost:5000
 ENV DD_PROFILING_LOG_DIR=/var/log/datadog/dotnet
 
+# see https://github.com/DataDog/dd-trace-dotnet/pull/3579
+ENV DD_INTERNAL_WORKAROUND_77973_ENABLED=1
+
 # Copy the app across
 COPY --from=builder /src/publish /app/.
 

tracer/build/_build/docker/smoke.dotnet-tool.nuget.dockerfile

@@ -32,6 +32,9 @@ ENV DD_TRACE_DEBUG=1
 ENV ASPNETCORE_URLS=http://localhost:5000
 ENV DD_PROFILING_LOG_DIR=/var/log/datadog/dotnet
 
+# see https://github.com/DataDog/dd-trace-dotnet/pull/3579
+ENV DD_INTERNAL_WORKAROUND_77973_ENABLED=1
+
 # Copy the app across
 COPY --from=builder /src/publish /app/.
 

tracer/build/_build/docker/smoke.dotnet-tool.self-instrument.dockerfile

@@ -41,6 +41,9 @@ ENV DD_TRACE_DEBUG=1
 ENV ASPNETCORE_URLS=http://localhost:5000
 ENV DD_PROFILING_LOG_DIR=/var/log/datadog/dotnet
 
+# see https://github.com/DataDog/dd-trace-dotnet/pull/3579
+ENV DD_INTERNAL_WORKAROUND_77973_ENABLED=1
+
 # Copy the app across
 COPY --from=builder /src/publish /app/.
 

tracer/build/_build/docker/smoke.windows.dockerfile

@@ -43,6 +43,9 @@ ENV DD_PROFILING_ENABLED=1 \
     DD_PROFILING_LOG_DIR="C:\logs" \
     ASPNETCORE_URLS=http://localhost:5000
 
+# see https://github.com/DataDog/dd-trace-dotnet/pull/3579
+ENV DD_INTERNAL_WORKAROUND_77973_ENABLED=1
+
 # Copy the app across
 COPY --from=builder /src/publish /app/.
 

tracer/build/_build/docker/smoke.windows.dotnet-tool.dockerfile

@@ -43,6 +43,9 @@ ENV DD_PROFILING_ENABLED=1 \
     DD_PROFILING_LOG_DIR="C:\logs" \
     ASPNETCORE_URLS=http://localhost:5000
 
+# see https://github.com/DataDog/dd-trace-dotnet/pull/3579
+ENV DD_INTERNAL_WORKAROUND_77973_ENABLED=1
+
 # Copy the app across
 COPY --from=builder /src/publish /app/.
 

tracer/build/_build/docker/smoke.windows.nuget.dockerfile

@@ -46,6 +46,9 @@ ENV CORECLR_ENABLE_PROFILING=1 \
     DD_PROFILING_ENABLED=1 \
     ASPNETCORE_URLS=http://localhost:5000
 
+# see https://github.com/DataDog/dd-trace-dotnet/pull/3579
+ENV DD_INTERNAL_WORKAROUND_77973_ENABLED=1
+
 # Copy the app across
 COPY --from=builder /src/publish /app/.
 

tracer/build/_build/docker/smoke.windows.tracer-home.dockerfile

@@ -50,6 +50,9 @@ ENV DD_PROFILING_ENABLED=1 \
     DD_PROFILING_LOG_DIR="C:\logs" \
     ASPNETCORE_URLS=http://localhost:5000
 
+# see https://github.com/DataDog/dd-trace-dotnet/pull/3579
+ENV DD_INTERNAL_WORKAROUND_77973_ENABLED=1
+
 # Copy the app across
 COPY --from=builder /src/publish /app/.
 

tracer/src/Datadog.Tracer.Native/cor_profiler.cpp

@@ -219,8 +219,9 @@ HRESULT STDMETHODCALLTYPE CorProfiler::Initialize(IUnknown* cor_profiler_info_un
     Logger::Info("Runtime Information: ", runtimeType, " ", runtime_information_.major_version, ".", runtime_information_.minor_version, ".", runtime_information_.build_version);
 
     // Check if we have to disable tiered compilation (due to https://github.com/dotnet/runtime/issues/77973)
+    // see https://github.com/DataDog/dd-trace-dotnet/pull/3579 for more details
     bool disableTieredCompilation = false;
-    bool internal_workaround_77973_enabled = true;
+    bool internal_workaround_77973_enabled = false;
     shared::TryParseBooleanEnvironmentValue(shared::GetEnvironmentValue(environment::internal_workaround_77973_enabled), internal_workaround_77973_enabled);
     if (internal_workaround_77973_enabled)
     {

tracer/test/Datadog.Trace.TestHelpers/EnvironmentHelper.cs

@@ -190,6 +190,9 @@ public void SetEnvironmentVariables(
             string profilerEnabled = AutomaticInstrumentationEnabled ? "1" : "0";
             environmentVariables["DD_DOTNET_TRACER_HOME"] = MonitoringHome;
 
+            // see https://github.com/DataDog/dd-trace-dotnet/pull/3579
+            environmentVariables["DD_INTERNAL_WORKAROUND_77973_ENABLED"] = "1";
+
             // Everything should be using the native loader now
             var nativeLoaderPath = GetNativeLoaderPath();