-
Notifications
You must be signed in to change notification settings - Fork 104
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 Micrometer tracing #554
Add Micrometer tracing #554
Conversation
Adds the required dependencies for Micrometer Tracing w/ the Brave bridge reporting to Wavefront or Zipkin. Resolves spring-cloud#553
It is already brought in transitively via prometheus-rsocket-spring.
@onobc , looks like something is missing yet: https://github.com/spring-cloud/stream-applications/actions/runs/10707057201. Apparently the deps management must be not only in one place. Thanks |
Looking into it. I think the update I did w/ transitive dependency had a side effect as this was working locally. I will get it squared away and a fix submitted. Thanks for the review and merge. |
This PR adds the required dependencies for Micrometer Tracing with the Brave bridge
with the ability to report trace to Zipkin or Wavefront.
Also the following unrelated commits are piggybacking:
Important
During merge please keep the commits separate as they are unrelated