-
Notifications
You must be signed in to change notification settings - Fork 372
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Auto-instrumentation with service_name from environmental variable #2455
Auto-instrumentation with service_name from environmental variable #2455
Conversation
b620137
to
26637c8
Compare
26637c8
to
3d4593f
Compare
Codecov Report
@@ Coverage Diff @@
## master #2455 +/- ##
==========================================
+ Coverage 98.01% 98.03% +0.01%
==========================================
Files 1105 1109 +4
Lines 59276 59951 +675
==========================================
+ Hits 58100 58771 +671
- Misses 1176 1180 +4
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Suggestion: rename My thought is |
@@ -43,6 +44,7 @@ | |||
|
|||
it_behaves_like 'measured span for integration' | |||
it_behaves_like 'a peer service span' | |||
it_behaves_like 'environment service name', 'DD_TRACE_AWS_SERVICE_NAME' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job on creating a shared example! Super clean! 🧹
What does this PR do?
Allow integration service name being configured by environment variable
aws
DD_TRACE_AWS_SERVICE_NAME
dalli
DD_TRACE_DALLI_SERVICE_NAME
elasticsearch
DD_TRACE_ELASTICSEARCH_SERVICE_NAME
ethon
DD_TRACE_ETHON_SERVICE_NAME
excon
DD_TRACE_EXCON_SERVICE_NAME
faraday
DD_TRACE_FARADAY_SERVICE_NAME
grpc
DD_TRACE_GRPC_SERVICE_NAME
http
DD_TRACE_NET_HTTP_SERVICE_NAME
httpclient
DD_TRACE_HTTPCLIENT_SERVICE_NAME
httprb
DD_TRACE_HTTPRB_SERVICE_NAME
mongodb
DD_TRACE_MONGO_SERVICE_NAME
mysql2
DD_TRACE_MYSQL2_SERVICE_NAME
pg
DD_TRACE_PG_SERVICE_NAME
presto
DD_TRACE_PRESTO_SERVICE_NAME
redis
DD_TRACE_REDIS_SERVICE_NAME
rest-client
DD_TRACE_REST_CLIENT_SERVICE_NAME