diff options
author | Valerij Fredriksen <valerijf@verizonmedia.com> | 2019-02-19 15:13:08 +0100 |
---|---|---|
committer | Valerij Fredriksen <valerijf@verizonmedia.com> | 2019-02-19 15:13:08 +0100 |
commit | 574451272ca27938c1585cbaa7268a9372745d57 (patch) | |
tree | 6157bac2846613eb3c05d677fab751c034d4324b /node-repository | |
parent | 98bdc0347a0408eb17a24b1b269a4f0d742fa921 (diff) |
Do not log entire stack trace for simple errors
Diffstat (limited to 'node-repository')
-rw-r--r-- | node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostProvisionMaintainer.java | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostProvisionMaintainer.java b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostProvisionMaintainer.java index 5acdc56ffda..ef13731ada5 100644 --- a/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostProvisionMaintainer.java +++ b/node-repository/src/main/java/com/yahoo/vespa/hosted/provision/maintenance/HostProvisionMaintainer.java @@ -7,8 +7,9 @@ import com.yahoo.vespa.hosted.provision.Node; import com.yahoo.vespa.hosted.provision.NodeList; import com.yahoo.vespa.hosted.provision.NodeRepository; import com.yahoo.vespa.hosted.provision.node.Agent; -import com.yahoo.vespa.hosted.provision.provisioning.HostProvisioner; import com.yahoo.vespa.hosted.provision.provisioning.FatalProvisioningException; +import com.yahoo.vespa.hosted.provision.provisioning.HostProvisioner; +import com.yahoo.yolean.Exceptions; import java.time.Duration; import java.util.List; @@ -43,6 +44,8 @@ public class HostProvisionMaintainer extends Maintainer { try { List<Node> updatedNodes = hostProvisioner.provision(host, children); nodeRepository().write(updatedNodes); + } catch (IllegalArgumentException | IllegalStateException e) { + log.log(Level.INFO, "Failed to provision " + host.hostname() + ": " + Exceptions.toMessageString(e)); } catch (FatalProvisioningException e) { log.log(Level.SEVERE, "Failed to provision " + host.hostname() + ", failing out the host recursively", e); // Fail out as operator to force a quick redeployment |