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

refactor: move system cpu-related metrics to cpu namespace #1896

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

rogercoll
Copy link
Contributor

@rogercoll rogercoll commented Feb 10, 2025

Partially fixes #1873 (renaming)

Changes

Current system.cpu.* metrics do not reflect the CPU time/usage over the whole system but relative to a logical CPU. Each reported metric is specific to a logical CPU (system.cpu.logical_number). This PR proposes moving them to a new CPU namespace that will define metrics relative to a unique logical CPU. Additionally, the system.cpu.logical_number has been renamed to cpu.logical_number as it only applies to cpu namespace metrics.

Merge requirement checklist

@rogercoll rogercoll marked this pull request as ready for review February 10, 2025 11:36
@rogercoll rogercoll requested review from a team as code owners February 10, 2025 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Development

Successfully merging this pull request may close these issues.

Introduce an optional normalized total CPU utilization metric
1 participant