diff options
author | jonmv <venstad@gmail.com> | 2022-10-20 19:01:35 +0200 |
---|---|---|
committer | jonmv <venstad@gmail.com> | 2022-10-20 19:01:35 +0200 |
commit | 90ee7ea1f576a1363b753df5f957b7cffb61a655 (patch) | |
tree | f6bcadb11421b766bbf0c635b0883dc94db50bea /controller-server | |
parent | e6d7682b8a972aea6fc1376504ca2ee9355805f2 (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.java | 5 |
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()); } |