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
Propagate Resource from Span to proto (Tonic) #386
Comments
Ah yeah looks like that one got missed, PRs appreciated 👍 |
markdingram
added a commit
to markdingram/opentelemetry-rust
that referenced
this issue
Dec 17, 2020
jtescher
pushed a commit
that referenced
this issue
Dec 17, 2020
markdingram
added a commit
to markdingram/opentelemetry-rust
that referenced
this issue
Dec 17, 2020
markdingram
added a commit
to markdingram/opentelemetry-rust
that referenced
this issue
Dec 17, 2020
jtescher
pushed a commit
that referenced
this issue
Dec 18, 2020
this was merged in 4390213 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The excellent change to add Tonic in #352 didn't inherit the fix to populate Resource from the Span to proto from #366
Will provide a PR unless something is already in the works?
The text was updated successfully, but these errors were encountered: