Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Access headers in OTLP exporter #5264

Closed
tonychoe opened this issue Apr 26, 2022 · 2 comments
Closed

Access headers in OTLP exporter #5264

tonychoe opened this issue Apr 26, 2022 · 2 comments

Comments

@tonychoe
Copy link

Is your feature request related to a problem? Please describe.

The otlp exporter supports the headers option so we can specify the header value we want to send to the downstream.

  exporters:
    otlp/archlab:
      endpoint: distributor.tempo:4317
        headers:
          X-Scope-OrgID: archlab

However, in the multi-tenant scenario, passing different headers mean we need to create a pipeline of multiple exporters and each exporter has the hard-coded header.

Describe the solution you'd like
Enable accessing headers passed from the upstream. So, the exporter simply use the passed one.

  exporters:
    otlp/archlab:
      endpoint: distributor.tempo:4317
        headers:
          key: X-Scope-OrgID
          from_context: X-Scope-OrgID

Describe alternatives you've considered
Tried this pipeline

  • Have multiple exporters, each with the different header value hard-coded.
  • Have the routing processor that determines which exporter will be used by the header value passed from the upstream.
    Cons is the more tenant we have, the more exporter we must create.
@jpkrohling
Copy link
Member

I believe this is the same as #4814 . If you think this is not the same, feel free to reopen.

@tonychoe
Copy link
Author

Thanks @jpkrohling This is the same thing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants