Skip to content

Commit 3be7e73

Browse files
committed
fix(nx-dev): remove OTel-based tracing for now to remove local dev erros (#30096)
Locally, when running `nx serve-docs nx-dev`, you'll see errors coming from `/blog` URLs due to instrumentation problems. This is caused by the `@grafana/faro-web-tracing` packages as soon as it is imported in an app router environment. We don't need it for now, so we can just remove it. It means that we will be missing tracing on HTTP requests like XHRs, but we still get performance metrics and error reporting. ## Current Behavior See errors in logs during development ## Expected Behavior No errors in logs ## Related Issue(s) <!-- Please link the issue being fixed so it gets closed when this is merged. --> Fixes #
1 parent ea5d1ed commit 3be7e73

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

nx-dev/nx-dev/lib/components/frontend-observability.tsx

+1-5
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
'use client';
22
import { useEffect, useRef } from 'react';
33
import { getWebInstrumentations, initializeFaro } from '@grafana/faro-web-sdk';
4-
import { TracingInstrumentation } from '@grafana/faro-web-tracing';
54

65
export function FrontendObservability() {
76
const initialized = useRef(false);
@@ -25,10 +24,7 @@ export function FrontendObservability() {
2524
version,
2625
environment,
2726
},
28-
instrumentations: [
29-
...getWebInstrumentations(),
30-
new TracingInstrumentation(),
31-
],
27+
instrumentations: [...getWebInstrumentations()],
3228
});
3329
}, []);
3430
return null;

0 commit comments

Comments
 (0)