Skip to content

Add Linux process cgroup attribute#1364

Merged
lmolkova merged 16 commits intoopen-telemetry:mainfrom
rogercoll:add_process_cgroup
Nov 25, 2024
Merged

Add Linux process cgroup attribute#1364
lmolkova merged 16 commits intoopen-telemetry:mainfrom
rogercoll:add_process_cgroup

Conversation

@rogercoll
Copy link
Copy Markdown
Contributor

Fixes # #1357

Changes

Please provide a brief description of the changes here.

Note: if the PR is touching an area that is not listed in the existing areas, or the area does not have sufficient domain experts coverage, the PR might be tagged as experts needed and move slowly until experts are identified.

Merge requirement checklist

@rogercoll rogercoll requested review from a team August 23, 2024 18:39
Comment thread model/registry/linux.yaml Outdated
@github-actions
Copy link
Copy Markdown

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

@github-actions github-actions Bot added Stale and removed Stale labels Sep 15, 2024
Comment thread model/registry/linux.yaml Outdated
Comment thread model/registry/linux.yaml Outdated
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Oct 8, 2024

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

@github-actions github-actions Bot added the Stale label Oct 8, 2024
@rogercoll rogercoll requested review from a team as code owners October 9, 2024 15:42
@github-actions github-actions Bot removed the Stale label Oct 10, 2024
@rogercoll rogercoll requested a review from a team as a code owner October 11, 2024 12:47
@rogercoll
Copy link
Copy Markdown
Contributor Author

@open-telemetry/semconv-system-approvers Could you take a look at this PR when you have a moment? Thanks!

Comment thread model/linux/registry.yaml Outdated
@jsuereth
Copy link
Copy Markdown
Contributor

Would you mind taking a look at the changelog error and fixing that?

@joaopgrassi
Copy link
Copy Markdown
Member

Would you mind taking a look at the changelog error and fixing that?

This was a problem on main, updating the PR has now fixed it 👍

Comment thread model/linux/registry.yaml Outdated
Comment thread model/linux/registry.yaml Outdated
@mmanciop
Copy link
Copy Markdown
Contributor

I love the idea of having cgroup identifiers at hand: in Kubernetes that means that the k8s.pod.ui, which is part of the cgroup identifier, is just one OTTL rule away :-)

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

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

9 participants