remove OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID

Signed-off-by: maryliag <[email protected]>

Author: maryliag

CHANGELOG.md

@@ -12,14 +12,11 @@ For experimental package changes, see the [experimental CHANGELOG](experimental/
 ### :rocket: (Enhancement)
 
 * feat(sdk-trace-base): log resource attributes in ConsoleSpanExporter [#4605](https://github.com/open-telemetry/opentelemetry-js/pull/4605) @pichlermarc
-* feat(resources): new experimental detector ServiceInstanceIdDetectorSync that sets the value for `service.instance.id` as random UUID.
-* feat(resources): add usage for the detector ServiceInstanceIDDetector.
-  * The resource detector can be added to default resource detector list by
-    * setting the environment variable `OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID` as `true`
-    * adding the value `serviceinstance` to the list of resource detectors on the environment variable `OTEL_NODE_RESOURCE_DETECTORS`, e.g `OTEL_NODE_RESOURCE_DETECTORS=env,host,os,serviceinstance`
+* feat(resources): new experimental detector ServiceInstanceIdDetectorSync that sets the value for `service.instance.id` as random UUID. [#4608](https://github.com/open-telemetry/opentelemetry-js/pull/4608) @maryliag
+* feat(resources): add usage for the detector ServiceInstanceIdDetectorSync. [#4626](https://github.com/open-telemetry/opentelemetry-js/pull/4626) @maryliag
+  * The resource detector can be added to default resource detector list by adding the value `serviceinstance` to the list of resource detectors on the environment variable `OTEL_NODE_RESOURCE_DETECTORS`, e.g `OTEL_NODE_RESOURCE_DETECTORS=env,host,os,serviceinstance`
   * The value can be overwritten by
     * merging a resource containing the `service.instance.id` attribute
-    * setting `service.instance.id` via the `OTEL_RESOURCE_ATTRIBUTES` environment variable when using `envDetector`
     * using another resource detector which writes `service.instance.id`
 
 ### :bug: (Bug Fix)

experimental/packages/opentelemetry-sdk-node/src/sdk.ts

@@ -36,7 +36,6 @@ import {
   processDetector,
   Resource,
   ResourceDetectionConfig,
-  serviceInstanceIDDetector,
 } from '@opentelemetry/resources';
 import { LogRecordProcessor, LoggerProvider } from '@opentelemetry/sdk-logs';
 import { MeterProvider, MetricReader, View } from '@opentelemetry/sdk-metrics';
@@ -130,9 +129,6 @@ export class NodeSDK {
       defaultDetectors = getResourceDetectorsFromEnv();
     } else {
       defaultDetectors = [envDetector, processDetector, hostDetector];
-      if (env.OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID) {
-        defaultDetectors.push(serviceInstanceIDDetector);
-      }
     }
 
     this._resourceDetectors =

experimental/packages/opentelemetry-sdk-node/src/utils.ts

@@ -20,13 +20,12 @@ import {
   InstrumentationOption,
 } from '@opentelemetry/instrumentation';
 import {
-  Detector,
   DetectorSync,
   envDetectorSync,
   hostDetectorSync,
   osDetectorSync,
   processDetectorSync,
-  serviceInstanceIDDetectorSync,
+  serviceInstanceIdDetectorSync,
 } from '@opentelemetry/resources';
 
 // TODO: This part of a workaround to fix https://github.com/open-telemetry/opentelemetry-js/issues/3609
@@ -58,15 +57,12 @@ const RESOURCE_DETECTOR_OS = 'os';
 const RESOURCE_DETECTOR_PROCESS = 'process';
 const RESOURCE_DETECTOR_SERVICE_INSTANCE_ID = 'serviceinstance';
 
-export function getResourceDetectorsFromEnv(): Array<Detector | DetectorSync> {
-  const resourceDetectors = new Map<
-    string,
-    Detector | DetectorSync | Detector[]
-  >([
+export function getResourceDetectorsFromEnv(): Array<DetectorSync> {
+  const resourceDetectors = new Map<string, DetectorSync>([
     [RESOURCE_DETECTOR_ENVIRONMENT, envDetectorSync],
     [RESOURCE_DETECTOR_HOST, hostDetectorSync],
     [RESOURCE_DETECTOR_OS, osDetectorSync],
-    [RESOURCE_DETECTOR_SERVICE_INSTANCE_ID, serviceInstanceIDDetectorSync],
+    [RESOURCE_DETECTOR_SERVICE_INSTANCE_ID, serviceInstanceIdDetectorSync],
     [RESOURCE_DETECTOR_PROCESS, processDetectorSync],
   ]);
 

experimental/packages/opentelemetry-sdk-node/test/sdk.test.ts

@@ -40,7 +40,7 @@ import {
 } from '@opentelemetry/sdk-metrics';
 import { NodeTracerProvider } from '@opentelemetry/sdk-trace-node';
 import {
-  assertServiceInstanceIDIsUUID,
+  assertServiceInstanceIdIsUUID,
   assertServiceResource,
 } from './util/resource-assertions';
 import {
@@ -707,7 +707,7 @@ describe('Node SDK', () => {
       const resource = sdk['_resource'];
       await resource.waitForAsyncAttributes?.();
 
-      assertServiceInstanceIDIsUUID(resource);
+      assertServiceInstanceIdIsUUID(resource);
       delete process.env.OTEL_NODE_RESOURCE_DETECTORS;
       await sdk.shutdown();
     });
@@ -727,56 +727,7 @@ describe('Node SDK', () => {
       const resource = sdk['_resource'];
       await resource.waitForAsyncAttributes?.();
 
-      assertServiceInstanceIDIsUUID(resource);
-      delete process.env.OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID;
-      await sdk.shutdown();
-    });
-
-    it('should configure service instance id with random UUID with OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID env var', async () => {
-      process.env.OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID = 'true';
-      const sdk = new NodeSDK();
-
-      sdk.start();
-      const resource = sdk['_resource'];
-      await resource.waitForAsyncAttributes?.();
-
-      assertServiceInstanceIDIsUUID(resource);
-      delete process.env.OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID;
-      await sdk.shutdown();
-    });
-
-    it('should configure service instance id via OTEL_RESOURCE_ATTRIBUTES env var even with OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID env var', async () => {
-      process.env.OTEL_RESOURCE_ATTRIBUTES =
-        'service.instance.id=627cc493,service.name=my-service';
-      process.env.OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID = 'true';
-      const sdk = new NodeSDK();
-
-      sdk.start();
-      const resource = sdk['_resource'];
-      await resource.waitForAsyncAttributes?.();
-
-      assertServiceResource(resource, {
-        name: 'my-service',
-        instanceId: '627cc493',
-      });
-      delete process.env.OTEL_RESOURCE_ATTRIBUTES;
-      delete process.env.OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID;
-      sdk.shutdown();
-    });
-
-    it('should not configure service instance id with no value for it on OTEL_RESOURCE_ATTRIBUTES env var and  OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID env var as false', async () => {
-      process.env.OTEL_RESOURCE_ATTRIBUTES = 'service.name=my-service';
-      process.env.OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID = 'false';
-      const sdk = new NodeSDK();
-
-      sdk.start();
-      const resource = sdk['_resource'];
-      await resource.waitForAsyncAttributes?.();
-
-      assertServiceResource(resource, {
-        name: 'my-service',
-      });
-      delete process.env.OTEL_RESOURCE_ATTRIBUTES;
+      assertServiceInstanceIdIsUUID(resource);
       await sdk.shutdown();
     });
   });

experimental/packages/opentelemetry-sdk-node/test/util/resource-assertions.ts

@@ -338,7 +338,7 @@ const assertHasOneLabel = (prefix: string, resource: Resource): void => {
   );
 };
 
-export const assertServiceInstanceIDIsUUID = (resource: Resource): void => {
+export const assertServiceInstanceIdIsUUID = (resource: Resource): void => {
   const UUID_REGEX =
     /^[0-9a-f]{8}-[0-9a-f]{4}-[1-5][0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/i;
   assert.equal(

packages/opentelemetry-core/src/utils/environment.ts

@@ -24,10 +24,7 @@ const DEFAULT_LIST_SEPARATOR = ',';
  * Environment interface to define all names
  */
 
-const ENVIRONMENT_BOOLEAN_KEYS = [
-  'OTEL_SDK_DISABLED',
-  'OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID',
-] as const;
+const ENVIRONMENT_BOOLEAN_KEYS = ['OTEL_SDK_DISABLED'] as const;
 
 type ENVIRONMENT_BOOLEANS = {
   [K in (typeof ENVIRONMENT_BOOLEAN_KEYS)[number]]?: boolean;
@@ -241,7 +238,6 @@ export const DEFAULT_ENVIRONMENT: Required<ENVIRONMENT> = {
   OTEL_EXPORTER_OTLP_METRICS_PROTOCOL: 'http/protobuf',
   OTEL_EXPORTER_OTLP_LOGS_PROTOCOL: 'http/protobuf',
   OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE: 'cumulative',
-  OTEL_NODE_EXPERIMENTAL_DEFAULT_SERVICE_INSTANCE_ID: false,
 };
 
 /**