Skip to content

Comments

tracing: otel multitenant support#43554

Open
paul-r-gall wants to merge 10 commits intoenvoyproxy:mainfrom
paul-r-gall:OtelMultitenantTracer
Open

tracing: otel multitenant support#43554
paul-r-gall wants to merge 10 commits intoenvoyproxy:mainfrom
paul-r-gall:OtelMultitenantTracer

Conversation

@paul-r-gall
Copy link
Contributor

Commit Message: Add Multitenancy support for the OpenTelemetry Tracer.
Additional Description: Allow different OpenTelemetry resources to be associated with different Routes or Streams, addressing Multitenancy requirements.

  1. ResourceProvider and ResourceDetector have an additional method which can get/detect a resource based on StreamInfo.
  2. The Tracer now carries a shared_ptr to the ResourceProvider. This resource provider is invoked on span initialization in order to deduce the resource. There is no way (yet) to change the resource mid-span.
  3. Child spans always have the same Resource as parent spans.
  4. Tracer maintains a per_tenant_span_buffer_ to group spans by Resource ptr. We group by pointer in order to avoid copies and complicated hashing.

Risk Level: Low
Testing: Unit, Integration
Docs Changes: upcoming commit
Release Notes: upcoming commit

@repokitteh-read-only
Copy link

CC @envoyproxy/api-shepherds: Your approval is needed for changes made to (api/envoy/|docs/root/api-docs/).
envoyproxy/api-shepherds assignee is @markdroth
CC @envoyproxy/api-watchers: FYI only for changes made to (api/envoy/|docs/root/api-docs/).

🐱

Caused by: #43554 was opened by paul-r-gall.

see: more, trace.

@paul-r-gall paul-r-gall changed the title Otel multitenant tracer tracing: otel multitenant support Feb 19, 2026
@paul-r-gall
Copy link
Contributor Author

Relevant to #43184

@@ -1,5 +1,5 @@
{
"checksum": "b864c94e442ea41673dcae0f7039f7afb9ef5c4287962b4464b406f670a8e6d7",
"checksum": "72d5662849b82d7e405b70de2c4c9d4218abc468a128df52e05e1c9ebff3f2a9",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unnecessary change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants