Skip to content

Conversation

@ChrsMark
Copy link
Member

@ChrsMark ChrsMark commented Jun 23, 2025

Description

This PR is a follow up from #40745 to:

  • Fix the docs so as to mention that container.id can be used in rules
  • Fix the docs so as to mention that service.instance.id cannot be used in rules
  • Fix the docs so as to mention that service.version cannot be used in rules when it's calculated based on container's image
  • Fix the code so as to allow service.namespace to be used in rules

Testing

Documentation

Fixed

/cc @florianl @dmitryax @zeitlinger

@github-actions github-actions bot added the processor/k8sattributes k8s Attributes processor label Jun 23, 2025
@ChrsMark ChrsMark force-pushed the fix_k8sattrss_docs branch 2 times, most recently from 6c38397 to abd3f43 Compare June 23, 2025 08:19
@ChrsMark ChrsMark force-pushed the fix_k8sattrss_docs branch from abd3f43 to 5808818 Compare June 23, 2025 08:40
@atoulme
Copy link
Contributor

atoulme commented Jul 14, 2025

need a codeowner review from @dmitryax, @fatsheep9146, @TylerHelmuth

@github-actions
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Aug 12, 2025
@zeitlinger
Copy link
Member

need a codeowner review from @dmitryax, @fatsheep9146, @TylerHelmuth

@github-actions github-actions bot removed the Stale label Aug 13, 2025
@github-actions
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Aug 28, 2025
@dmitryax dmitryax merged commit 9d363cc into open-telemetry:main Aug 29, 2025
186 checks passed
@github-actions github-actions bot added this to the next release milestone Aug 29, 2025
@zeitlinger
Copy link
Member

thanks for fixing this!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants