@daemonitor/plugins 1.1.21 → 1.1.23

package/dist/plugins/monitoring/DockerPlugin.js

@@ -128,6 +128,38 @@ async function collectContainers(bin) {
     catch (err) {
         console.error("docker: inspect collection failed:", err.message);
     }
+    // 4. For UNHEALTHY containers only, read the healthcheck's own last output.
+    //
+    // "1/13 containers unhealthy" says a problem exists and nothing about what it
+    // is. Docker already stores the failing command's output in
+    // .State.Health.Log[] — that is the actual error text, and without it an alert
+    // sends someone to ssh into the box to run the same healthcheck by hand.
+    //
+    // Read one container at a time and only for the unhealthy ones. The batched
+    // inspect above avoids .State.Health because the template errors for
+    // containers that have no healthcheck; a container reporting "(unhealthy)"
+    // necessarily has one, so this is safe, and it is a handful of calls at most
+    // precisely when something is wrong.
+    const unhealthy = containers.filter((c) => c.health === "unhealthy").slice(0, 5);
+    for (const c of unhealthy) {
+        try {
+            const { stdout } = await execAsync(`${bin} inspect --format '{{json .State.Health}}' ${c.id}`, EXEC_OPTS);
+            const h = JSON.parse(stdout.trim() || "null");
+            if (!h)
+                continue;
+            c.healthFailingStreak = Number(h.FailingStreak) || 0;
+            const last = Array.isArray(h.Log) && h.Log.length ? h.Log[h.Log.length - 1] : null;
+            if (last?.Output) {
+                // Healthcheck output is often multi-line and ends in a newline; collapse
+                // it so it fits an alert line, and keep the head where the error is.
+                c.healthOutput = String(last.Output).replace(/\s+/g, " ").trim().slice(0, 300);
+            }
+        }
+        catch (err) {
+            // Best-effort: an alert naming the container is still better than none.
+            console.error(`docker: health log read failed for ${c.name}:`, err.message);
+        }
+    }
     return containers;
 }
 export function createDockerPlugin() {

package/package.json

@@ -1,15 +1,10 @@
 {
   "name": "@daemonitor/plugins",
-  "version": "1.1.21",
+  "version": "1.1.23",
   "description": "daemonitor plugins",
   "types": "types/index.d.ts",
   "main": "dist/index.js",
   "type": "module",
-  "scripts": {
-    "build": "tsc",
-    "test": "echo \"Error: no test specified\" && exit 1",
-    "prepublishOnly": "npm run build"
-  },
   "files": [
     "dist",
     "types"
@@ -53,5 +48,9 @@
   "bugs": {
     "url": "https://github.com/daemonitor/plugins/issues"
   },
-  "homepage": "https://github.com/daemonitor/plugins#readme"
-}
+  "homepage": "https://github.com/daemonitor/plugins#readme",
+  "scripts": {
+    "build": "tsc",
+    "test": "echo \"Error: no test specified\" && exit 1"
+  }
+}