diff options
Diffstat (limited to 'controller-server')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/billing/BillingApiHandler.java | 22 |
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); |