summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJon Bratseth <bratseth@oath.com>2018-10-02 14:35:04 +0200
committerGitHub <noreply@github.com>2018-10-02 14:35:04 +0200
commit0439976541823283f86dfaf4621a1a70deb2ce31 (patch)
tree42dd7de039ca7ab5ac1a619ee63831f4cff0c926
parentb234a0daf5076d4ece4b373240883073d2b932ee (diff)
parent318853c6ad0821b7d456f298c7cd2ba0a0590cd1 (diff)
Merge pull request #7175 from vespa-engine/bratseth/status-is-known-on-failure
Status is known when failing
-rw-r--r--container-search/src/main/java/com/yahoo/prelude/cluster/NodeMonitor.java4
1 files changed, 3 insertions, 1 deletions
diff --git a/container-search/src/main/java/com/yahoo/prelude/cluster/NodeMonitor.java b/container-search/src/main/java/com/yahoo/prelude/cluster/NodeMonitor.java
index 298eda9b201..894b9ebd8ce 100644
--- a/container-search/src/main/java/com/yahoo/prelude/cluster/NodeMonitor.java
+++ b/container-search/src/main/java/com/yahoo/prelude/cluster/NodeMonitor.java
@@ -65,6 +65,7 @@ public class NodeMonitor {
*/
public void failed(ErrorMessage error) {
long respondedAt = System.currentTimeMillis();
+ statusIsKnown = true;
if (error.getCode() == NO_ANSWER_WHEN_PINGING_NODE.code) {
// Only count not being able to talk to backend at all
@@ -86,10 +87,11 @@ public class NodeMonitor {
*/
public void responded(boolean searchNodesOnline) {
succeededAt = System.currentTimeMillis();
+ statusIsKnown = true;
+
this.searchNodesOnline = searchNodesOnline;
if (! isWorking)
setWorking(true, "Responds correctly");
- statusIsKnown = true;
}
/** Changes the state of this node if required */