Closed rafal-dudek closed 3 weeks ago
We are also facing this issue Spring boot - 3.3.2 com.google.cloud:spring-cloud-gcp-dependencies - 5.5.0 org.springframework.cloud:spring-cloud-dependencies - 2023.0.3
Anyone up for making a pull request to fix this and add tests for it? Looks like the AttributesExtractor used by the other encoder has null checks for IP: https://github.com/openzipkin/zipkin-gcp/blob/829844a2b12b17d49ffc3010d0020eb4bf3bba1d/translation-stackdriver/src/main/java/zipkin2/translation/stackdriver/AttributesExtractor.java#L64-L71
We get this problem when running spring on cloud run
Would folks affected by this be able to test with snapshots to confirm there aren't any remaining issues with the fix now merged?
We've tested with Os's PR and it fixes the issue for us, thanks
oh, that's great to hear! 🥳 how is your release schedule? when can we expect the fix to be released?
Releases for this repo are ad hoc. I'll try to find time to do a release this week.
With #226 merged, I think we have everything ready for the release. If anyone would like to try out snapshots before the release to make sure everything is good, that'd be appreciated.
I tried the 2.2.5-SNAPSHOT and it worked for our deployment
2.2.5 is released now with this fix.
Describe the Bug
When localIP is missing in Server braveSpan, it causes exception:
Steps to Reproduce
Make sure that your Server Span is missing local IP address. Not sure how to reproduce it exactly - the problem occurs in our Integration tests only in Jenkins Agent on Kubernetes, locally the IP address is present. In our case it is for Spring Boot 3.3.2 MVC RestController span.
Expected Behaviour
There should be check if
braveSpan.localIp()!=null
before invokingInetAddresses.forString(String ipString)
on it: https://github.com/openzipkin/zipkin-gcp/blob/master/encoder-stackdriver-brave/src/main/java/zipkin2/reporter/stackdriver/brave/AttributesExtractor.java#L76If localIP is null, do not set any attribute for it.