Support OpenTelemetry deployment environment names - #12324
Support OpenTelemetry deployment environment names#12324gh-worker-dd-mergequeue-cf854d[bot] merged 1 commit into
Conversation
There was a problem hiding this comment.
More details
The change maps deployment.environment.name to the Datadog environment. It keeps deployment.environment as a fallback and removes both keys from generic tags.
🤖 Datadog Autotest · Commit 65b75ba · What is Autotest? · @DataDog review to ask questions · Any feedback? Reach out in #autotest
|
🎯 Code Coverage (details) 🔗 Commit SHA: 7c2b26c | Docs | View more details | Give us feedback! |
🟢 Java Benchmark SLOs — All performance SLOs passed
PR vs. master results
Commit: Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion. |
zacharycmontoya
left a comment
There was a problem hiding this comment.
Functionally, LGTM so we can understand either OTel convention for specifying the application environment. I'd love to get a Java reviewer to confirm the testing strategy
mhlidd
left a comment
There was a problem hiding this comment.
LGTM following an additional test case.
65b75ba to
7c2b26c
Compare
|
/merge |
|
View all feedbacks in Devflow UI.
The expected merge time in
|
What Does This Do
Maps the stable OpenTelemetry
deployment.environment.nameresource attribute to the Datadog environment while retainingdeployment.environmentas a legacy fallback. When both attributes are present, the stable name takes precedence.Motivation
OpenTelemetry deprecated
deployment.environmentin favor ofdeployment.environment.name. Applications using the current semantic convention were not assigned the expected Datadog environment.Additional Notes
Validated with focused configuration tests and Spotless checks.
Contributor Checklist