diff options
author | Alex Auvolat <alex@adnab.me> | 2021-04-05 20:33:24 +0200 |
---|---|---|
committer | Alex Auvolat <alex@adnab.me> | 2021-04-05 20:33:24 +0200 |
commit | f11bd80d2a0b8ba0c108deab1293daa31109ce5f (patch) | |
tree | 1feb09f40f0c61201ea2a7c6570bef817c40f0ee /src/rpc/membership.rs | |
parent | 595dc0ed0d6994cbe0f628b41df85f44c576dc64 (diff) | |
download | garage-f11bd80d2a0b8ba0c108deab1293daa31109ce5f.tar.gz garage-f11bd80d2a0b8ba0c108deab1293daa31109ce5f.zip |
Keep old data
Diffstat (limited to 'src/rpc/membership.rs')
-rw-r--r-- | src/rpc/membership.rs | 24 |
1 files changed, 20 insertions, 4 deletions
diff --git a/src/rpc/membership.rs b/src/rpc/membership.rs index 330c154f..f8223420 100644 --- a/src/rpc/membership.rs +++ b/src/rpc/membership.rs @@ -582,7 +582,7 @@ impl System { .map(|ip| (*ip, None)) .collect::<Vec<_>>(); - match self.persist_status.load() { + match self.persist_status.load_async().await { Ok(peers) => { bp2.extend(peers.iter().map(|x| (x.addr, Some(x.id)))); } @@ -653,9 +653,25 @@ impl System { async fn update_status(self: &Arc<Self>, updaters: &Updaters, status: Status) { if status.hash != self.status.borrow().hash { info!("Persisting new peer list"); - let serializable_status = status.to_serializable_membership(&self); - self.persist_status.save_async(&serializable_status).await - .expect("Unable to persist peer list"); + + let mut list = status.to_serializable_membership(&self); + + // Combine with old peer list to make sure no peer is lost + match self.persist_status.load_async().await { + Ok(old_list) => { + for pp in old_list { + if !list.iter().any(|np| pp.id == np.id) { + list.push(pp); + } + } + } + _ => (), + } + + if list.len() > 0 { + self.persist_status.save_async(&list).await + .expect("Unable to persist peer list"); + } } let status = Arc::new(status); |