Parse Docker image sizes to MB and sort services alphabetically

Cargo.lock

@@ -279,7 +279,7 @@ checksum = "a1d728cc89cf3aee9ff92b05e62b19ee65a02b5702cff7d5a377e32c6ae29d8d"
 
 [[package]]
 name = "cm-dashboard"
-version = "0.1.182"
+version = "0.1.184"
 dependencies = [
  "anyhow",
  "chrono",
@@ -301,7 +301,7 @@ dependencies = [
 
 [[package]]
 name = "cm-dashboard-agent"
-version = "0.1.182"
+version = "0.1.184"
 dependencies = [
  "anyhow",
  "async-trait",
@@ -324,7 +324,7 @@ dependencies = [
 
 [[package]]
 name = "cm-dashboard-shared"
-version = "0.1.182"
+version = "0.1.184"
 dependencies = [
  "chrono",
  "serde",

agent/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard-agent"
-version = "0.1.183"
+version = "0.1.185"
 edition = "2021"
 
 [dependencies]

agent/src/collectors/disk.rs

@@ -413,9 +413,7 @@ impl DiskCollector {
 
     /// Get SMART data for a single drive
     async fn get_smart_data(&self, drive_name: &str) -> Result<SmartData, CollectorError> {
-        use tracing::info;
-
-        // Use direct smartctl (no sudo) - service has CAP_SYS_RAWIO capability
+        // Use direct smartctl (no sudo) - service has CAP_SYS_RAWIO and CAP_SYS_ADMIN capabilities
         // For NVMe drives, specify device type explicitly
         let mut cmd = Command::new("smartctl");
         if drive_name.starts_with("nvme") {
@@ -433,10 +431,6 @@ impl DiskCollector {
         let output_str = String::from_utf8_lossy(&output.stdout);
 
         if !output.status.success() {
-            if drive_name.starts_with("nvme") {
-                info!("NVMe SMART failed for {}: status={}, stderr={}",
-                    drive_name, output.status, String::from_utf8_lossy(&output.stderr));
-            }
             // Return unknown data rather than failing completely
             return Ok(SmartData {
                 health: "UNKNOWN".to_string(),
@@ -521,19 +515,12 @@ impl DiskCollector {
             }
         }
 
-        let smart_data = SmartData {
-            health: health.clone(),
-            serial_number: serial_number.clone(),
+        Ok(SmartData {
+            health,
+            serial_number,
             temperature_celsius: temperature,
             wear_percent,
-        };
-
-        if drive_name.starts_with("nvme") {
-            info!("NVMe SMART parsed for {}: serial={:?}, temp={:?}, health={}, wear={:?}",
-                drive_name, serial_number, temperature, health, wear_percent);
-        }
-
-        Ok(smart_data)
+        })
     }
 
     /// Populate drives data into AgentData

agent/src/collectors/systemd.rs

@@ -133,16 +133,16 @@ impl SystemdCollector {
 
                         // Add Docker images
                         let docker_images = self.get_docker_images();
-                        for (image_name, image_status, image_size) in docker_images {
+                        for (image_name, image_status, image_size_str, image_size_mb) in docker_images {
                             let mut metrics = Vec::new();
                             metrics.push(SubServiceMetric {
                                 label: "size".to_string(),
-                                value: 0.0, // Size as string in name instead
-                                unit: None,
+                                value: image_size_mb,
+                                unit: Some("MB".to_string()),
                             });
 
                             sub_services.push(SubServiceData {
-                                name: format!("{} ({})", image_name, image_size),
+                                name: format!("{} ({})", image_name, image_size_str),
                                 service_status: self.calculate_service_status(&image_name, &image_status),
                                 metrics,
                             });
@@ -169,6 +169,10 @@ impl SystemdCollector {
             }
         }
 
+        // Sort services alphabetically by name
+        agent_data.services.sort_by(|a, b| a.name.cmp(&b.name));
+        complete_service_data.sort_by(|a, b| a.name.cmp(&b.name));
+
         // Update cached state
         {
             let mut state = self.state.write().unwrap();
@@ -815,7 +819,7 @@ impl SystemdCollector {
     }
 
     /// Get docker images as sub-services
-    fn get_docker_images(&self) -> Vec<(String, String, String)> {
+    fn get_docker_images(&self) -> Vec<(String, String, String, f32)> {
         let mut images = Vec::new();
         // Check if docker is available (cm-agent user is in docker group)
         let output = Command::new("docker")
@@ -845,23 +849,55 @@ impl SystemdCollector {
             let parts: Vec<&str> = line.split(',').collect();
             if parts.len() >= 2 {
                 let image_name = parts[0].trim();
-                let size = parts[1].trim();
+                let size_str = parts[1].trim();
 
                 // Skip <none>:<none> images (dangling images)
                 if image_name.contains("<none>") {
                     continue;
                 }
 
+                // Parse size to MB (sizes come as "142MB", "1.5GB", "512kB", etc.)
+                let size_mb = self.parse_docker_size(size_str);
+
                 images.push((
                     format!("image_{}", image_name),
                     "active".to_string(), // Images are always "active" (present)
-                    size.to_string()
+                    size_str.to_string(),
+                    size_mb
                 ));
             }
         }
 
         images
     }
+
+    /// Parse Docker size string to MB
+    fn parse_docker_size(&self, size_str: &str) -> f32 {
+        let size_upper = size_str.to_uppercase();
+
+        // Extract numeric part and unit
+        let mut num_str = String::new();
+        let mut unit = String::new();
+
+        for ch in size_upper.chars() {
+            if ch.is_ascii_digit() || ch == '.' {
+                num_str.push(ch);
+            } else if ch.is_alphabetic() {
+                unit.push(ch);
+            }
+        }
+
+        let value: f32 = num_str.parse().unwrap_or(0.0);
+
+        // Convert to MB
+        match unit.as_str() {
+            "KB" | "K" => value / 1024.0,
+            "MB" | "M" => value,
+            "GB" | "G" => value * 1024.0,
+            "TB" | "T" => value * 1024.0 * 1024.0,
+            _ => value, // Assume bytes if no unit
+        }
+    }
 }
 
 #[async_trait]

dashboard/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard"
-version = "0.1.183"
+version = "0.1.185"
 edition = "2021"
 
 [dependencies]

shared/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "cm-dashboard-shared"
-version = "0.1.183"
+version = "0.1.185"
 edition = "2021"
 
 [dependencies]