-
Notifications
You must be signed in to change notification settings - Fork 713
Deprecate cpu-utilization and add cpu-utilization-pct for multi-core support #1515
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
base: master
Are you sure you want to change the base?
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -25,7 +25,15 @@ module openconfig-procmon { | |
| "This module provides data definitions for process health | ||
| monitoring of one or more processes running on the system."; | ||
|
|
||
| oc-ext:openconfig-version "0.4.0"; | ||
| oc-ext:openconfig-version "0.5.0"; | ||
|
|
||
| revision "2026-07-14" { | ||
| description | ||
| "Deprecate cpu-utilization and add cpu-utilization-pct as | ||
| uint32 to support multi-core systems where CPU utilization | ||
| can exceed 100 percent."; | ||
| reference "0.5.0"; | ||
| } | ||
|
|
||
| revision "2019-03-15" { | ||
| description | ||
|
|
@@ -153,9 +161,22 @@ module openconfig-procmon { | |
| } | ||
|
|
||
| leaf cpu-utilization { | ||
| status deprecated; | ||
| type oc-types:percentage; | ||
| description | ||
| "The percentage of CPU that is being used by the process."; | ||
| "The percentage of CPU that is being used by the process. | ||
| This leaf is deprecated because its type constrains values | ||
| to 0-100, which is insufficient for multi-core systems. | ||
| Use cpu-utilization-pct instead."; | ||
| } | ||
|
|
||
| leaf cpu-utilization-pct { | ||
| type uint32; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. One additional thought that I think would be helpful here for various renderings is make this a typedef, take the type, units, description verbatim - call the typedef something like |
||
| units "percent"; | ||
| description | ||
| "The percentage of CPU that is being used by the process. | ||
| On multi-core systems, this value may exceed 100 percent, | ||
| up to 100 percent per core."; | ||
| } | ||
|
charanjith-anet marked this conversation as resolved.
|
||
|
|
||
| leaf memory-usage { | ||
|
|
||
Uh oh!
There was an error while loading. Please reload this page.