jason/socktop
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: refresh component values to collect NVMe temperatures

Browse Source
This commit is contained in:
jasonwitty 2025-10-06 11:15:36 -07:00
parent bae2ecb79a
commit 47e96c7d92
1 changed files with 5 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
socktop_agent/src/metrics.rs
View File

@ -336,7 +336,7 @@ pub async fn collect_disks(state: &AppState) -> Vec<DiskInfo> {
// NVMe temps show up as "Composite" under different chip names // NVMe temps show up as "Composite" under different chip names
let disk_temps = { let disk_temps = {
let mut components = state.components.lock().await; let mut components = state.components.lock().await;
components.refresh(false); components.refresh(true); // true = refresh values, not just the list
let mut composite_temps = Vec::new(); let mut composite_temps = Vec::new();
for c in components.iter() { for c in components.iter() {
@ -398,7 +398,7 @@ pub async fn collect_disks(state: &AppState) -> Vec<DiskInfo> {
None None
} }
}); });
if temperature.is_none() && !name.starts_with("loop") && !name.starts_with("ram") { if temperature.is_none() && !name.starts_with("loop") && !name.starts_with("ram") {
tracing::debug!("No temperature found for disk: {}", name); tracing::debug!("No temperature found for disk: {}", name);
} }
@ -449,11 +449,9 @@ pub async fn collect_disks(state: &AppState) -> Vec<DiskInfo> {
}); });
// Aggregate partition stats into parent // Aggregate partition stats into parent
let entry = parent_disks.entry(parent_name.to_string()).or_insert(( let entry = parent_disks
0, .entry(parent_name.to_string())
0, .or_insert((0, 0, parent_temp));
parent_temp,
));
entry.0 += partition.total; entry.0 += partition.total;
entry.1 += partition.available; entry.1 += partition.available;
// Keep temperature if any partition has it (or if we just found one) // Keep temperature if any partition has it (or if we just found one)