diff options
author | jonmv <venstad@gmail.com> | 2022-04-19 13:21:45 +0200 |
---|---|---|
committer | jonmv <venstad@gmail.com> | 2022-04-22 13:02:02 +0200 |
commit | 6e987c3053ffb1268aa8fd15ebf6ac7389f5ffde (patch) | |
tree | 8da00749a18d17918e9e125c6430b382b4fd0628 | |
parent | 4b6dc99190764e00976599911b860124dcead542 (diff) |
Verify zone exists before starting a direct deployment
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/JobController.java | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/JobController.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/JobController.java index b012e115db7..67cf7c9dfd9 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/JobController.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/deployment/JobController.java @@ -566,6 +566,9 @@ public class JobController { /** Stores the given package and starts a deployment of it, after aborting any such ongoing deployment.*/ public void deploy(ApplicationId id, JobType type, Optional<Version> platform, ApplicationPackage applicationPackage, boolean dryRun) { + if ( ! controller.zoneRegistry().hasZone(type.zone())) + throw new IllegalArgumentException(type.zone() + " is not present in this system"); + controller.applications().lockApplicationOrThrow(TenantAndApplicationId.from(id), application -> { if ( ! application.get().instances().containsKey(id.instance())) application = controller.applications().withNewInstance(application, id); |