diff options
author | Harald Musum <musum@yahooinc.com> | 2022-10-06 13:03:47 +0200 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2022-10-06 13:03:47 +0200 |
commit | 319206b87b689dfe19007500cc6dac6e84f02103 (patch) | |
tree | 6665a3d0544f85159eff536b0ba6fa56d600df37 /node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTest.java | |
parent | bbb47ccadb603a84b3a9d4e695aa098ea46a25ad (diff) |
Change exception when unable to find all requested hosts
Throwing an IllegalArgumentException in this case will lead to an
invalid application package error returned from config server, which
is misleading,as there is nothing the customer can do with the
application to fix this, it is a transient issue in node repo or a bug
in vespa code. Throwing a RuntimeException will lead to an internal
server error being returned, which is more appropriate.
Diffstat (limited to 'node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTest.java')
-rw-r--r-- | node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTest.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTest.java b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTest.java index 8c9c8939616..c1c4b1e96ea 100644 --- a/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTest.java +++ b/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTest.java @@ -651,7 +651,7 @@ public class ProvisioningTest { tester.activate(application, state.allHosts); fail("Expected exception"); } - catch (IllegalArgumentException e) { + catch (RuntimeException e) { assertTrue(e.getMessage().startsWith("Activation of " + application + " failed")); } } |