JetStream consumer does not properly propagate tracing context

[dan-j]

Describe the bug

This bit of code attaches the trace to c.ctx and not ctx, then ctx is used in SendMessage()

eventing-natss/pkg/channel/jetstream/dispatcher/consumer.go

Lines 155 to 177 in ab53a99

	sc, ok := tracing.ParseSpanContext(event)
	var span *trace.Span
	if !ok {
	c.logger.Warn("Cannot parse the spancontext, creating a new span")
	c.ctx, span = trace.StartSpan(c.ctx, jsmChannel+"-"+string(c.sub.UID))
	} else {
	c.ctx, span = trace.StartSpanWithRemoteParent(c.ctx, jsmChannel+"-"+string(c.sub.UID), sc)
	}
	defer span.End()
	te := TypeExtractorTransformer("")
	dispatchExecutionInfo, err := c.dispatcher.SendMessage(
	ctx,
	message,
	c.sub.Subscriber,
	kncloudevents.WithReply(c.sub.Reply),
	kncloudevents.WithDeadLetterSink(c.sub.DeadLetter),
	kncloudevents.WithRetryConfig(c.sub.RetryConfig),
	kncloudevents.WithTransformers(&te),
	kncloudevents.WithHeader(additionalHeaders),
	)

[github-actions]

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

[astelmashenko]

/remove-lifecycle stale

[github-actions]

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.