Comments (12)
@tylerbenson Just verified it's working properly. 👍
from dd-trace-java.
@duke-cliff Would you be able to create a separate issue for this? It would also help if you could provide a small sample app that demonstrates the problem. Thanks!
from dd-trace-java.
I can confirm that switching to use the following fixes the traces.
<dependency>
<groupId>org.springframework.boot</groupId>
<artifactId>spring-boot-starter-web</artifactId>
</dependency>
I also tried using a non-flux/mono endpoint when running Netty, but this didn't fix the issue as both of these do not trace when using flux/Netty
@RequestMapping(value = "/", produces = MediaType.APPLICATION_JSON_UTF8_VALUE)
public Mono<Response> index() {
return Mono.just(new Response("OK"));
}
@RequestMapping(value = "/home", produces = MediaType.APPLICATION_JSON_UTF8_VALUE)
public Response index2() {
return new Response("OK");
}
from dd-trace-java.
Thanks for your report. The netty framework is on our short list of things to instrument. When that is done, we will look into compatibility with Spring Boot 2.
from dd-trace-java.
What's the roadmap for this? We are having the same issue few of reactive services are kind of blind right now.
from dd-trace-java.
@duke-cliff @ctoestreich I recently implemented netty support in #352, which was just merged into master. We will be testing compatibility of that instrumentation with webflux over the next few weeks. If you'd like to try it out before hand, you can download the snapshot build and enable it by setting this system property at startup: -Ddd.integration.netty.enabled=true
. If you do try that out, let me know if it works.
Thanks!
from dd-trace-java.
I was able to verify that the the APM tracing is not working in a http://micronaut.io application that is based on netty. I am unable to find anywhere in this repo where the check the actually done for dd.integration.netty.enabled. Perhaps I missed it, but here is my app stack that isn't working.
https://github.com/tardisoft/micronaut-metrics-example
I will double check for spring boot 2 to see if that works.
from dd-trace-java.
@duke-cliff Can you please test the latest release of the java datadog agent on one of our Spring Webflux apps and one of our Vert.x apps? Please report your findings here and for @tylerbenson to see. Thank you.
from dd-trace-java.
Okay will do
from dd-trace-java.
It appears the netty instrumentation was broken in some cases. I believe this is resolved by a combination of #411 (released in 0.12.0
) and #417 (which will be released in 0.13.0 -- you can find the snapshot build here).
We have tests (#434) that verify the netty instrumentation works for webflux on master, but are continuing to improve specific instrumentation for it.
This will still require -Ddd.integration.netty.enabled=true
to enable it though.
Thanks for your patience and look forward to feedback.
from dd-trace-java.
This is one more issue, I found the stack could not show our reactive redis calls. I have -Ddd.integration.lettuce.enabled=true in the arguments. @tylerbenson
from dd-trace-java.
@duke-cliff @ctoestreich Additional Spring Webflux instrumentation has been added to our latest release: https://github.com/DataDog/dd-trace-java/releases/tag/v0.15.0
from dd-trace-java.
Related Issues (20)
- Memory Leak Issue with Datadog Integration in Kotlin/Ktor Application
- Add ability to provide custom normalization of URI mapping saved as resource name
- Cannot decode v0.4 traces payload: msgp: attempted to decode type "array" with method for "str" HOT 1
- feature request - instrumentation for https://github.com/http-kit/http-kit
- Trace id in logs are different from datadog trace id in web HOT 2
- dd-trace-java v1.31.2 crashes the JVM HOT 4
- RabbitMQ instrumentation tests is flaky HOT 1
- Spring Webflux 5 instrumentation tests are flaky
- Vert.X Redis client 4 instrumentation tests are flaky
- Grizzly http 2.3.20 instrumentation tests are flaky HOT 1
- Feature Request: Missing trace for JUnit5 BeforeAll/AfterAll methods in CIVisibility Instrumentation HOT 3
- BUG: dd-trace-java with version 1.33.0 causes InvalidCookieDomainException. In 1.32.0 works fine. HOT 3
- Live-heap profiler (alpha) in dd-trace-java v1.33.0 crashes the JVM. (After updated from v1.32.0) HOT 6
- Play Scala integration tests are flaky
- Spark instrumentation tests are flaky
- DD agent is not compatible with Quarkus 3.10 HOT 1
- Gradle smoke tests are flaky
- Maven smoke tests are flaky
- JDBC connection service mapping not being used, and traceparent missing, in SQL comment HOT 2
- With Unix Domain Socket, DD_AGENT_HOST not ignored despite DD_TRACE_AGENT_URL HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from dd-trace-java.