NETWAYS / check_system_basics

A Monitoring Plugin to test various basic operating system properties (on Linux for now)
GNU General Public License v2.0
6 stars 1 forks source link

[Bug]: logical swap in psi-feature #37

Closed K0nne closed 4 months ago

K0nne commented 4 months ago

Please try to fill out as much of the information below as you can. Thank you!

Which version contains the bug?

latest, but check doesn't have a feature to show the current version

Describe the bug

The check returns "critical", but the "avg" values are at 0.00. Instead the "some" values are increased. Btw, currently there is no "Version" flag.

call
./check_system_basics psi

result

[CRITICAL] - states: critical=1 ok=2
\_ [CRITICAL] CPU Full Pressure - Avg10: 0.00, Avg60: 0.00, Avg300: 0.00
\_ [OK] IO Pressure - Avg10: 0.00, Avg60: 0.00, Avg300: 0.00
\_ [OK] Memory Pressure - Avg10: 0.00, Avg60: 0.00, Avg300: 0.00
|cpu-some-avg10=99.97%;@30:100;@95:100;0;100 cpu-some-avg60=98.94%;@30:100;@95:100;0;100 cpu-some-avg300=62.73%;@30:100;@95:100;0;100 cpu-some-total=568928247c;@30:100;@95:100;0 cpu-full-avg10=0%;@30:100;@95:100;0;100
cpu-full-avg60=0%;@30:100;@95:100;0;100 cpu-full-avg300=0%;;;0;100 cpu-full-total=0c;;;0 io-some-avg10=0%;@30:100;@95:100;0;100 io-some-avg60=0%;@30:100;@95:100;0;100 io-some-avg300=0%;@30:100;@95:100;0;100 
io-some-total=218528202c;@30:100;@95:100;0 io-full-avg10=0%;@30:100;@95:100;0;100 io-full-avg60=0%;@30:100;@95:100;0;100 io-full-avg300=0%;;;0;100 io-full-total=206215269c;;;0 memory-some-avg10=0%;@30:100;@95:100;0;100 
memory-some-avg60=0%;@30:100;@95:100;0;100 memory-some-avg300=0%;@30:100;@95:100;0;100 memory-some-total=11147249c;@30:100;@95:100;0 memory-full-avg10=0%;@30:100;@95:100;0;100 memory-full-avg60=0%;@30:100;@95:100;0;100 
memory-full-avg300=0%;;;0;100 memory-full-total=10606884c;;;0

How to recreate the bug?

./check_system_basics psi and give it some load (yes > /dev/null &).

martialblog commented 4 months ago

@RincewindsHat Can you have a look at this?

RincewindsHat commented 4 months ago

Jup, got it on the screen. @K0nne is right, this is at least confusing. I will think about it.