Merge pull request #6 from Dharin-shah/update-opentelemetry

Update opentelemetry

Author: Dharin-shah

packages/opentelemetry/lib/tracing/exporterFactory.js

@@ -3,8 +3,6 @@ import { ZipkinExporter } from '@opentelemetry/exporter-zipkin';
 import { PrometheusExporter } from '@opentelemetry/exporter-prometheus';
 import { ConsoleSpanExporter, SpanExporter } from '@opentelemetry/tracing'; // eslint-disable-line no-unused-vars
 
-const DEFAULT_SERVICE_NAME = 'appium';
-
 const AVAILABLE_EXPORTERS = {
   JAEGER: 'jaeger',
   ZIPKIN: 'zipkin',
@@ -22,9 +20,9 @@ const AVAILABLE_EXPORTERS = {
 function buildExporter (exporterType, config = null) {
   switch (exporterType) {
     case AVAILABLE_EXPORTERS.JAEGER:
-      return new JaegerExporter(config || { serviceName: DEFAULT_SERVICE_NAME });
+      return new JaegerExporter(config);
     case AVAILABLE_EXPORTERS.ZIPKIN:
-      return new ZipkinExporter(config || { serviceName: DEFAULT_SERVICE_NAME });
+      return new ZipkinExporter(config || undefined);
     case AVAILABLE_EXPORTERS.PROMETHEUS:
       return new PrometheusExporter(config || {});
     case AVAILABLE_EXPORTERS.CONSOLE:

packages/opentelemetry/lib/tracing/tracerProvider.js

@@ -8,11 +8,17 @@ import { SpanProcessor, SpanExporter } from '@opentelemetry/tracing'; // eslint-
 import { ExporterConfig } from '@opentelemetry/exporter-jaeger'; // eslint-disable-line no-unused-vars
 import _ from 'lodash';
 import { forEachPromise } from '../utils';
+import { Resource } from '@opentelemetry/resources';
+import { SemanticResourceAttributes } from '@opentelemetry/semantic-conventions';
+
+const DEFAULT_SERVICE_NAME = 'appium';
 
 //Delegate
 class TracerProvider {
-  constructor () {
-    this.provider = new NodeTracerProvider();
+  constructor (serviceName = DEFAULT_SERVICE_NAME) {
+    this.provider = new NodeTracerProvider({
+      resource: new Resource({ [SemanticResourceAttributes.SERVICE_NAME]: serviceName }),
+    });
     this.exporterInstances = {};
     this._currentConfig = {
       active: true,