-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
Update proc plugin yaml #15460
Update proc plugin yaml #15460
Conversation
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.
@Ancairon Please fix the indentation issues flagged by yamllint.
most_popular: false | ||
overview: | ||
data_collection: | ||
metrics_description: '' | ||
metrics_description: 'Examine Entropy metrics for insights into random number generation.' |
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.
I would suggest leaving this empty. There’s a single metric, and there will always be a single metric, so not much point in providing a description at the top.
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.
will leave empty
most_popular: false | ||
overview: | ||
data_collection: | ||
metrics_description: '' | ||
metrics_description: "This integration provides the number of service requests that have been sent to the system's various interrupt lines." |
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.
Wrong section, load average has nothing to do with interrupts.
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.
that's true, must've slipped from my awareness, and inputted the wrong thing.
Co-authored-by: Austin S. Hemmelgarn <austin@netdata.cloud>
icon_filename: '' | ||
categories: | ||
- data-collection.linux-systems.filesystem-metrics.nfs | ||
icon_filename: 'syslog.png' |
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.
A network or filesystem related icon is probably better for this one.
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.
will search for one
icon_filename: '' | ||
categories: | ||
- data-collection.linux-systems.filesystem-metrics.nfs | ||
icon_filename: 'syslog.png' |
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.
Just like with the NFS client, this should probably be a filesystem or network related icon instead.
Co-authored-by: Austin S. Hemmelgarn <austin@netdata.cloud>
Summary
Using this message from slack and googling where needed, updated the proc plugin. Also added our first nested categories inside linux-systems.
This was done manually, by hand, so maybe it is best to review quickly and actually see the result on testing to pick out any mistakes or hiccups.