aboutsummaryrefslogtreecommitdiffstats
path: root/controller-server
diff options
context:
space:
mode:
authorJon Marius Venstad <jvenstad@yahoo-inc.com>2018-09-24 13:54:14 +0200
committerJon Marius Venstad <jvenstad@yahoo-inc.com>2018-09-24 13:54:14 +0200
commitfcd6c5af4540a9cd1710103a895a28a65dda95bb (patch)
treebc662a0a39adfa24b74e1e6e45b33345686f4621 /controller-server
parent64f36e530db94f80e9358f06434c70a4dacdf84e (diff)
404 when requesting endpoints for non-existent deployments
Diffstat (limited to 'controller-server')
-rw-r--r--controller-server/src/main/java/com/yahoo/vespa/hosted/controller/ApplicationController.java5
1 files changed, 5 insertions, 0 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/ApplicationController.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/ApplicationController.java
index ad1e4ded76f..2dd3b1dda23 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/ApplicationController.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/ApplicationController.java
@@ -504,6 +504,11 @@ public class ApplicationController {
/** Returns the endpoints of the deployment, or an empty list if the request fails */
public Optional<List<URI>> getDeploymentEndpoints(DeploymentId deploymentId) {
+ if ( ! get(deploymentId.applicationId())
+ .map(application -> application.deployments().containsKey(deploymentId.zoneId()))
+ .orElse(deploymentId.applicationId().instance().isTester()))
+ throw new NotExistsException("Deployment", deploymentId.toString());
+
try {
return Optional.of(ImmutableList.copyOf(routingGenerator.endpoints(deploymentId).stream()
.map(RoutingEndpoint::getEndpoint)