@glassmkr/crucible 0.9.0 → 0.9.1

package/dist/lib/ipmi-sensor-filter.js

@@ -0,0 +1,74 @@
+// Per-socket pre-filter for IPMI CPU thermal sensors.
+//
+// Background: Gigabyte BMC firmware (observed on H262-Z63 and MC12-LE0
+// with firmware 12.61) reports a `CPU<N>_DTS` sensor that runs ~30°C
+// hotter than the actual AMD die temperature read directly via the
+// kernel's k10temp driver. The same boards also expose a much closer
+// `CPU<N>_TEMP` sensor on each socket. The IPMI fallback path of
+// Forge's `cpu_temperature_high` evaluator picks the maximum across
+// all CPU thermal sensors, which over-fires when both are present.
+//
+// Fix: when the IPMI sensor list contains both `CPU<N>_TEMP` and
+// `CPU<N>_DTS` for the same socket, drop the DTS variant. If only DTS
+// exists for a socket (rare, but possible on firmware that doesn't
+// expose `*_TEMP`), keep it as the only available CPU thermal reading.
+//
+// Tracked: glassmkr/crucible#2. Closed in 0.9.1.
+//
+// This filter only inspects names matching `^CPU\d+_DTS$` (Gigabyte's
+// specific convention). Non-CPU sensors and DTS-shaped sensors that
+// don't fit the per-socket prefix pattern are passed through unchanged
+// so we don't accidentally hide a useful reading on a vendor we
+// haven't characterised.
+const DTS_RE = /^CPU(\d+)_DTS$/i;
+function tempSensorPatternsForSocket(socket) {
+    // Match the two observed temperature naming conventions on the same
+    // socket: `CPU0_TEMP` (Gigabyte) and `CPU0 Temp` (Supermicro-style
+    // with a space). Either form being present means we have a non-DTS
+    // reading for this socket and can safely drop the DTS variant.
+    const escaped = socket.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
+    return [
+        new RegExp(`^CPU${escaped}_TEMP$`, "i"),
+        new RegExp(`^CPU${escaped}\\s+Temp$`, "i"),
+    ];
+}
+/**
+ * Drop `CPU<N>_DTS` sensors whose socket has a sibling `CPU<N>_TEMP`
+ * (or `CPU<N> Temp`) sensor in the same list. Returns a new array.
+ *
+ * Stable order: kept sensors retain their original input order. Pure
+ * function; safe to call multiple times.
+ */
+export function filterRedundantCpuDtsSensors(sensors) {
+    const sockets = new Set();
+    const dtsBySocket = new Map();
+    for (const s of sensors) {
+        const m = DTS_RE.exec(s.name);
+        if (!m)
+            continue;
+        sockets.add(m[1]);
+        const list = dtsBySocket.get(m[1]) ?? [];
+        list.push(s);
+        dtsBySocket.set(m[1], list);
+    }
+    if (sockets.size === 0)
+        return sensors.slice();
+    // For each socket with a DTS sensor, check whether any non-DTS CPU
+    // thermal sensor exists for the same socket.
+    const dropDts = new Set();
+    for (const socket of sockets) {
+        const patterns = tempSensorPatternsForSocket(socket);
+        const hasTemp = sensors.some((s) => patterns.some((re) => re.test(s.name)));
+        if (hasTemp)
+            dropDts.add(socket);
+    }
+    if (dropDts.size === 0)
+        return sensors.slice();
+    return sensors.filter((s) => {
+        const m = DTS_RE.exec(s.name);
+        if (!m)
+            return true;
+        return !dropDts.has(m[1]);
+    });
+}
+//# sourceMappingURL=ipmi-sensor-filter.js.map

package/dist/lib/ipmi-sensor-filter.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ipmi-sensor-filter.js","sourceRoot":"","sources":["../../src/lib/ipmi-sensor-filter.ts"],"names":[],"mappings":"AAAA,sDAAsD;AACtD,EAAE;AACF,uEAAuE;AACvE,qEAAqE;AACrE,mEAAmE;AACnE,qEAAqE;AACrE,iEAAiE;AACjE,oEAAoE;AACpE,mEAAmE;AACnE,EAAE;AACF,iEAAiE;AACjE,sEAAsE;AACtE,mEAAmE;AACnE,uEAAuE;AACvE,EAAE;AACF,iDAAiD;AACjD,EAAE;AACF,sEAAsE;AACtE,oEAAoE;AACpE,uEAAuE;AACvE,gEAAgE;AAChE,yBAAyB;AAMzB,MAAM,MAAM,GAAG,iBAAiB,CAAC;AAEjC,SAAS,2BAA2B,CAAC,MAAc;IACjD,oEAAoE;IACpE,mEAAmE;IACnE,mEAAmE;IACnE,+DAA+D;IAC/D,MAAM,OAAO,GAAG,MAAM,CAAC,OAAO,CAAC,qBAAqB,EAAE,MAAM,CAAC,CAAC;IAC9D,OAAO;QACL,IAAI,MAAM,CAAC,OAAO,OAAO,QAAQ,EAAE,GAAG,CAAC;QACvC,IAAI,MAAM,CAAC,OAAO,OAAO,WAAW,EAAE,GAAG,CAAC;KAC3C,CAAC;AACJ,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,4BAA4B,CAAuB,OAAY;IAC7E,MAAM,OAAO,GAAG,IAAI,GAAG,EAAU,CAAC;IAClC,MAAM,WAAW,GAAG,IAAI,GAAG,EAAe,CAAC;IAC3C,KAAK,MAAM,CAAC,IAAI,OAAO,EAAE,CAAC;QACxB,MAAM,CAAC,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAC9B,IAAI,CAAC,CAAC;YAAE,SAAS;QACjB,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAClB,MAAM,IAAI,GAAG,WAAW,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QACzC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QACb,WAAW,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,IAAI,CAAC,CAAC;IAC9B,CAAC;IACD,IAAI,OAAO,CAAC,IAAI,KAAK,CAAC;QAAE,OAAO,OAAO,CAAC,KAAK,EAAE,CAAC;IAE/C,mEAAmE;IACnE,6CAA6C;IAC7C,MAAM,OAAO,GAAG,IAAI,GAAG,EAAU,CAAC;IAClC,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,MAAM,QAAQ,GAAG,2BAA2B,CAAC,MAAM,CAAC,CAAC;QACrD,MAAM,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAC5E,IAAI,OAAO;YAAE,OAAO,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC;IACnC,CAAC;IACD,IAAI,OAAO,CAAC,IAAI,KAAK,CAAC;QAAE,OAAO,OAAO,CAAC,KAAK,EAAE,CAAC;IAE/C,OAAO,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE;QAC1B,MAAM,CAAC,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;QAC9B,IAAI,CAAC,CAAC;YAAE,OAAO,IAAI,CAAC;QACpB,OAAO,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;IAC5B,CAAC,CAAC,CAAC;AACL,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@glassmkr/crucible",
-  "version": "0.9.0",
+  "version": "0.9.1",
   "description": "Lightweight bare metal server monitoring. IPMI, SMART, OS, network. Opinionated alerts.",
   "type": "module",
   "main": "./dist/index.js",