Merge pull request #13746 from vespa-engine/jonmv/log-log-reader-exceptions

Log any error when reading logs on logserver

1 files changed, 7 insertions, 1 deletions

diff --git a/container-core/src/main/java/com/yahoo/container/handler/LogHandler.java b/container-core/src/main/java/com/yahoo/container/handler/LogHandler.java
index 1d6e1a0893d..f3149ed4998 100644
--- a/container-core/src/main/java/com/yahoo/container/handler/LogHandler.java
+++ b/container-core/src/main/java/com/yahoo/container/handler/LogHandler.java
@@ -11,6 +11,7 @@ import java.io.OutputStream;
 import java.time.Instant;
 import java.util.Optional;
 import java.util.concurrent.Executor;
+import java.util.logging.Level;
 
 public class LogHandler extends ThreadedHttpRequestHandler {
 
@@ -37,7 +38,12 @@ public class LogHandler extends ThreadedHttpRequestHandler {
         return new HttpResponse(200) {
             @Override
             public void render(OutputStream outputStream) {
-                logReader.writeLogs(outputStream, from, to);
+                try {
+                    logReader.writeLogs(outputStream, from, to);
+                }
+                catch (Throwable t) {
+                    log.log(Level.WARNING, "Failed reading logs from " + from + " to " + to, t);
+                }
             }
         };
     }