Skip to content

Commit

Permalink
BREAKING: Remove http.status_code attribute from the http.server.acti…
Browse files Browse the repository at this point in the history
…ve_requests metric (#3366)
  • Loading branch information
trask authored Apr 7, 2023
1 parent 7956b56 commit 52b3578
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 2 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -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

Expand Down
1 change: 0 additions & 1 deletion semantic_conventions/metrics/http.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
1 change: 0 additions & 1 deletion specification/metrics/semantic_conventions/http-metrics.md
Original file line number Diff line number Diff line change
Expand Up @@ -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] |

Expand Down

0 comments on commit 52b3578

Please sign in to comment.