fix(otel): fix async attributes access warning

Author: kschelonka

.env.local.example

@@ -69,5 +69,5 @@ PUBSUB_EMULATOR_HOST=localhost:8085
 OTEL_EXPORTER_OTLP_ENDPOINT=http://localhost:4318
 OTEL_EXPORTER_OTLP_PROTOCOL=http/protobuf
 OTEL_SERVICE_NAME=monitor
-OTEL_NODE_ENABLED_INSTRUMENTATIONS="http,knex,fs,grpc,aws-sdk,http,ioredis,knex,winston,undici,runtime-node,redis"
+OTEL_NODE_ENABLED_INSTRUMENTATIONS="http,knex,fs,grpc,aws-sdk,ioredis,winston,undici,runtime-node,redis"
 OTEL_RESOURCE_ATTRIBUTES="service.namespace=monitor-local,service.version=0.0.1"

src/instrumentation.node.ts

@@ -10,7 +10,13 @@ import {
   ConsoleMetricExporter,
   PeriodicExportingMetricReader,
 } from "@opentelemetry/sdk-metrics";
-import { resourceFromAttributes } from "@opentelemetry/resources";
+import {
+  envDetector,
+  processDetector,
+  resourceFromAttributes,
+  hostDetector,
+  ResourceDetectionConfig,
+} from "@opentelemetry/resources";
 import { NodeSDK } from "@opentelemetry/sdk-node";
 import {
   AlwaysOnSampler,
@@ -62,6 +68,15 @@ export async function nodeSDKBuilder() {
 
   const sentryClient = Sentry.getClient();
 
+  // All required resources are injected through env vars
+  // in cloud deployment, but locally we can use detectors
+  // The host detector has a known issue with async attributes
+  // and isn't super important
+  // https://github.com/open-telemetry/opentelemetry-js/issues/4638
+  const resourceDetectors = isLocalCollector
+    ? [processDetector, envDetector]
+    : undefined;
+
   if (sentryClient) {
     // https://github.com/getsentry/sentry-javascript/tree/develop/packages/opentelemetry
     setupEventContextTrace(sentryClient);
@@ -118,6 +133,10 @@ export async function nodeSDKBuilder() {
     // Ensure the correct subset of traces is sent to Sentry
     // This also ensures trace propagation works as expected
     sampler,
+    // Kuberenetes infrastructure automatically injects
+    // required resources through environment variables
+    autoDetectResources: isLocalCollector ? true : false,
+    resourceDetectors,
     instrumentations: [
       getNodeAutoInstrumentations({
         "@opentelemetry/instrumentation-fs": {