summaryrefslogtreecommitdiffstats
path: root/storage
diff options
context:
space:
mode:
authorTor Brede Vekterli <vekterli@verizonmedia.com>2021-03-02 13:14:32 +0000
committerTor Brede Vekterli <vekterli@verizonmedia.com>2021-03-02 13:14:32 +0000
commitde374ebb0cff7c612bdb41c9b7c1b3f31d46f03f (patch)
tree6a0820b5b395ad6d9dcec32dea4a0bd941414ec7 /storage
parent616be2724603dfac17d45db8eccd52369988a7d9 (diff)
Remove notion of node-specific reliability from C++ distribution code
I have never seen this in use anywhere, and can find no code that ever sets it. Bonus is that node candidate trimming can be vastly simplified.
Diffstat (limited to 'storage')
-rw-r--r--storage/src/vespa/storage/storageserver/servicelayernode.cpp12
1 files changed, 2 insertions, 10 deletions
diff --git a/storage/src/vespa/storage/storageserver/servicelayernode.cpp b/storage/src/vespa/storage/storageserver/servicelayernode.cpp
index c3cb1d8d6d6..15ff731d328 100644
--- a/storage/src/vespa/storage/storageserver/servicelayernode.cpp
+++ b/storage/src/vespa/storage/storageserver/servicelayernode.cpp
@@ -87,14 +87,12 @@ void
ServiceLayerNode::initializeNodeSpecific()
{
// Give node state to mount point initialization, such that we can
- // get capacity and reliability set in reported node state.
+ // get capacity set in reported node state.
NodeStateUpdater::Lock::SP lock(_component->getStateUpdater().grabStateChangeLock());
lib::NodeState ns(*_component->getStateUpdater().getReportedNodeState());
ns.setCapacity(_serverConfig->nodeCapacity);
- ns.setReliability(_serverConfig->nodeReliability);
- LOG(debug, "Adjusting reported node state to include capacity and reliability: %s",
- ns.toString().c_str());
+ LOG(debug, "Adjusting reported node state to include capacity: %s", ns.toString().c_str());
_component->getStateUpdater().setReportedNodeState(ns);
}
@@ -118,12 +116,6 @@ ServiceLayerNode::handleLiveConfigUpdate(const InitialGuard & initGuard)
ASSIGN(nodeCapacity);
ns.setCapacity(newC.nodeCapacity);
}
- if (DIFFER(nodeReliability)) {
- LOG(info, "Live config update: Node reliability changed from %u to %u.",
- oldC.nodeReliability, newC.nodeReliability);
- ASSIGN(nodeReliability);
- ns.setReliability(newC.nodeReliability);
- }
if (updated) {
_serverConfig.reset(new vespa::config::content::core::StorServerConfig(oldC));
_component->getStateUpdater().setReportedNodeState(ns);