[checkmk-commits] [Checkmk/checkmk] cb3857: nfsiostat: fix units

Branch: refs/heads/2.2.0 Home: https://github.com/Checkmk/checkmk Commit: cb385786e9071865eb3f424a78f4eddcb21575e1 https://github.com/Checkmk/checkmk/commit/cb385786e9071865eb3f424a78f4eddcb… Author: Benedikt Seidl <benedikt.seidl(a)checkmk.com> Date: 2023-10-31 (Tue, 31 Oct 2023) Changed paths: M cmk/base/plugins/agent_based/nfsiostat.py M cmk/gui/plugins/metrics/network.py M tests/unit/cmk/base/plugins/agent_based/test_nfsiostat.py Log Message: ----------- nfsiostat: fix units metric name mentions ms. output of nfsiostat mentions ms. now the graphs also display ms for rtt and exe metrics. Change-Id: I65212fabed8f539766e06a0fbcdc8dbec7bca128 Commit: 66c4cf66a16f932a52ccd6f1e6051d556029fb03 https://github.com/Checkmk/checkmk/commit/66c4cf66a16f932a52ccd6f1e6051d556… Author: Benedikt Seidl <benedikt.seidl(a)checkmk.com> Date: 2023-10-31 (Tue, 31 Oct 2023) Changed paths: A .werks/15299 M checkman/nfsiostat M cmk/base/plugins/agent_based/nfsiostat.py M tests/unit/cmk/base/plugins/agent_based/test_nfsiostat.py Log Message: ----------- 15299 FIX nfsiostat: fix units and parsing Units falsely displayed seconds, although milliseconds are corret. Newer versions of nfsiostat output more data, this confused our parser. Before we parsed just the first 16 numbers: 2 global, then 7 for read and 7 for write. But nfsiostat will now report 22 numbers: 2 global, 10 for read, and 10 for write, so the first 16 numbers have a total different meaning now. We now parse the first 2 numbers, and then 7 numbers after "read:" and 7 numbers after "write:", so we are compatible with the old and new format with just one parse function. Change-Id: I6a3b467f1f5d0ac7a2e343ebd55e3af1b5e67465 Compare: https://github.com/Checkmk/checkmk/compare/c0d3648b15a1...66c4cf66a16f