-
Notifications
You must be signed in to change notification settings - Fork 494
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
NAS-133683 / 25.04 / Refactor CPU reporting #15445
base: master
Are you sure you want to change the base?
Conversation
6ddce00
to
23059e4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thank you! Please do not merge until UI has a PR up and can adapt to these changes since this affects the dashboard widgets.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
temp isn't working in my tests.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I was wrong. This is working, re-approving (but still don't merge yet).
23059e4
to
9479e43
Compare
Hello. I am getting this response in
|
Context
It was requested that we make some refactoring changes to our CPU reporting. Following points have been catered to:
/proc/stat
has been disableduser/nice etc
are not being saved/retrievedExample payload of what we send in realtime reporting for CPU metrics