summaryrefslogtreecommitdiffstats
path: root/controller-server
diff options
context:
space:
mode:
authorjonmv <venstad@gmail.com>2022-10-20 19:01:35 +0200
committerjonmv <venstad@gmail.com>2022-10-20 19:01:35 +0200
commit90ee7ea1f576a1363b753df5f957b7cffb61a655 (patch)
treef6bcadb11421b766bbf0c635b0883dc94db50bea /controller-server
parente6d7682b8a972aea6fc1376504ca2ee9355805f2 (diff)
Throw the right exception instead
Diffstat (limited to 'controller-server')
-rw-r--r--controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/pkg/ApplicationPackageValidator.java5
1 files changed, 4 insertions, 1 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/pkg/ApplicationPackageValidator.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/pkg/ApplicationPackageValidator.java
index 062dc261194..63bfe871c76 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/pkg/ApplicationPackageValidator.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/application/pkg/ApplicationPackageValidator.java
@@ -123,7 +123,10 @@ public class ApplicationPackageValidator {
endpoint.regions().stream().sorted().toList());
}
}
- else { // Also covers unknown, future clouds. Expand above clauses when needed.
+ else if (clouds.size() == 1) {
+ throw new IllegalArgumentException("unknown cloud '" + clouds.iterator().next() + "'");
+ }
+ else {
throw new IllegalArgumentException(endpointString + " cannot contain regions in different clouds: " +
endpoint.regions().stream().sorted().toList());
}