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
add demo for otel-collector for metrics #2796
add demo for otel-collector for metrics #2796
Conversation
|
Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com>
561f58c
to
22c928f
Compare
) | ||
|
||
var ( | ||
meterProvider metric.MeterProvider |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have seen the same example in the collector repo. Should we avoid using "globals" and simply return the provider as the initProvider() return value, and use that instance to create a meter and stop?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok, I will fix like that.
Codecov Report
@@ Coverage Diff @@
## main #2796 +/- ##
=====================================
Coverage 75.7% 75.7%
=====================================
Files 179 179
Lines 11818 11818
=====================================
+ Hits 8951 8953 +2
+ Misses 2634 2632 -2
Partials 233 233
|
Signed-off-by: Ziqi Zhao <zhaoziqi9146@gmail.com>
Closing as this PR is outdated and the issue was solved by #5283 |
fix #2161
Signed-off-by: Ziqi Zhao zhaoziqi9146@gmail.com