Skip to content

Conversation

@ChrsMark
Copy link
Member

@ChrsMark ChrsMark commented Dec 2, 2025

Description

Related to #22095.

This PR verifies the analysis of open-telemetry/semantic-conventions#3120 which only identifies container.image.tag as a breaking change.

For now I changed that to use a local reference and added a todo to handle this later. The rest of attributes can be referenced directly from the latest semconv.

The container.image.tag transition will be handled by #44589.

Link to tracking issue

Fixes #44696

Testing

Documentation

@github-actions github-actions bot requested a review from povilasv December 2, 2025 14:10
@ChrsMark ChrsMark force-pushed the k8sattributes_upgratesemconv branch 3 times, most recently from a955b01 to 58ab581 Compare December 2, 2025 14:37
Copy link
Contributor

@atoulme atoulme left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ChrsMark ChrsMark force-pushed the k8sattributes_upgratesemconv branch 2 times, most recently from 784cac7 to f3684e7 Compare December 2, 2025 15:48
@ChrsMark ChrsMark force-pushed the k8sattributes_upgratesemconv branch from f3684e7 to b21d021 Compare December 2, 2025 15:49
@ChrsMark
Copy link
Member Author

ChrsMark commented Dec 4, 2025

@TylerHelmuth @dmitryax PTAL

@TylerHelmuth TylerHelmuth merged commit 9cbe6fb into open-telemetry:main Dec 4, 2025
189 checks passed
@github-actions github-actions bot added this to the next release milestone Dec 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

processor/k8sattributes k8s Attributes processor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[k8sattributes] Update semconv version

6 participants