feat(metrics): add opentelemetry support - #575
Conversation
a1f41a2 to
5470970
Compare
|
Hi @roblaszczak and @m110, Could you please help review this PR when you have a moment? Thanks! |
4182b09 to
c51f8a9
Compare
|
We had this issue with OpenTelemtry in many projects: open-telemetry/opentelemetry-go#4476 To avoid that it would be good to move OpenTelemetry implementation to a separate package, with a separate go.mod. So the main go.mod won't be affected. I would moving those changes to a separate Do you feel you'll have time for that @ahkui? Let us know! |
@roblaszczak yes, i can handle that the new pkg i will move to |
Hey, I discussed it with @m110 and since it looks orthogonal to the So I would suggest |
|
Hey @ahkui and @roblaszczak, I would love to have OTEL metrics in Watermill, do you need any help to finalize this Pull Request? |
ab49b0b to
a998544
Compare
Signed-off-by: ahkui <ahkui@outlook.com>
a998544 to
b75dea7
Compare
|
Hi @roblaszczak, @m110, and @Thorleon, First, I really apologize for the long delay and the radio silence! Life has been extremely busy on my end over the past few months, so I had to put this on hold for a while. I have now refactored the PR and isolated the OpenTelemetry implementation into Could you please take another look when you have a moment? Thank you so much for your patience and support! |
Signed-off-by: ahkui ahkui@outlook.com
#574 #319
Motivation / Background
Implement metrics collection using the OpenTelemetry MeterProvider
Detail
Implement metrics collection using the OpenTelemetry MeterProvider
Alternative approaches considered (if applicable)
Checklist
The resources of our team are limited. There are a couple of things that you can do to help us merge your PR faster:
make up.make test_shortfor a quick check.make test.