Skip to content

Followup changes to implement OpenTelemetry HTTP semantic conventions for additional frameworks - #12901

Draft
zacharycmontoya wants to merge 8 commits into
otel-http-semconvfrom
zach.montoya/otel-http-semconv-followup
Draft

zacharycmontoya wants to merge 8 commits into
otel-http-semconvfrom
zach.montoya/otel-http-semconv-followup

Conversation

@zacharycmontoya

Copy link
Copy Markdown

What Does This Do

Motivation

Additional Notes

Contributor Checklist

Jira ticket: [PROJ-IDENT]

- Http client error status codes default to 400-600 (instead of 400-500)
- The environment variable DD_TRACE_OTEL_SEMANTICS_ENABLED automatically configures the following
  - OTEL_TRACES_EXPORTER=otlp
  - DD_WRITER_TYPE=OtlpWriter
  - DD_TRACE_SPAN_ATTRIBUTE_SCHEMA=v0
  - DD_TRACE_PEER_SERVICE_DEFAULTS_ENABLED=false
http:
- http.request.method_original (OTel)
- server.address (OTel)
- server.port (OTel)

http.server:
- url.path (OTel)
- url.query (OTel)
- url.scheme (OTel)

mixins:
  peer:
    tags:
      - network.peer.address (OTel)
      - peer.port (DD) => network.peer.port (OTel)

Map peer port to the OTel network attribute

Fixup http tag conventions. Squash so we can edit the message
- Apache HTTP client
- Apache HTTP Async Client
- Google HTTP Client
- HttpURLConnection
- JAX-RS Client
- OkHttp
- Play WSClient
- Akka-Http Server
- JAX-RS Server
- Jersey Server
- Jetty
- Netty HTTP server
- Servlet
- Play HTTP server
- Ratpack HTTP server
- Spring MVC
- Spring WebFlux
- Tomcat
- Undertow
- Vert.x
…the http.route for the following frameworks:

- Akka HTTP
- Play
@datadog-prod-us1-4

This comment has been minimized.

@dd-octo-sts

dd-octo-sts Bot commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

🟢 Java Benchmark SLOs — All performance SLOs passed

Suite Status
Startup 🟢 pass

SLO thresholds are defined here based on automatically generated metrics. A warning is raised when results are within 5% of the threshold.

PR vs. master results
Scenario Candidate master Δ (95% CI of mean)
startup:insecure-bank:iast:Agent 13.93 s 13.94 s [-0.8%; +0.8%] (no difference)
startup:insecure-bank:tracing:Agent 12.88 s 12.94 s [-1.1%; +0.2%] (no difference)
startup:petclinic:appsec:Agent 17.06 s 16.97 s [-0.3%; +1.3%] (no difference)
startup:petclinic:iast:Agent 16.97 s 16.91 s [-0.6%; +1.3%] (no difference)
startup:petclinic:profiling:Agent 16.02 s 16.62 s [-9.2%; +1.9%] (unstable)
startup:petclinic:sca:Agent 17.05 s 16.99 s [-0.4%; +1.2%] (no difference)
startup:petclinic:tracing:Agent 15.96 s 15.55 s [-2.0%; +7.4%] (no difference)

Commit: 5f15074c · CI Pipeline · Benchmarking Platform UI


Load and DaCapo benchmarks can be triggered manually in the GitLab pipeline. Results will appear in the Benchmarking Platform UI after completion.

@bric3
bric3 added this pull request to stack #12903 October 16, 2026 07:34

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant