aboutsummaryrefslogtreecommitdiffstats
path: root/node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTest.java
diff options
context:
space:
mode:
authorHarald Musum <musum@yahooinc.com>2022-10-06 13:03:47 +0200
committerHarald Musum <musum@yahooinc.com>2022-10-06 13:03:47 +0200
commit319206b87b689dfe19007500cc6dac6e84f02103 (patch)
tree6665a3d0544f85159eff536b0ba6fa56d600df37 /node-repository/src/test/java/com/yahoo/vespa/hosted/provision/provisioning/ProvisioningTest.java
parentbbb47ccadb603a84b3a9d4e695aa098ea46a25ad (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.java2
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"));
}
}