quarkusio / quarkus

Quarkus: Supersonic Subatomic Java.
https://quarkus.io
Apache License 2.0
13.8k stars 2.68k forks source link

Context propagation for Vert.x `EventBus` #29955

Open knutwannheden opened 1 year ago

knutwannheden commented 1 year ago

Description

The Vert.x EventBus as described by the Quarkus guide looks like a powerful alternative to asynchronous CDI events and other alternatives. Unfortunately it doesn't support context propagation. Thus, for example, the OpenTelemetry tracing context is "lost" when sending an event over the EventBus.

If this works as designed and context propagation cannot be added, I think this should explicitly be stated in the Quarkus guide.

Implementation ideas

No response

quarkus-bot[bot] commented 1 year ago

/cc @FroMage(context-propagation), @manovotn(context-propagation)

gsmet commented 1 year ago

Also /cc @cescoffier and @mkouba

cescoffier commented 1 year ago

It's just because never integrated it. The event bus message header can be used to pass the tracing data.

Note that the event bus sender and consumer will use different duplicated context so, you can't just write the tracing data into the duplicated context.