Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[receiver/hostmetrics] Support collecting process cgroup as dimension #29282

Closed
atoulme opened this issue Nov 15, 2023 · 5 comments · Fixed by #29621
Closed

[receiver/hostmetrics] Support collecting process cgroup as dimension #29282

atoulme opened this issue Nov 15, 2023 · 5 comments · Fixed by #29621
Assignees
Labels

Comments

@atoulme
Copy link
Contributor

atoulme commented Nov 15, 2023

Component(s)

receiver/hostmetrics

Is your feature request related to a problem? Please describe.

I would like to collect the cgroup associated with a process when collecting process information through the hostmetrics receiver.

Describe the solution you'd like

The cgroup information should be offered as an additional dimension of the metric related to processes.

Describe alternatives you've considered

No response

Additional context

No response

@atoulme atoulme added enhancement New feature or request needs triage New item requiring triage labels Nov 15, 2023
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@atoulme
Copy link
Contributor Author

atoulme commented Nov 16, 2023

Opening a PR to support collect cgroup info in gopsutil: shirou/gopsutil#1552

@atoulme atoulme removed the needs triage New item requiring triage label Nov 16, 2023
@andrzej-stencel
Copy link
Member

I suppose it makes sense. Do you want to assign yourself to the issue?

@atoulme atoulme self-assigned this Nov 27, 2023
@atoulme
Copy link
Contributor Author

atoulme commented Nov 29, 2023

gopsutil turned down the enhancement of exposing cgroups, but we can probably perform this data capture in the receiver instead directly. I will turn my attention to that aspect.

@atoulme atoulme mentioned this issue Dec 1, 2023
Copy link
Contributor

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@github-actions github-actions bot added the Stale label Jan 29, 2024
dmitryax pushed a commit that referenced this issue Feb 16, 2024
**Description:**
Adds `process.cgroup` resource attribute to process metrics

**Link to tracking Issue:**
Fixes #29282

---------

Co-authored-by: Andrzej Stencel <astencel@sumologic.com>
djaglowski pushed a commit to djaglowski/opentelemetry-collector-contrib that referenced this issue Feb 19, 2024
**Description:**
Adds `process.cgroup` resource attribute to process metrics

**Link to tracking Issue:**
Fixes open-telemetry#29282

---------

Co-authored-by: Andrzej Stencel <astencel@sumologic.com>
djaglowski pushed a commit to djaglowski/opentelemetry-collector-contrib that referenced this issue Feb 19, 2024
**Description:**
Adds `process.cgroup` resource attribute to process metrics

**Link to tracking Issue:**
Fixes open-telemetry#29282

---------

Co-authored-by: Andrzej Stencel <astencel@sumologic.com>
XinRanZhAWS pushed a commit to XinRanZhAWS/opentelemetry-collector-contrib that referenced this issue Mar 13, 2024
**Description:**
Adds `process.cgroup` resource attribute to process metrics

**Link to tracking Issue:**
Fixes open-telemetry#29282

---------

Co-authored-by: Andrzej Stencel <astencel@sumologic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants