Move dryrun deploy to separate path

1 files changed, 4 insertions, 3 deletions

diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/systemflags/SystemFlagsHandler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/systemflags/SystemFlagsHandler.java
index 8e5376f9b9c..08bb7628080 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/systemflags/SystemFlagsHandler.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/systemflags/SystemFlagsHandler.java
@@ -49,18 +49,19 @@ class SystemFlagsHandler extends LoggingRequestHandler {
 
     private HttpResponse put(HttpRequest request) {
         Path path = new Path(request.getUri());
-        if (path.matches(API_PREFIX + "/deploy")) return deploy(request);
+        if (path.matches(API_PREFIX + "/deploy")) return deploy(request, /*dryRun*/false);
+        if (path.matches(API_PREFIX + "/dryrun")) return deploy(request, /*dryRun*/true);
         return ErrorResponse.notFoundError("Nothing at " + path);
     }
 
-    private HttpResponse deploy(HttpRequest request) {
+    private HttpResponse deploy(HttpRequest request, boolean dryRun) {
         // TODO Error handling
         String contentType = request.getHeader("Content-Type");
         if (!contentType.equalsIgnoreCase("application/zip")) {
             return ErrorResponse.badRequest("Invalid content type: " + contentType);
         }
         SystemFlagsDataArchive archive = SystemFlagsDataArchive.fromZip(request.getData());
-        SystemFlagsDeployResult result = deployer.deployFlags(archive, request.getBooleanProperty("dry-run"));
+        SystemFlagsDeployResult result = deployer.deployFlags(archive, dryRun);
         return new JacksonJsonResponse<>(200, result.toWire());
     }