Propagate error message when setting plan fails

1 files changed, 3 insertions, 19 deletions

diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/billing/BillingApiHandler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/billing/BillingApiHandler.java
index ccbee15d2c5..e4a0e2af6b3 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/billing/BillingApiHandler.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/billing/BillingApiHandler.java
@@ -139,12 +139,9 @@ public class BillingApiHandler extends LoggingRequestHandler {
         var planId = PlanId.from(slime.field("plan").asString());
         var hasApplications = applicationController.asList(tenantName).size() > 0;
 
-        if (billingController.setPlan(tenantName, planId, hasApplications)) {
-            return new StringResponse("Plan: " + planId.value());
-        } else {
-            return ErrorResponse.forbidden("Invalid plan change with active deployments");
-        }
-
+        return billingController.setPlan(tenantName, planId, hasApplications)
+                .<HttpResponse>map(ErrorResponse::forbidden)
+                .orElse(new StringResponse("Plan: " + planId.value()));
     }
 
     private HttpResponse getBillingAllTenants(String until) {
@@ -374,19 +371,6 @@ public class BillingApiHandler extends LoggingRequestHandler {
         return inspector.field(field).asString();
     }
 
-    private DeploymentId getDeploymentIdOrNull(Inspector inspector) {
-        if (inspector.field("applicationId").valid() != inspector.field("zoneId").valid() ) {
-            throw new BadRequestException("Either both application id and zone id should be set, or neither.");
-        }
-        if (inspector.field("applicationId").valid()) {
-            return new DeploymentId(
-                    ApplicationId.fromSerializedForm(inspector.field("applicationId").asString()),
-                    com.yahoo.config.provision.zone.ZoneId.from(inspector.field("zoneId").asString())
-            );
-        }
-        return null;
-    }
-
     private LocalDate untilParameter(String until) {
         if (until == null || until.isEmpty() || until.isBlank())
             return LocalDate.now().plusDays(1);