diff options
author | Harald Musum <musum@yahooinc.com> | 2022-04-29 15:54:40 +0200 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2022-04-29 15:54:40 +0200 |
commit | 46e7689040c0986654e6bd495b726f010aa71bc1 (patch) | |
tree | 15f68c9cc32d21008c0d945d56bb2ab498c3446a /node-repository/src | |
parent | 4b98768ff0b2772631e9bcce55568f3171b2015d (diff) |
Use log level INFO for exceptions getting application lock
Getting lock will sometimes fail due to other maintainers holding the
lock, no need to log a warning.
Diffstat (limited to 'node-repository/src')
2 files changed, 6 insertions, 2 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MaintenanceDeployment.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MaintenanceDeployment.java index 511d3efe313..b2c3859e33a 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MaintenanceDeployment.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/MaintenanceDeployment.java @@ -110,7 +110,7 @@ class MaintenanceDeployment implements Closeable { return Optional.of(nodeRepository.nodes().lock(application, timeout)); } catch (ApplicationLockException e) { - log.log(Level.WARNING, () -> "Could not lock " + application + " for maintenance deployment within " + timeout); + log.log(Level.INFO, () -> "Could not lock " + application + " for maintenance deployment within " + timeout); return Optional.empty(); } } diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiHandler.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiHandler.java index f05492162db..c5d8b2518e5 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiHandler.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/restapi/NodesV2ApiHandler.java @@ -3,6 +3,7 @@ package com.yahoo.vespa.hosted.provision.restapi; import com.yahoo.component.Version; import com.yahoo.config.provision.ApplicationId; +import com.yahoo.config.provision.ApplicationLockException; import com.yahoo.config.provision.Flavor; import com.yahoo.config.provision.HostFilter; import com.yahoo.config.provision.NodeFlavors; @@ -43,7 +44,6 @@ import com.yahoo.vespa.hosted.provision.node.filter.ParentHostFilter; import com.yahoo.vespa.hosted.provision.restapi.NodesResponse.ResponseType; import com.yahoo.vespa.orchestrator.Orchestrator; import com.yahoo.yolean.Exceptions; - import javax.inject.Inject; import java.io.IOException; import java.io.UncheckedIOException; @@ -102,6 +102,10 @@ public class NodesV2ApiHandler extends ThreadedHttpRequestHandler { catch (IllegalArgumentException e) { return ErrorResponse.badRequest(Exceptions.toMessageString(e)); } + catch (ApplicationLockException e) { + log.log(Level.INFO, "Timed out getting lock when handling '" + request.getUri() + "': " + Exceptions.toMessageString(e)); + return ErrorResponse.internalServerError(Exceptions.toMessageString(e)); + } catch (RuntimeException e) { log.log(Level.WARNING, "Unexpected error handling '" + request.getUri() + "'", e); return ErrorResponse.internalServerError(Exceptions.toMessageString(e)); |