diff options
author | Tor Brede Vekterli <vekterli@verizonmedia.com> | 2021-03-02 13:14:32 +0000 |
---|---|---|
committer | Tor Brede Vekterli <vekterli@verizonmedia.com> | 2021-03-02 13:14:32 +0000 |
commit | de374ebb0cff7c612bdb41c9b7c1b3f31d46f03f (patch) | |
tree | 6a0820b5b395ad6d9dcec32dea4a0bd941414ec7 /storage | |
parent | 616be2724603dfac17d45db8eccd52369988a7d9 (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.cpp | 12 |
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); |