fix: disable the pino instrumentation

The pino instrumentation is causing us issues because we import our
logger before the instrumentation can happen. It's logging an obnoxious
warning. I think this is fine to disable because:

1. We use pino to log to stdout so we're not missing a lot of useful
   info about the way it runs

2. The instrumentation only adds traces, not metrics. We run pino async
   so it doesn't block further execution and shouldn't impact the
   timings of the routes etc that use it

3. By default, this instrumentation tries to use the OpenTelemetry logs
   SDK. I'd rather not load some stuff that we don't want to use.

Author: rowanmanning

packages/opentelemetry/lib/config/instrumentations.js

@@ -28,6 +28,9 @@ exports.createInstrumentationConfig = function createInstrumentationConfig() {
 			},
 			'@opentelemetry/instrumentation-fs': {
 				enabled: false
+			},
+			'@opentelemetry/instrumentation-pino': {
+				enabled: false
 			}
 		}),
 		new RuntimeNodeInstrumentation()

packages/opentelemetry/test/unit/lib/config/instrumentations.spec.js

@@ -41,6 +41,9 @@ describe('@dotcom-reliability-kit/opentelemetry/lib/config/instrumentation', ()
 				},
 				'@opentelemetry/instrumentation-fs': {
 					enabled: false
+				},
+				'@opentelemetry/instrumentation-pino': {
+					enabled: false
 				}
 			});
 		});