-
Notifications
You must be signed in to change notification settings - Fork 32
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
Tiny improvements for template details #241
base: main
Are you sure you want to change the base?
Conversation
Hello @blindzero and thank you for contributing! Have you also tested the disk graph? Best, |
Hi @Al2Klimov |
d9a85e3
to
645f230
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.
Ran check_disk and check_swap on the command line. Swap seems to actually yield free space, but disk yields the used space.
@bobapple Does msec vs sec matter depending on whether your Graphite data is in s or ms?
ping.ini icmp.ini hostalive.ini
Changed yUnitSystem from "none" to "msec" to be more accurate
disk.ini swap.ini
Changed legend from "Used" to "Free" as the default perfdata output of Icinga plugins delivered are based on this. Otherwise the graphs are "wrong way around". The screenshot of changed behaviour should explain it as a living example. If the filled area would be "Used" the state would be critical (nearly all Swap used), but it isn't as the filled area is still available.
P.S.: This is my first other project pull request, so please be gentle ;)