fix(tracing): Check for otel before loading db module (#6461)

To make sure we don't override possible OpenTelemetry instrumentation,
check for the instrumenter to be configured as `otel` before we
dynamically load the database modules to instrument.

Author: AbhiPrasad

packages/tracing/src/integrations/node/apollo.ts

@@ -28,6 +28,11 @@ export class Apollo implements Integration {
    * @inheritDoc
    */
   public setupOnce(_: (callback: EventProcessor) => void, getCurrentHub: () => Hub): void {
+    if (shouldDisableAutoInstrumentation(getCurrentHub)) {
+      __DEBUG_BUILD__ && logger.log('Apollo Integration is skipped because of instrumenter configuration.');
+      return;
+    }
+
     const pkg = loadModule<{
       ApolloServerBase: {
         prototype: {
@@ -41,11 +46,6 @@ export class Apollo implements Integration {
       return;
     }
 
-    if (shouldDisableAutoInstrumentation(getCurrentHub)) {
-      __DEBUG_BUILD__ && logger.log('Apollo Integration is skipped because of instrumenter configuration.');
-      return;
-    }
-
     /**
      * Iterate over resolvers of the ApolloServer instance before schemas are constructed.
      */

packages/tracing/src/integrations/node/graphql.ts

@@ -20,6 +20,11 @@ export class GraphQL implements Integration {
    * @inheritDoc
    */
   public setupOnce(_: (callback: EventProcessor) => void, getCurrentHub: () => Hub): void {
+    if (shouldDisableAutoInstrumentation(getCurrentHub)) {
+      __DEBUG_BUILD__ && logger.log('GraphQL Integration is skipped because of instrumenter configuration.');
+      return;
+    }
+
     const pkg = loadModule<{
       [method: string]: (...args: unknown[]) => unknown;
     }>('graphql/execution/execute.js');
@@ -29,11 +34,6 @@ export class GraphQL implements Integration {
       return;
     }
 
-    if (shouldDisableAutoInstrumentation(getCurrentHub)) {
-      __DEBUG_BUILD__ && logger.log('GraphQL Integration is skipped because of instrumenter configuration.');
-      return;
-    }
-
     fill(pkg, 'execute', function (orig: () => void | Promise<unknown>) {
       return function (this: unknown, ...args: unknown[]) {
         const scope = getCurrentHub().getScope();

packages/tracing/src/integrations/node/mongo.ts

@@ -127,6 +127,11 @@ export class Mongo implements Integration {
    * @inheritDoc
    */
   public setupOnce(_: (callback: EventProcessor) => void, getCurrentHub: () => Hub): void {
+    if (shouldDisableAutoInstrumentation(getCurrentHub)) {
+      __DEBUG_BUILD__ && logger.log('Mongo Integration is skipped because of instrumenter configuration.');
+      return;
+    }
+
     const moduleName = this._useMongoose ? 'mongoose' : 'mongodb';
     const pkg = loadModule<{ Collection: MongoCollection }>(moduleName);
 
@@ -135,11 +140,6 @@ export class Mongo implements Integration {
       return;
     }
 
-    if (shouldDisableAutoInstrumentation(getCurrentHub)) {
-      __DEBUG_BUILD__ && logger.log('Mongo Integration is skipped because of instrumenter configuration.');
-      return;
-    }
-
     this._instrumentOperations(pkg.Collection, this._operations, getCurrentHub);
   }
 

packages/tracing/src/integrations/node/mysql.ts

@@ -24,18 +24,18 @@ export class Mysql implements Integration {
    * @inheritDoc
    */
   public setupOnce(_: (callback: EventProcessor) => void, getCurrentHub: () => Hub): void {
+    if (shouldDisableAutoInstrumentation(getCurrentHub)) {
+      __DEBUG_BUILD__ && logger.log('Mysql Integration is skipped because of instrumenter configuration.');
+      return;
+    }
+
     const pkg = loadModule<MysqlConnection>('mysql/lib/Connection.js');
 
     if (!pkg) {
       __DEBUG_BUILD__ && logger.error('Mysql Integration was unable to require `mysql` package.');
       return;
     }
 
-    if (shouldDisableAutoInstrumentation(getCurrentHub)) {
-      __DEBUG_BUILD__ && logger.log('Mysql Integration is skipped because of instrumenter configuration.');
-      return;
-    }
-
     // The original function will have one of these signatures:
     //    function (callback) => void
     //    function (options, callback) => void

packages/tracing/src/integrations/node/postgres.ts

@@ -36,18 +36,18 @@ export class Postgres implements Integration {
    * @inheritDoc
    */
   public setupOnce(_: (callback: EventProcessor) => void, getCurrentHub: () => Hub): void {
+    if (shouldDisableAutoInstrumentation(getCurrentHub)) {
+      __DEBUG_BUILD__ && logger.log('Postgres Integration is skipped because of instrumenter configuration.');
+      return;
+    }
+
     const pkg = loadModule<{ Client: PgClient; native: { Client: PgClient } }>('pg');
 
     if (!pkg) {
       __DEBUG_BUILD__ && logger.error('Postgres Integration was unable to require `pg` package.');
       return;
     }
 
-    if (shouldDisableAutoInstrumentation(getCurrentHub)) {
-      __DEBUG_BUILD__ && logger.log('Postgres Integration is skipped because of instrumenter configuration.');
-      return;
-    }
-
     if (this._usePgNative && !pkg.native?.Client) {
       __DEBUG_BUILD__ && logger.error("Postgres Integration was unable to access 'pg-native' bindings.");
       return;