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

Propagate Resource from Span to proto (Tonic) (#386) #390

Merged
merged 2 commits into from Dec 17, 2020

Conversation

markdingram
Copy link
Contributor

No description provided.

@markdingram markdingram requested a review from a team as a code owner December 17, 2020 14:33
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Dec 17, 2020

CLA Signed

The committers are authorized under a signed CLA.

@codecov
Copy link

codecov bot commented Dec 17, 2020

Codecov Report

Merging #390 (8ad8455) into master (aa14bf7) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #390   +/-   ##
=======================================
  Coverage   49.42%   49.42%           
=======================================
  Files          66       66           
  Lines        5356     5356           
=======================================
  Hits         2647     2647           
  Misses       2709     2709           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update aa14bf7...8ad8455. Read the comment docs.

Copy link
Member

@jtescher jtescher left a comment

Choose a reason for hiding this comment

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

thanks!

@jtescher jtescher merged commit 41fdac6 into open-telemetry:master Dec 17, 2020
jan-xyz added a commit to jan-xyz/opentelemetry-rust that referenced this pull request Oct 1, 2021
0.13.0

Bug fix: Avoid panics from Instant::elapsed (open-telemetry#406)

Improvement: Allow trailing comma on macros (open-telemetry#390)

Improvement: Add macros for custom registry (open-telemetry#396)

Improvement: Export thread count from process_collector (open-telemetry#401)

Improvement: Add convenience TextEncoder functions to encode directly to string (open-telemetry#402)

Internal change: Clean up the use of macro_use and extern crate (open-telemetry#398)

Internal change: Update dependencies
TommyCpp added a commit that referenced this pull request Oct 5, 2021
* bump prometheus dependency to 0.13

0.13.0

Bug fix: Avoid panics from Instant::elapsed (#406)

Improvement: Allow trailing comma on macros (#390)

Improvement: Add macros for custom registry (#396)

Improvement: Export thread count from process_collector (#401)

Improvement: Add convenience TextEncoder functions to encode directly to string (#402)

Internal change: Clean up the use of macro_use and extern crate (#398)

Internal change: Update dependencies

* also update in example

Co-authored-by: Zhongyang Wu <zhongyang.wu@outlook.com>
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

Successfully merging this pull request may close these issues.

None yet

2 participants