Spec update: Changes to disabled config must be eventually visible (#7706)

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: trask <[email protected]>

Author: trask

sdk/logs/src/main/java/io/opentelemetry/sdk/logs/SdkLogger.java

@@ -33,9 +33,7 @@ class SdkLogger implements Logger {
   private final LoggerSharedState loggerSharedState;
   private final InstrumentationScopeInfo instrumentationScopeInfo;
 
-  // deliberately not volatile because of performance concerns
-  // - which means its eventually consistent
-  protected boolean loggerEnabled;
+  protected volatile boolean loggerEnabled;
 
   SdkLogger(
       LoggerSharedState loggerSharedState,

sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/SdkMeter.java

@@ -89,7 +89,7 @@ final class SdkMeter implements Meter {
   private final InstrumentationScopeInfo instrumentationScopeInfo;
   private final Map<RegisteredReader, MetricStorageRegistry> readerStorageRegistries;
 
-  private boolean meterEnabled;
+  private volatile boolean meterEnabled;
 
   SdkMeter(
       MeterProviderSharedState meterProviderSharedState,

sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/internal/state/AsynchronousMetricStorage.java

@@ -85,7 +85,7 @@ public final class AsynchronousMetricStorage<T extends PointData> implements Met
   private long startEpochNanos;
   private long epochNanos;
 
-  private boolean enabled;
+  private volatile boolean enabled;
 
   private AsynchronousMetricStorage(
       RegisteredReader registeredReader,

sdk/metrics/src/main/java/io/opentelemetry/sdk/metrics/internal/state/DefaultSynchronousMetricStorage.java

@@ -73,7 +73,7 @@ public final class DefaultSynchronousMetricStorage<T extends PointData>
   private final ConcurrentLinkedQueue<AggregatorHandle<T>> aggregatorHandlePool =
       new ConcurrentLinkedQueue<>();
 
-  private boolean enabled;
+  private volatile boolean enabled;
 
   DefaultSynchronousMetricStorage(
       RegisteredReader registeredReader,

sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkTracer.java

@@ -30,9 +30,8 @@ class SdkTracer implements Tracer {
 
   private final TracerSharedState sharedState;
   private final InstrumentationScopeInfo instrumentationScopeInfo;
-  // deliberately not volatile because of performance concerns
-  // - which means its eventually consistent
-  protected boolean tracerEnabled;
+
+  protected volatile boolean tracerEnabled;
 
   SdkTracer(
       TracerSharedState sharedState,