From 45c2bb6293e7e93bc47a4f9e629ad83d93c63cdf Mon Sep 17 00:00:00 2001 From: Trask Stalnaker Date: Thu, 6 Apr 2023 21:36:41 -0700 Subject: [PATCH] BREAKING: Remove http.status_code attribute from the http.server.active_requests metric (#3366) --- CHANGELOG.md | 3 +++ semantic_conventions/metrics/http.yaml | 1 - specification/metrics/semantic_conventions/http-metrics.md | 1 - 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index fc550d05eb7..306bce3fa06 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -71,6 +71,9 @@ release. ([#3347](https://github.com/open-telemetry/opentelemetry-specification/pull/3347)) - Change `db.statement` to only be collected if there is sanitization. ([#3127](https://github.com/open-telemetry/opentelemetry-specification/pull/3127)) +- BREAKING: Remove `http.status_code` attribute from the + `http.server.active_requests` metric. + ([#3366](https://github.com/open-telemetry/opentelemetry-specification/pull/3366)) ### Compatibility diff --git a/semantic_conventions/metrics/http.yaml b/semantic_conventions/metrics/http.yaml index 2ae7581c4e1..ed1fc781844 100644 --- a/semantic_conventions/metrics/http.yaml +++ b/semantic_conventions/metrics/http.yaml @@ -21,7 +21,6 @@ groups: unit: "{request}" attributes: - ref: http.method - - ref: http.status_code - ref: http.scheme - ref: net.host.name requirement_level: required diff --git a/specification/metrics/semantic_conventions/http-metrics.md b/specification/metrics/semantic_conventions/http-metrics.md index b76087662dc..e18bb475bae 100644 --- a/specification/metrics/semantic_conventions/http-metrics.md +++ b/specification/metrics/semantic_conventions/http-metrics.md @@ -90,7 +90,6 @@ This metric is optional. |---|---|---|---|---| | `http.method` | string | HTTP request method. | `GET`; `POST`; `HEAD` | Required | | `http.scheme` | string | The URI scheme identifying the used protocol. | `http`; `https` | Required | -| `http.status_code` | int | [HTTP response status code](https://tools.ietf.org/html/rfc7231#section-6). | `200` | Conditionally Required: If and only if one was received/sent. | | [`net.host.name`](../../trace/semantic_conventions/span-general.md) | string | Name of the local HTTP server that received the request. [1] | `localhost` | Required | | [`net.host.port`](../../trace/semantic_conventions/span-general.md) | int | Port of the local HTTP server that received the request. [2] | `8080` | Conditionally Required: [3] |