fix(otel): Remove sampling decision in propagator

Author: AbhiPrasad

packages/opentelemetry-node/src/propagator.ts

@@ -62,12 +62,12 @@ export class SentryPropagator implements TextMapPropagator {
       const traceparentData = extractTraceparentData(header);
       newContext = newContext.setValue(SENTRY_TRACE_PARENT_CONTEXT_KEY, traceparentData);
       if (traceparentData) {
-        const traceFlags = traceparentData.parentSampled ? TraceFlags.SAMPLED : TraceFlags.NONE;
         const spanContext = {
           traceId: traceparentData.traceId || '',
           spanId: traceparentData.parentSpanId || '',
           isRemote: true,
-          traceFlags,
+          // Always sample if traceparent exists, we use SentrySpanProcessor to make sampling decisions with `startTransaction`.
+          traceFlags: TraceFlags.SAMPLED,
         };
         newContext = trace.setSpanContext(newContext, spanContext);
       }

packages/opentelemetry-node/src/spanprocessor.ts

@@ -133,6 +133,7 @@ function getTraceData(otelSpan: OtelSpan, parentContext: Context): Partial<Trans
     spanId,
     traceId,
     parentSpanId,
+    ...traceparentData,
     metadata: {
       // only set dynamic sampling context if sentry-trace header was set
       dynamicSamplingContext: traceparentData && !dynamicSamplingContext ? {} : dynamicSamplingContext,