diff options
author | Bjørn Christian Seime <bjorn.christian@seime.no> | 2017-10-27 17:16:15 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-10-27 17:16:15 +0200 |
commit | 517466ab20440bd16336cc24e7bbdd5c0360ef95 (patch) | |
tree | 8fae63c28685c0d168edfc674e1caef5f58fb7b5 | |
parent | 85cf841e3f599130e11d6f283e23708c6741cb57 (diff) | |
parent | 69f3af9011a73b8efb4fb5ec6ddbb4bd7678d8ea (diff) |
Merge pull request #3931 from vespa-engine/hakonhall/fix-comments-from-pr-3913
Fix comments from PR 3913
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"); } } |