Skip to content

Commit

Permalink
Merge pull request #2047 from jacderida/rc-2024.08.1
Browse files Browse the repository at this point in the history
fix: save node registry on every upgrade
  • Loading branch information
jacderida authored Aug 16, 2024
2 parents e5d1354 + 6563387 commit 20dc1d2
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion sn_node_manager/src/cmd/node.rs
Original file line number Diff line number Diff line change
Expand Up @@ -553,18 +553,19 @@ pub async fn upgrade(
service_manager.service.service_data.service_name.clone(),
upgrade_result,
));
node_registry.save()?;
}
Err(err) => {
error!("Error upgrading service {service_name}: {err}");
upgrade_summary.push((
node.service_name.clone(),
UpgradeResult::Error(format!("Error: {err}")),
));
node_registry.save()?;
}
}
}

node_registry.save()?;
print_upgrade_summary(upgrade_summary.clone());

if upgrade_summary.iter().any(|(_, r)| {
Expand Down

0 comments on commit 20dc1d2

Please sign in to comment.