From b08edfe3e23ae85f44af3b1adcb9b3b5acfeff19 Mon Sep 17 00:00:00 2001
From: Olivier Albertini <olivier.albertini@montreal.ca>
Date: Wed, 11 Dec 2019 11:37:27 -0500
Subject: [PATCH] fix(plugin-dns): remove from default plugin list

closes #612

Signed-off-by: Olivier Albertini <olivier.albertini@montreal.ca>
---
 packages/opentelemetry-node/src/config.ts   |  1 -
 packages/opentelemetry-plugin-dns/README.md | 17 +++++++++++++++++
 2 files changed, 17 insertions(+), 1 deletion(-)

diff --git a/packages/opentelemetry-node/src/config.ts b/packages/opentelemetry-node/src/config.ts
index 23fe0ff63e..7fb1b7dd5b 100644
--- a/packages/opentelemetry-node/src/config.ts
+++ b/packages/opentelemetry-node/src/config.ts
@@ -28,7 +28,6 @@ export interface NodeTracerConfig extends BasicTracerConfig {
 /** List of all default supported plugins */
 export const DEFAULT_INSTRUMENTATION_PLUGINS: Plugins = {
   'mongodb-core': { enabled: true, path: '@opentelemetry/plugin-mongodb-core' },
-  dns: { enabled: true, path: '@opentelemetry/plugin-dns' },
   grpc: { enabled: true, path: '@opentelemetry/plugin-grpc' },
   http: { enabled: true, path: '@opentelemetry/plugin-http' },
   https: { enabled: true, path: '@opentelemetry/plugin-https' },
diff --git a/packages/opentelemetry-plugin-dns/README.md b/packages/opentelemetry-plugin-dns/README.md
index 6f0c28432c..50bf77fd7c 100644
--- a/packages/opentelemetry-plugin-dns/README.md
+++ b/packages/opentelemetry-plugin-dns/README.md
@@ -32,6 +32,23 @@ const tracer = new NodeTracer({
 });
 ```
 
+### Zipkin
+
+If you use Zipkin, you must use `ignoreHostnames` in order to not trace those calls. If the server is local. You can set :
+
+```
+const tracer = new NodeTracer({
+  plugins: {
+    dns: {
+      enabled: true,
+      // You may use a package name or absolute path to the file.
+      path: '@opentelemetry/plugin-dns',
+      ignoreHostnames: ['localhost']
+    }
+  }
+});
+```
+
 ### Dns Plugin Options
 
 Dns plugin has currently one option. You can set the following: