diff options
author | Øyvind Grønnesby <oyving@verizonmedia.com> | 2021-09-14 08:39:41 +0200 |
---|---|---|
committer | Øyvind Grønnesby <oyving@verizonmedia.com> | 2021-09-14 08:39:41 +0200 |
commit | dd9c6e1cc54bb5f0541546937f7e1814bab02354 (patch) | |
tree | 174a582a61fd5b725c60ff46197496af764ed5d0 /controller-server | |
parent | 4a01cd89f3bccdb62c2371eba96b5a0423c04868 (diff) |
Sort API output for stable tests
Diffstat (limited to 'controller-server')
-rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/routing/RoutingApiHandler.java | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/routing/RoutingApiHandler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/routing/RoutingApiHandler.java index f16f30b1a6c..ff8a97611f1 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/routing/RoutingApiHandler.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/routing/RoutingApiHandler.java @@ -96,12 +96,15 @@ public class RoutingApiHandler extends AuditLoggingRequestHandler { private HttpResponse endpoints(Path path) { var instanceId = instanceFrom(path); - var endpoints = controller.routing().endpointsOf(instanceId); + var endpoints = controller.routing().endpointsOf(instanceId) + .sortedBy(Comparator.comparing(Endpoint::name)) + .asList(); - var deployments = endpoints.asList().stream() + var deployments = endpoints.stream() .flatMap(e -> e.zones().stream()) .distinct() .map(zoneId -> new DeploymentId(instanceId, zoneId)) + .sorted(Comparator.comparing(DeploymentId::dottedString)) .collect(Collectors.toList()); var deploymentsStatus = deployments.stream() @@ -120,7 +123,7 @@ public class RoutingApiHandler extends AuditLoggingRequestHandler { var endpointRoot = endpointsRoot.addObject(); endpointToSlime(endpointRoot, endpoint); var zonesRoot = endpointRoot.setArray("zones"); - endpoint.zones().forEach(zoneId -> { + endpoint.zones().stream().sorted(Comparator.comparing(ZoneId::value)).forEach(zoneId -> { var deploymentId = new DeploymentId(instanceId, zoneId); deploymentsStatus.getOrDefault(deploymentId, List.of()).forEach(status -> { deploymentStatusToSlime(zonesRoot.addObject(), deploymentId, status, endpoint.routingMethod()); |