aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorValerij Fredriksen <valerijf@oath.com>2018-07-24 12:13:18 +0200
committerValerij Fredriksen <valerijf@oath.com>2018-07-24 12:13:18 +0200
commitc6bc116c57550c953767e9e722aa287ed3903809 (patch)
tree0b71459fae2a8f67e9347992710a281704958fa8
parent86f6c9456fe7bf15434e22d47fb3f84eae7c1fe6 (diff)
Improve error messages
-rw-r--r--controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java6
1 files changed, 4 insertions, 2 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java
index ce16b31981c..9e45526e5e4 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java
@@ -1241,7 +1241,9 @@ public class ApplicationApiHandler extends LoggingRequestHandler {
controller.jobController().jobs(appId)
.forEach(jobType -> jobMap.put(jobType, controller.jobController()
.last(appId, jobType)
- .orElseThrow(() -> new RuntimeException("This is a data violation right?"))));
+ .orElseThrow(() -> new RuntimeException(String.format("Job %s for application %s appears in " +
+ "the list of previously ran jobs, but no status of the last execution found",
+ jobType.jobName(), appId.toShortString())))));
return jobMap;
}
@@ -1250,7 +1252,7 @@ public class ApplicationApiHandler extends LoggingRequestHandler {
Map<String, byte[]> dataParts = new MultipartParser().parse(request);
Inspector submitOptions = SlimeUtils.jsonToSlime(dataParts.get(EnvironmentResource.SUBMIT_OPTIONS)).get();
SourceRevision sourceRevision = toSourceRevision(submitOptions).orElseThrow(() ->
- new IllegalArgumentException("Must specify 'repository', 'branch' and 'commit"));
+ new IllegalArgumentException("Must specify 'repository', 'branch', and 'commit'"));
return JobControllerApiHandlerHelper.submitResponse(controller.jobController(), tenant, application,
sourceRevision,