Thanks for the PR!

Can you say a little more about the use case for this change? It appears thatterraform itselfdoesn't yet process TRACEPARENT. Are there provisioners you use that do process it?

Can you say a little more about the use case for this change? It appears that terraform itselfhashicorp/terraform#35444. Are there provisioners you use that do process it?

Correct, terraform itself won't use it, but it will maintain it in the environment for the provider binaries it calls. We (Netflix) have a custom provider to implement our deployment strategies and would like the ability to continue traces that were started in Coder or further upstream. For example, our provider Create and Update methods will just call:

ctx := env.ContextWithRemoteSpanContext(context.Background(), os.Environ())

Then any new spans created will use the inherited span context.

The primary use case is to enhance our integration testing. In our tests, we start a trace and can see the spans between our test into Coder where it invokes terraform. Currently, in our terraform provider we have to start a new trace, where we collect spans through to workspace startup. With this patch, we will be able to connect our traces from the test entirely through to workspace startup, which should help with visibility/debugging.

That makes sense!

I have to say, from a supply-chain security standpoint, we are very reluctant to add a dependency ongithub.com/coryb/otelbundle if we can avoid it. If you're OK with it, we'd be willing to accept thepropagation/env implementation as a contribution directly to this repo.

The other thing I'd ask for is a unit test of the propagation.

I have to say, from a supply-chain security standpoint, we are very reluctant to add a dependency on github.com/coryb/otelbundle if we can avoid it. If you're OK with it, we'd be willing to accept the propagation/env implementation as a contribution directly to this repo.

Thanks, works for me, I will update. You should be able to swap out the implementation with an official OTEL support once they get that sorted.

@spikecurtis copied the required code to inject the trace env, and made all the functions unexported. Also added a test to ensure the TRACEPARENT env var is set/updated.

updated for lint rules...