Skip to content

Conversation

@thaJeztah
Copy link
Member

cherry-pick was mostly clean; conflict in vendor/modules.txt due to #12604 not being in this branch.

pkg/tracing: HTTPStatusCodeAttributes: remove use of deprecated SemConv const

The "http.status_code" attribute was deprecated in SemConv v1.21 in favor of "http.response.status_code", and the HTTPStatusCodeKey const was deprecated in SemConv v1.22.

The HTTPStatusCodeAttributes utility doesn't appear to be used currently, but let's update it to use both the old and new variants in case someone still uses it.

(cherry picked from commit 0d27fce)

…nv const

The "http.status_code" attribute was deprecated in [SemConv v1.21] in favor
of "http.response.status_code", and the `HTTPStatusCodeKey` const was deprecated
in [SemConv v1.22].

The `HTTPStatusCodeAttributes` utility doesn't appear to be used currently, but
let's update it to use both the old and new variants in case someone still uses
it.

[SemConv v1.21]: https://github.com/open-telemetry/opentelemetry-go/blob/v1.38.0/semconv/v1.21.0/attribute_group.go#L106-L114
[SemConv v1.22]: https://github.com/open-telemetry/opentelemetry-go/blob/v1.38.0/semconv/v1.22.0/attribute_group.go#L1444-L1452

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
(cherry picked from commit 0d27fce)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@github-project-automation github-project-automation bot moved this to Needs Triage in Pull Request Review Dec 17, 2025
@dosubot dosubot bot added the dependencies Pull requests that update a dependency file label Dec 17, 2025
@github-project-automation github-project-automation bot moved this from Needs Triage to Review In Progress in Pull Request Review Dec 17, 2025
@estesp estesp merged commit 815932a into containerd:release/2.2 Dec 17, 2025
52 checks passed
@github-project-automation github-project-automation bot moved this from Review In Progress to Done in Pull Request Review Dec 17, 2025
@thaJeztah thaJeztah deleted the release_2.2_backport_bump_semconv branch December 17, 2025 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file size/XXL

Projects

Development

Successfully merging this pull request may close these issues.

4 participants