fix(smart v0.1.17): smart_status optionnel + AmbientCapabilities CAP_SYS_ADMIN

- SmartJson.smart_status devient Option<SmartStatus> avec #[serde(default)]
  → parsing non-bloquant si le champ est absent (ex: NVME_IOCTL_ADMIN_CMD échoue)
- Service: suppression NoNewPrivileges, ajout AmbientCapabilities=CAP_SYS_ADMIN
  → smartctl hérite la capability via execve (kernel ≥ 5.2)
- Nettoyage logs debug (suppression dump JSON brut)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

agent/Cargo.lock

@@ -248,7 +248,7 @@ dependencies = [
 
 [[package]]
 name = "nanometrics-agent"
-version = "0.1.12"
+version = "0.1.17"
 dependencies = [
  "libc",
  "rumqttc",

agent/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "nanometrics-agent"
-version = "0.1.12"
+version = "0.1.17"
 edition = "2021"
 
 [lib]

agent/src/metrics/smart.rs

@@ -2,17 +2,24 @@ use serde::Deserialize;
 
 #[derive(Deserialize)]
 struct SmartJson {
-    smart_status: SmartStatus,
+    #[serde(default)]
+    smart_status: Option<SmartStatus>,
     temperature: Option<SmartTemp>,
     ata_smart_attributes: Option<SmartAttrs>,
     nvme_smart_health_information_log: Option<NvmeHealth>,
 }
 
 #[derive(Deserialize)]
-struct SmartStatus { passed: bool }
+struct SmartStatus {
+    #[serde(default)]
+    passed: bool,
+}
 
 #[derive(Deserialize)]
-struct SmartTemp { current: i64 }
+struct SmartTemp {
+    #[serde(default)]
+    current: Option<i64>,
+}
 
 #[derive(Deserialize)]
 struct SmartAttrs { table: Vec<SmartAttr> }
@@ -44,7 +51,7 @@ pub fn is_available() -> bool {
 pub fn parse_json(json: &str) -> Result<crate::payload::SmartMetrics, serde_json::Error> {
     let s: SmartJson = serde_json::from_str(json)?;
 
-    let temperature = s.temperature.as_ref().map(|t| t.current)
+    let temperature = s.temperature.as_ref().and_then(|t| t.current)
         .or_else(|| s.nvme_smart_health_information_log.as_ref()?.temperature);
 
     let mut reallocated = None;
@@ -72,7 +79,7 @@ pub fn parse_json(json: &str) -> Result<crate::payload::SmartMetrics, serde_json
 
     Ok(crate::payload::SmartMetrics {
         device: String::new(),
-        passed: s.smart_status.passed,
+        passed: s.smart_status.as_ref().map(|s| s.passed).unwrap_or(false),
         temperature,
         reallocated_sectors: reallocated,
         power_on_hours: power_hours,
@@ -85,23 +92,28 @@ pub fn collect() -> Option<Vec<crate::payload::SmartMetrics>> {
         eprintln!("[smart] smartctl introuvable dans PATH");
         return None;
     }
-    let mut devs: Vec<String> = std::fs::read_dir("/sys/block")
+    let mut set = std::collections::HashSet::new();
-        .into_iter()
+
-        .flatten()
+    // SATA/SAS : /sys/block/sd* → /dev/sda, /dev/sdb…
-        .flatten()
+    for e in std::fs::read_dir("/sys/block").into_iter().flatten().flatten() {
-        .map(|e| e.file_name().into_string().unwrap_or_default())
+        let n = e.file_name().into_string().unwrap_or_default();
-        .filter_map(|n| {
+        if n.starts_with("sd") {
-            if n.starts_with("sd") {
+            set.insert(format!("/dev/{}", n));
-                Some(format!("/dev/{}", n))
+        }
-            } else if n.starts_with("nvme") && n.contains('n') {
+    }
-                Some(format!("/dev/{}", n))
+
-            } else {
+    // NVMe : /sys/class/nvme/nvme* → /dev/nvme0, /dev/nvme1…
-                None
+    // On utilise le contrôleur (char device), pas le namespace (block device),
-            }
+    // car smartctl ne peut exécuter les commandes admin SMART que via le contrôleur.
-        })
+    // La règle udev 99-nanometrics-smart.rules lui donne l'accès groupe disk.
-        .collect::<std::collections::HashSet<_>>()
+    for e in std::fs::read_dir("/sys/class/nvme").into_iter().flatten().flatten() {
-        .into_iter()
+        let n = e.file_name().into_string().unwrap_or_default();
-        .collect();
+        if n.starts_with("nvme") {
+            set.insert(format!("/dev/{}", n));
+        }
+    }
+
+    let mut devs: Vec<String> = set.into_iter().collect();
     devs.sort();
     eprintln!("[smart] disques détectés: {:?}", devs);
 
@@ -114,12 +126,9 @@ pub fn collect() -> Option<Vec<crate::payload::SmartMetrics>> {
             Ok(o) => o,
             Err(e) => { eprintln!("[smart] erreur exec smartctl {}: {}", dev, e); continue }
         };
-        eprintln!("[smart] smartctl {} → exit={}, stdout={} octets, stderr={} octets",
-            dev, output.status, output.stdout.len(), output.stderr.len());
         let json = String::from_utf8_lossy(&output.stdout);
         match parse_json(&json) {
             Ok(metrics) => {
-                eprintln!("[smart] {} parsé OK (passed={})", dev, metrics.passed);
                 results.push(crate::payload::SmartMetrics {
                     device: dev.trim_start_matches("/dev/").to_string(),
                     ..metrics
@@ -127,7 +136,6 @@ pub fn collect() -> Option<Vec<crate::payload::SmartMetrics>> {
             }
             Err(e) => {
                 eprintln!("[smart] {} parse JSON échoué: {}", dev, e);
-                eprintln!("[smart] premiers 200 octets stdout: {:?}", &json.chars().take(200).collect::<String>());
             }
         }
     }

deploy/99-nanometrics-smart.rules

@@ -0,0 +1,4 @@
+# Nanometrics: accès groupe disk aux contrôleurs NVMe pour SMART
+# Sans cette règle, /dev/nvme0 est crw------- root root (root only),
+# ce qui empêche smartctl d'exécuter les commandes admin et omet smart_status du JSON.
+KERNEL=="nvme[0-9]*", SUBSYSTEM=="nvme", GROUP="disk", MODE="0660"

deploy/install.sh

@@ -45,6 +45,17 @@ else
 fi
 echo ""
 
+# ── 2. Règle udev NVMe (accès SMART pour le groupe disk) ──────────────────────
+UDEV_RULE="/etc/udev/rules.d/99-nanometrics-smart.rules"
+cat > "$UDEV_RULE" << 'UDEVRULE'
+# Nanometrics: accès groupe disk aux contrôleurs NVMe pour SMART
+KERNEL=="nvme[0-9]*", SUBSYSTEM=="nvme", GROUP="disk", MODE="0660"
+UDEVRULE
+udevadm control --reload-rules
+udevadm trigger --subsystem-match=nvme 2>/dev/null || true
+ok "Règle udev NVMe installée ($UDEV_RULE)"
+echo ""
+
 # ── 3. Détection de l'architecture ────────────────────────────────────────────
 ARCH="$(uname -m)"
 case "$ARCH" in

deploy/nanometrics-agent.service

@@ -17,7 +17,12 @@ ConfigurationDirectoryMode=0755
 ProtectSystem=strict
 ProtectHome=read-only
 PrivateTmp=yes
-NoNewPrivileges=yes
+# CAP_SYS_ADMIN est requis par le noyau pour NVME_IOCTL_ADMIN_CMD (lecture SMART NVMe).
+# NoNewPrivileges est retiré car il efface les ambient capabilities sur exec (noyau ≥ 5.2),
+# ce qui empêcherait smartctl enfant d'hériter la capability.
+# CapabilityBoundingSet borne à la seule cap nécessaire.
+CapabilityBoundingSet=CAP_SYS_ADMIN
+AmbientCapabilities=CAP_SYS_ADMIN
 
 RestrictAddressFamilies=AF_INET AF_INET6 AF_UNIX