Comments (9)
Same issue fore the "enabled" config.
It's not copied properly.
from dd-trace-go.
I can see how this can be confusing. The integrations (contrib/*
) are not yet migrated over to use OpenTracing. We are planning to do this for 0.7.0, which will be all OpenTracing compatible. Until then, please hold tight 😊
from dd-trace-go.
OK I understand, thanks.
from dd-trace-go.
The integrations (contrib/*) are not yet migrated over to use OpenTracing
Indeed it doesn't work currently.
In my code, I do this:
- create a root span with the gorilla/mux integration
- create new child span from request's context with open tracing
The result:
- in https://app.datadoghq.com/apm/services I only see the spans created by gorilla/mux, and the child span are not there
- in https://app.datadoghq.com/apm/search I see all span (gorilla/mux + child)
If I understand correctly, the issue is that the span types are not compatible, and the parent span is not found from context (different key probably).
I see several solutions for me:
- wait for the integration migration (when will it be available ?)
- build my own integration compatible with opentracing
- contribute to the existing integration (it should be very similar to my own code)
I'd like to contribute, but I don't know what are your plans.
from dd-trace-go.
PS: if I use only the opentracing span as root, there is no issue.
from dd-trace-go.
wait for the integration migration (when will it be available ?)
I am working on this as we speak and is my main priority at the moment. We plan to release this with 0.7.0 once we dog food it internally. You are welcome to help with that if you want, at your own risk :-)
contribute to the existing integration (it should be very similar to my own code)
I would not advise to do this now, as it will result in terrible merge conflicts.
I'd like to contribute, but I don't know what are your plans.
I would be very happy to have you as a contributor. I think that after we finish this piece of work, contributing to this library should be much easier and straight-forward. I am also glad to hear that you have this intention, your help would be most welcomed.
The best thing you can do for now is to be patient and give us the time to do this. I reckon in a couple of weeks it will be ready. If this is urgent for you, of course you can build your own integration in the mean time. Contributing now is not a very good time because this switch affects the entire codebase.
I am happy to answer any more questions you might have. You are also welcome to reach me on our public Slack if you want.
from dd-trace-go.
quick and dirty hack https://gist.github.com/pierrre/d6f86351ce384a30b548c8ef2ecb6ebf
from dd-trace-go.
@pierrre I will close this as a duplicate of #175 - I hope you don't mind. If that issue doesn't cover it or if this is for something else, then please feel free to re-open.
from dd-trace-go.
ok, no problem
from dd-trace-go.
Related Issues (20)
- [BUG] dd-trace-go states that it is an Apache license, but it pulls in DataDog/datadog-agent which includes eBPF code which is GPL 2.0 licensed HOT 4
- Stats for contrib/jackc/pgxpool HOT 4
- Add option to gqlgen contrib for skipping spans for field resolvers w/o methods HOT 5
- internal/telemetry: TestProductChange/profiler_start,_tracer_start is flaky HOT 1
- Snyk is reporting CVE-2023-7104 HOT 4
- propose : support Amazon EventBridge Scheduler tracing. HOT 3
- pgx query times report as application times in the Datadog UI HOT 3
- Full DBM propagation for pgx HOT 7
- Profiler: Add ability to override logger HOT 1
- Headers not propagated across custom spans within trace HOT 1
- contrib/jackc/pgx.v5: trace acquire and release HOT 1
- [BUG] Datadog Tracer v1.64.0 WARN: Turning off Data Streams Monitoring. Upgrade your agent to 7.34+ HOT 6
- proposal: integrate with github.com/twmb/franz-go HOT 1
- [BUG] fatal error: concurrent map writes when using contrib/net/http.(*roundTripper).RoundTrip HOT 1
- Remove warn logs from tracer/option newConfig HOT 1
- Go 1.19 no longer works HOT 1
- Failed to configure Datadog Profiler locally HOT 1
- labstack/echo v3.3.10 dependency vulnerability HOT 5
- Add WithStatsdClient StartOption for tracer HOT 1
- [BUG] 99designs/gqlgen/tracer.go: panic cause nil pointer
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-go.