summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHåkon Hallingstad <hakon@oath.com>2017-10-27 15:52:06 +0200
committerHåkon Hallingstad <hakon@oath.com>2017-10-27 15:52:06 +0200
commit69f3af9011a73b8efb4fb5ec6ddbb4bd7678d8ea (patch)
tree98517ce749d5535c591a946caf1d5eed749a2ede
parente4b69755c3a31a0d4d6539a65a8ac8c828fab42c (diff)
Fix comments from PR 3913
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java6
-rw-r--r--node-repository/src/main/java/com/yahoo/vespa/hosted/provision/testutils/ServiceMonitorStub.java2
2 files changed, 5 insertions, 3 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java
index 2ed03e13f22..1416fb23ece 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MetricsReporter.java
@@ -140,11 +140,13 @@ public class MetricsReporter extends Maintainer {
metric.set(
"numberOfServicesUp",
- servicesCount.getOrDefault(ServiceStatus.UP, 0L), context);
+ servicesCount.getOrDefault(ServiceStatus.UP, 0L),
+ context);
metric.set(
"numberOfServicesNotChecked",
- servicesCount.getOrDefault(ServiceStatus.NOT_CHECKED, 0L), context);
+ servicesCount.getOrDefault(ServiceStatus.NOT_CHECKED, 0L),
+ context);
long numberOfServicesDown = servicesCount.getOrDefault(ServiceStatus.DOWN, 0L);
metric.set("numberOfServicesDown", numberOfServicesDown, context);
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/testutils/ServiceMonitorStub.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/testutils/ServiceMonitorStub.java
index 0c4f937637c..0bb054ae3ae 100644
--- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/testutils/ServiceMonitorStub.java
+++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/testutils/ServiceMonitorStub.java
@@ -92,6 +92,6 @@ public class ServiceMonitorStub implements ServiceMonitor {
@Override
public ServiceModel getServiceModelSnapshot() {
- throw new IllegalStateException("getServicemodelSnapshot has not been implemented");
+ throw new UnsupportedOperationException("getServicemodelSnapshot has not been implemented");
}
}