Add Micrometer tracing - Githubissues

spring-cloud / stream-applications

Functions and Spring Cloud Stream Applications for data driven microservices

https://spring.io/projects/spring-cloud-stream-applications

250 stars 104 forks source link

Add Micrometer tracing #554

Closed onobc closed 1 week ago

onobc commented 2 weeks ago

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:

covenience script to do a full build w/o tests
update to Prometheus Rsocket 2.0.0-M1

[!IMPORTANT] During merge please keep the commits separate as they are unrelated

artembilan commented 1 week ago

@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

onobc commented 1 week ago

@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.