summaryrefslogtreecommitdiffstats
path: root/metrics-proxy
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2022-04-30 21:14:38 +0200
committerGitHub <noreply@github.com>2022-04-30 21:14:38 +0200
commita4e669f0634b5555fdc9f57130dd9a1b5ca3afcd (patch)
treec521ade71f06dfbf9bc05db5d0616faaca280bc2 /metrics-proxy
parenta106018d72fad65f64487164bd11e479947e68fa (diff)
parentbb53d37efdf590c0eaa47037d39849b6c31b2f03 (diff)
Merge pull request #22376 from vespa-engine/balder/do-not-log-when-failure-is-expectedv7.580.54
Only log when failure is unexpected.
Diffstat (limited to 'metrics-proxy')
-rw-r--r--metrics-proxy/src/main/java/ai/vespa/metricsproxy/service/RemoteHealthMetricFetcher.java4
1 files changed, 3 insertions, 1 deletions
diff --git a/metrics-proxy/src/main/java/ai/vespa/metricsproxy/service/RemoteHealthMetricFetcher.java b/metrics-proxy/src/main/java/ai/vespa/metricsproxy/service/RemoteHealthMetricFetcher.java
index 43588f063cc..cb8e07b0282 100644
--- a/metrics-proxy/src/main/java/ai/vespa/metricsproxy/service/RemoteHealthMetricFetcher.java
+++ b/metrics-proxy/src/main/java/ai/vespa/metricsproxy/service/RemoteHealthMetricFetcher.java
@@ -34,7 +34,9 @@ public class RemoteHealthMetricFetcher extends HttpMetricFetcher {
try (InputStream stream = getJson()) {
return createHealthMetrics(stream, fetchCount);
} catch (IOException | InterruptedException | ExecutionException e) {
- logMessageNoResponse(errMsgNoResponse(e), fetchCount);
+ if (service.isAlive()) {
+ logMessageNoResponse(errMsgNoResponse(e), fetchCount);
+ }
return HealthMetric.getUnknown("Failed fetching metrics for service: " + service.getMonitoringName());
}
}