opentelemetry-go-instrumentation icon indicating copy to clipboard operation
opentelemetry-go-instrumentation copied to clipboard

Context correlation documentation is outdated

Open pellared opened this issue 2 years ago • 3 comments

See https://cloud-native.slack.com/archives/C03S01YSAS0/p1695913090688189

Is https://github.com/keyval-dev/opentelemetry-go-instrumentation/blob/master/docs/design/context-propagation.md up-to-date? Do we still rely on goroutine ID for correlating incoming/outgoing requests?

I think this https://github.com/open-telemetry/opentelemetry-go-instrumentation/pull/118 changed this

CC @RonFed @edeNFed @dashpole

pellared avatar Sep 29 '23 09:09 pellared

I will take a look at this, hopefully this week.

edeNFed avatar Oct 02 '23 12:10 edeNFed

@edeNFed bump 😉

pellared avatar Jan 30 '24 17:01 pellared

We are still relying on Go routine IDs for anything new. Old versions of Go the routines were allocated to the stack.

MrAlias avatar Apr 16 '24 17:04 MrAlias