feat(auto-instrumentation-node): add azure detector

The Azure detectors were not being added to the list of possible detectors. All other cloud providers
were possible values, so this commit adds the missing detectors for Azure.

Signed-off-by: maryliag <marylia.gutierrez@grafana.com>

metapackages/auto-instrumentations-node/README.md

@@ -65,6 +65,7 @@ By default, all SDK resource detectors are used, but you can use the environment
 - `container`
 - `alibaba`
 - `aws`
+- `azure`
 - `gcp`
 - `all` - enable all resource detectors
 - `none` - disable resource detection

metapackages/auto-instrumentations-node/package.json

@@ -89,6 +89,7 @@
     "@opentelemetry/instrumentation-winston": "^0.37.0",
     "@opentelemetry/resource-detector-alibaba-cloud": "^0.28.9",
     "@opentelemetry/resource-detector-aws": "^1.4.2",
+    "@opentelemetry/resource-detector-azure": "^0.2.6",
     "@opentelemetry/resource-detector-container": "^0.3.9",
     "@opentelemetry/resource-detector-gcp": "^0.29.9",
     "@opentelemetry/resources": "^1.12.0",

metapackages/auto-instrumentations-node/src/utils.ts

@@ -73,6 +73,11 @@ import {
   osDetectorSync,
   processDetectorSync,
 } from '@opentelemetry/resources';
+import {
+  azureAppServiceDetector,
+  azureFunctionsDetector,
+  azureVmDetector,
+} from '@opentelemetry/resource-detector-azure';
 
 const RESOURCE_DETECTOR_CONTAINER = 'container';
 const RESOURCE_DETECTOR_ENVIRONMENT = 'env';
@@ -81,6 +86,7 @@ const RESOURCE_DETECTOR_OS = 'os';
 const RESOURCE_DETECTOR_PROCESS = 'process';
 const RESOURCE_DETECTOR_ALIBABA = 'alibaba';
 const RESOURCE_DETECTOR_AWS = 'aws';
+const RESOURCE_DETECTOR_AZURE = 'azure';
 const RESOURCE_DETECTOR_GCP = 'gcp';
 
 const InstrumentationMap = {
@@ -196,7 +202,7 @@ function getEnabledInstrumentationsFromEnv() {
 export function getResourceDetectorsFromEnv(): Array<Detector | DetectorSync> {
   const resourceDetectors = new Map<
     string,
-    Detector | DetectorSync | Detector[]
+    Detector | DetectorSync | Detector[] | DetectorSync[]
   >([
     [RESOURCE_DETECTOR_CONTAINER, containerDetector],
     [RESOURCE_DETECTOR_ENVIRONMENT, envDetectorSync],
@@ -215,6 +221,10 @@ export function getResourceDetectorsFromEnv(): Array<Detector | DetectorSync> {
         awsLambdaDetector,
       ],
     ],
+    [
+      RESOURCE_DETECTOR_AZURE,
+      [azureAppServiceDetector, azureFunctionsDetector, azureVmDetector],
+    ],
   ]);
 
   const resourceDetectorsFromEnv =

metapackages/auto-instrumentations-node/test/utils.test.ts

@@ -114,13 +114,13 @@ describe('utils', () => {
 
   describe('getResourceDetectorsFromEnv', () => {
     it('should return all resource detectors by default', () => {
-      assert.equal(getResourceDetectorsFromEnv().length, 12);
+      assert.equal(getResourceDetectorsFromEnv().length, 15);
     });
 
     it('should return all resource detectors when OTEL_NODE_RESOURCE_DETECTORS contains "all"', () => {
       process.env.OTEL_NODE_RESOURCE_DETECTORS = 'all';
 
-      assert.equal(getResourceDetectorsFromEnv().length, 12);
+      assert.equal(getResourceDetectorsFromEnv().length, 15);
 
       delete process.env.OTEL_NODE_RESOURCE_DETECTORS;
     });