aboutsummaryrefslogtreecommitdiffstats
path: root/controller-server
diff options
context:
space:
mode:
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
commitdd9c6e1cc54bb5f0541546937f7e1814bab02354 (patch)
tree174a582a61fd5b725c60ff46197496af764ed5d0 /controller-server
parent4a01cd89f3bccdb62c2371eba96b5a0423c04868 (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.java9
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());