bahman/scylla
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

topology_coordinator: adjust reason string in left_token_ring handler

Browse Source 
We were using the "finished decommission node" reason string for a
failed bootstrap and replace.
This commit is contained in:
Patryk Jędrzejczak
2024-01-26 11:16:13 +01:00
parent b0eef50b2e
commit 9b2d1a20a3
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
service/topology_coordinator.cc
View File

@@ -1629,7 +1629,9 @@ class topology_coordinator : public endpoint_lifecycle_subscriber {
builder.del_transition_state()
.with_node(node.id)
.set("node_state", node_state::left);
auto str = ::format("finished decommissioning node {}", node.id);
auto str = node.rs->state == node_state::decommissioning
? ::format("finished decommissioning node {}", node.id)
: ::format("finished rollback of {} after {} failure", node.id, node.rs->state);
co_await update_topology_state(take_guard(std::move(node)), {builder.build()}, std::move(str));
}
break;