[checkmk-commits] [Checkmk/checkmk] 797381: smart_stats: Look up ATA attribute names by their ID

Branch: refs/heads/2.2.0 Home: https://github.com/Checkmk/checkmk Commit: 7973817225aace5963d156b2f85f624716cdcfe2 https://github.com/Checkmk/checkmk/commit/7973817225aace5963d156b2f85f62471… Author: Andreas Umbreit &lt;andreas.umbreit(a)checkmk.com&gt; Date: 2023-06-02 (Fri, 02 Jun 2023) Changed paths: M cmk/base/plugins/agent_based/smart.py M tests/unit/cmk/base/plugins/agent_based/test_smart.py Log Message: ----------- smart_stats: Look up ATA attribute names by their ID The attribute names for ATA disks reported by smartutils turned out to be either not clearly defined (multiple names for the same attribute), or even plainly wrong. To avoid missing some attributes or to even report wrong data, we now look up the names of all actually used attributes by their attribute ID. Change-Id: Id0a196ce1a7663f1a236f28eaf4aeba14f4723dc Commit: 9b7b37e58488ae0a49ebdef124fb2f56cee00c6f https://github.com/Checkmk/checkmk/commit/9b7b37e58488ae0a49ebdef124fb2f56c… Author: Andreas Umbreit &lt;andreas.umbreit(a)checkmk.com&gt; Date: 2023-06-02 (Fri, 02 Jun 2023) Changed paths: M cmk/base/plugins/agent_based/smart.py M tests/unit/cmk/base/plugins/agent_based/test_smart.py Log Message: ----------- smart_stats: Distinguish "UDMA CRC Errors" from "CRC Errors" They come on the same attribute ID, but are labelled differently. Since we distinguish them with two different metrics, we should also apply this nomenclature to the check result. Change-Id: I7c9ec51d2fed9ac63105d71247c2ba1a9eb81eac Commit: a8ce286ef28000c4268f7d6c5a1ced1c6d215d4d https://github.com/Checkmk/checkmk/commit/a8ce286ef28000c4268f7d6c5a1ced1c6… Author: Andreas Umbreit &lt;andreas.umbreit(a)checkmk.com&gt; Date: 2023-06-02 (Fri, 02 Jun 2023) Changed paths: A .werks/15700 Log Message: ----------- 15700 FIX smart_stats: Prevent reporting mismatched temperature value In previous versions, the smart_stats check may have reported wrong temperature values for ATA disks.<br> This lead to false-positive CRIT states in "SMART ... Stats" services. Technical background: The attribute names for ATA disks reported by smartutils turned out to be either not clearly defined (multiple names for the same attribute), or even plainly wrong. To avoid missing some attributes or to even report wrong data, we now look up the names of all actually used attributes by their attribute ID. Change-Id: Iaec5c9d32a71cdbe161ce565c26984beffd10d58 Compare: https://github.com/Checkmk/checkmk/compare/163ee6701e37...a8ce286ef280