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

Branch: refs/heads/master Home: https://github.com/Checkmk/checkmk Commit: ed8f9fd023f200be4998a551e904319d41d4b1d2 https://github.com/Checkmk/checkmk/commit/ed8f9fd023f200be4998a551e904319d4… Author: Benedikt Seidl <benedikt.seidl(a)checkmk.com> Date: 2023-10-26 (Thu, 26 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: 327710334f9bac823dfff427dcde588101300f42 https://github.com/Checkmk/checkmk/commit/327710334f9bac823dfff427dcde58810… Author: Benedikt Seidl <benedikt.seidl(a)checkmk.com> Date: 2023-10-26 (Thu, 26 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/630088b62f20...327710334f9b