diff options
author | Alex Auvolat <alex@adnab.me> | 2022-12-12 17:16:49 +0100 |
---|---|---|
committer | Alex Auvolat <alex@adnab.me> | 2022-12-12 17:17:05 +0100 |
commit | de9d6cddf709e686ada3d1e71de7b31d7704b8b5 (patch) | |
tree | ce97129877b6d0b1839f22e42fa3d8e5fb9f07c7 /src/block/resync.rs | |
parent | f7c65e830e66c9887d989a8281d8fae7f76f9c8c (diff) | |
download | garage-de9d6cddf709e686ada3d1e71de7b31d7704b8b5.tar.gz garage-de9d6cddf709e686ada3d1e71de7b31d7704b8b5.zip |
Prettier worker list table; remove useless CLI log messages
Diffstat (limited to 'src/block/resync.rs')
-rw-r--r-- | src/block/resync.rs | 25 |
1 files changed, 10 insertions, 15 deletions
diff --git a/src/block/resync.rs b/src/block/resync.rs index ada3ac54..875ead9b 100644 --- a/src/block/resync.rs +++ b/src/block/resync.rs @@ -477,27 +477,22 @@ impl Worker for ResyncWorker { format!("Block resync worker #{}", self.index + 1) } - fn info(&self) -> Option<String> { + fn status(&self) -> WorkerStatus { let persisted = self.manager.resync.persisted.load(); if self.index >= persisted.n_workers { - return Some("(unused)".into()); + return WorkerStatus { + freeform: vec!["(unused)".into()], + ..Default::default() + }; } - let mut ret = vec![]; - ret.push(format!("tranquility = {}", persisted.tranquility)); - - let qlen = self.manager.resync.queue_len().unwrap_or(0); - if qlen > 0 { - ret.push(format!("{} blocks in queue", qlen)); - } - - let elen = self.manager.resync.errors_len().unwrap_or(0); - if elen > 0 { - ret.push(format!("{} blocks in error state", elen)); + WorkerStatus { + queue_length: Some(self.manager.resync.queue_len().unwrap_or(0) as u64), + tranquility: Some(persisted.tranquility), + persistent_errors: Some(self.manager.resync.errors_len().unwrap_or(0) as u64), + ..Default::default() } - - Some(ret.join(", ")) } async fn work(&mut self, _must_exit: &mut watch::Receiver<bool>) -> Result<WorkerState, Error> { |