Skip to content

Add Baggage Span Processor #85

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.

Sign up for GitHub

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

Jump to bottom
Closed
gugahoa wants to merge 133 commits into from
Closed

Add Baggage Span Processor #85

gugahoa wants to merge 133 commits into from

Conversation

@gugahoa
Copy link

@gugahoa gugahoa commented May 14, 2022

The purpose of the Baggage Span Processor is to add baggage as attributes to the span

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 14, 2022 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@gugahoa gugahoa force-pushed the baggage-processor branch 2 times, most recently from e4497f7 to 057900f Compare May 14, 2022 16:46
@gugahoa gugahoa marked this pull request as ready for review May 14, 2022 16:49
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented May 14, 2022 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@gugahoa gugahoa force-pushed the baggage-processor branch from 45335a7 to 2a70b5c Compare May 14, 2022 22:37
tsloughter
tsloughter reviewed May 26, 2022
View reviewed changes
@gugahoa gugahoa force-pushed the baggage-processor branch from f3cff5a to b565274 Compare May 27, 2022 00:25
@gugahoa gugahoa requested a review from tsloughter May 27, 2022 15:03
@andrewhr andrewhr mentioned this pull request Jun 28, 2022
Merged
@andrewhr
Copy link
Contributor

andrewhr commented Aug 24, 2023

Hey @gugahoa, any chance to get back to this so we can merge and publish?

I'm happy to help whatever I can, even push to finish line if that's the case cc @tsloughter (not sure how CA affects that)

@gugahoa gugahoa force-pushed the baggage-processor branch from d703f31 to b981ae8 Compare June 15, 2024 01:30
@gugahoa
Copy link
Author

gugahoa commented Jun 15, 2024

Long time no see team! I made the requested changes. Though I don't have any formatter configured so I think one of the changes I made is not following the proper format of the project. I'll see to it.

@tsloughter could you approve the workflow run for CI?

@gugahoa gugahoa requested a review from tsloughter June 17, 2024 18:11
@gugahoa
Copy link
Author

gugahoa commented Aug 2, 2025

I think I messed something up during the rebase

@gugahoa
Copy link
Author

gugahoa commented Aug 2, 2025

Created a new one here: #534

@gugahoa gugahoa closed this Aug 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tsloughter tsloughter Awaiting requested review from tsloughter tsloughter is a code owner

@bryannaegele bryannaegele Awaiting requested review from bryannaegele bryannaegele is a code owner

@whatyouhide whatyouhide Awaiting requested review from whatyouhide whatyouhide is a code owner

Assignees

No one assigned

Labels

elixir erlang examples instrumentation opentelemetry_bandit opentelemetry_broadway opentelemetry_cowboy opentelemetry_dataloader opentelemetry_ecto opentelemetry_finch opentelemetry_nebulex opentelemetry_oban opentelemetry_phoenix opentelemetry_process_propagator opentelemetry_redix opentelemetry_req opentelemetry_telemetry opentelemetry_tesla opentelemetry_xandra otel_http propagators scope-ci utilities

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.