Add datadog apm support#205
Conversation
ziru
left a comment
ziru
left a comment
There was a problem hiding this comment.
lgtm.
not sure how dd-apm works for go projects. ok to try and see.
hopefully, we will follow the similar service naming conventions.
https://app.datadoghq.com/apm/services?end=1548721474596&env=prod&paused=false&start=1548720574596
also, do you mind adding the instructions for golang project to this guru card?
https://app.getguru.com/card/czKok5ji/Enable-Datadog-APM-for-a-project
|
|
||
| if Config.StatsdAPMEnabled { | ||
| tracer.Start( | ||
| tracer.WithAgentAddr(fmt.Sprintf("%s:%s", Config.StatsdHost, Config.StatsdPort)), |
There was a problem hiding this comment.
@zhouzhuojie apm uses tcp port 8126, not udp port 8125.
There was a problem hiding this comment.
Ok, I will add another config then, is it by design that we use different ports in the agent setup?
| "github.com/sirupsen/logrus" | ||
| "github.com/urfave/negroni" | ||
| "github.com/yadvendar/negroni-newrelic-go-agent" | ||
| "gopkg.in/DataDog/dd-trace-go.v1/ddtrace/tracer" |
There was a problem hiding this comment.
So this lib includes tracers for http/gRPC out of the box? No need for further config
There was a problem hiding this comment.
no, it doesn't trace further. It logs the whole span. If we want to trace internal spans, we need to instrument the http client, DB, or other clients. Datadog provides wrappers for those clients.
And also, Flagr has no dependencies in the evaluation requests.
zhouzhuojie
force-pushed
the
zz/add-dd-apm-support
branch
from
a008848 to
6aed959
Compare
Codecov Report
@@ Coverage Diff @@
## master #205 +/- ##
==========================================
- Coverage 86.66% 86.14% -0.52%
==========================================
Files 23 23
Lines 1327 1335 +8
==========================================
Hits 1150 1150
- Misses 128 135 +7
- Partials 49 50 +1
Continue to review full report at Codecov.
|
5 participants
Description
Adding supprt of datadog APM. It's by default disabled. Docs:https://docs.datadoghq.com/tracing/languages/go/
Motivation and Context
To support APM
How Has This Been Tested?
Local, unit and integration tests.
Types of changes
Checklist: