Revert "feat(resource): create sync resource with some attributes tha…

…t resolve asynchronously (open-telemetry#3460)"

This reverts commit 47444f2.

README.md

@@ -84,7 +84,9 @@ const sdk = new opentelemetry.NodeSDK({
 
 // initialize the SDK and register with the OpenTelemetry API
 // this enables the API to record telemetry
-sdk.start();
+sdk.start()
+  .then(() => console.log('Tracing initialized'))
+  .catch((error) => console.log('Error initializing tracing', error));
 
 // gracefully shut down the SDK on process exit
 process.on('SIGTERM', () => {
@@ -275,11 +277,6 @@ These instrumentations are hosted at <https://github.com/open-telemetry/opentele
 
 ## Upgrade guidelines
 
-### 0.35.x to 0.36.0
-
-- `@opentelemetry/sdk-node` changed `await start()` to now be synchronous
-- `@opentelemetry/sdk-node` changed `await detectResources()` to now be synchronous
-
 ### 0.28.x to 0.29.x
 
 - `@opentelemetry/exporter-trace-otlp-http` is now exporting `scopeSpans` instead of `instrumentationLibrarySpans`

experimental/packages/opentelemetry-browser-detector/src/BrowserDetector.ts

@@ -17,7 +17,6 @@
 import { diag } from '@opentelemetry/api';
 import {
   Detector,
-  IResource,
   Resource,
   ResourceDetectionConfig,
 } from '@opentelemetry/resources';
@@ -28,7 +27,7 @@ import { BROWSER_ATTRIBUTES, UserAgentData } from './types';
  * BrowserDetector will be used to detect the resources related to browser.
  */
 class BrowserDetector implements Detector {
-  async detect(config?: ResourceDetectionConfig): Promise<IResource> {
+  async detect(config?: ResourceDetectionConfig): Promise<Resource> {
     const isBrowser = typeof navigator !== 'undefined';
     if (!isBrowser) {
       return Resource.empty();

experimental/packages/opentelemetry-browser-detector/test/BrowserDetector.test.ts

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 import * as sinon from 'sinon';
-import { IResource } from '@opentelemetry/resources';
+import { Resource } from '@opentelemetry/resources';
 import { browserDetector } from '../src/BrowserDetector';
 import { describeBrowser, assertResource, assertEmptyResource } from './util';
 
@@ -47,7 +47,7 @@ describeBrowser('browserDetector()', () => {
       },
     });
 
-    const resource: IResource = await browserDetector.detect();
+    const resource: Resource = await browserDetector.detect();
     assertResource(resource, {
       platform: 'platform',
       brands: ['Chromium 106', 'Google Chrome 106', 'Not;A=Brand 99'],
@@ -63,7 +63,7 @@ describeBrowser('browserDetector()', () => {
       userAgentData: undefined,
     });
 
-    const resource: IResource = await browserDetector.detect();
+    const resource: Resource = await browserDetector.detect();
     assertResource(resource, {
       language: 'en-US',
       user_agent: 'dddd',
@@ -74,7 +74,7 @@ describeBrowser('browserDetector()', () => {
     sinon.stub(globalThis, 'navigator').value({
       userAgent: '',
     });
-    const resource: IResource = await browserDetector.detect();
+    const resource: Resource = await browserDetector.detect();
     assertEmptyResource(resource);
   });
 });

experimental/packages/opentelemetry-browser-detector/test/util.ts

@@ -16,7 +16,7 @@
 import { Suite } from 'mocha';
 import * as assert from 'assert';
 import { BROWSER_ATTRIBUTES } from '../src/types';
-import { IResource } from '@opentelemetry/resources';
+import { Resource } from '@opentelemetry/resources';
 
 export function describeBrowser(title: string, fn: (this: Suite) => void) {
   title = `Browser: ${title}`;
@@ -27,7 +27,7 @@ export function describeBrowser(title: string, fn: (this: Suite) => void) {
 }
 
 export const assertResource = (
-  resource: IResource,
+  resource: Resource,
   validations: {
     platform?: string;
     brands?: string[];
@@ -74,6 +74,6 @@ export const assertResource = (
  *
  * @param resource the Resource to validate
  */
-export const assertEmptyResource = (resource: IResource) => {
+export const assertEmptyResource = (resource: Resource) => {
   assert.strictEqual(Object.keys(resource.attributes).length, 0);
 };

experimental/packages/opentelemetry-exporter-prometheus/src/PrometheusSerializer.ts

@@ -29,7 +29,7 @@ import {
   Histogram,
 } from '@opentelemetry/sdk-metrics';
 import { hrTimeToMilliseconds } from '@opentelemetry/core';
-import { IResource } from '@opentelemetry/resources';
+import { Resource } from '@opentelemetry/resources';
 
 type PrometheusDataTypeLiteral =
   | 'counter'
@@ -340,7 +340,7 @@ export class PrometheusSerializer {
     return results;
   }
 
-  protected _serializeResource(resource: IResource): string {
+  protected _serializeResource(resource: Resource): string {
     const name = 'target_info';
     const help = `# HELP ${name} Target metadata`;
     const type = `# TYPE ${name} gauge`;

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

@@ -27,10 +27,8 @@ import {
 } from '@opentelemetry/instrumentation';
 import {
   Detector,
-  DetectorSync,
-  detectResourcesSync,
+  detectResources,
   envDetector,
-  IResource,
   processDetector,
   Resource,
   ResourceDetectionConfig,
@@ -73,8 +71,8 @@ export class NodeSDK {
   private _meterProviderConfig?: MeterProviderConfig;
   private _instrumentations: InstrumentationOption[];
 
-  private _resource: IResource;
-  private _resourceDetectors: Array<Detector | DetectorSync>;
+  private _resource: Resource;
+  private _resourceDetectors: Detector[];
 
   private _autoDetectResources: boolean;
 
@@ -202,7 +200,7 @@ export class NodeSDK {
   }
 
   /** Detect resource attributes */
-  public detectResources(): void {
+  public async detectResources(): Promise<void> {
     if (this._disabled) {
       return;
     }
@@ -211,18 +209,18 @@ export class NodeSDK {
       detectors: this._resourceDetectors,
     };
 
-    this.addResource(detectResourcesSync(internalConfig));
+    this.addResource(await detectResources(internalConfig));
   }
 
   /** Manually add a resource */
-  public addResource(resource: IResource): void {
+  public addResource(resource: Resource): void {
     this._resource = this._resource.merge(resource);
   }
 
   /**
    * Once the SDK has been configured, call this method to construct SDK components and register them with the OpenTelemetry API.
    */
-  public start(): void {
+  public async start(): Promise<void> {
     if (this._disabled) {
       return;
     }
@@ -232,7 +230,7 @@ export class NodeSDK {
     });
 
     if (this._autoDetectResources) {
-      this.detectResources();
+      await this.detectResources();
     }
 
     this._resource =

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

@@ -17,7 +17,7 @@
 import type { ContextManager, SpanAttributes } from '@opentelemetry/api';
 import { TextMapPropagator } from '@opentelemetry/api';
 import { InstrumentationOption } from '@opentelemetry/instrumentation';
-import { Detector, DetectorSync, Resource } from '@opentelemetry/resources';
+import { Detector, Resource } from '@opentelemetry/resources';
 import { MetricReader, View } from '@opentelemetry/sdk-metrics';
 import {
   Sampler,
@@ -36,7 +36,7 @@ export interface NodeSDKConfiguration {
   views: View[];
   instrumentations: InstrumentationOption[];
   resource: Resource;
-  resourceDetectors: Array<Detector | DetectorSync>;
+  resourceDetectors: Detector[];
   sampler: Sampler;
   serviceName?: string;
   spanProcessor: SpanProcessor;