summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Polden <mpolden@mpolden.no>2023-09-06 14:14:51 +0200
committerGitHub <noreply@github.com>2023-09-06 14:14:51 +0200
commitdb8f9d346d37ec87833a87582ffb31cb0857f1eb (patch)
treebed7ffd269c37062173439e97e3e46ed36181609
parent6f7bfba77282ab5ef814e4e29db9d06b535e92be (diff)
parent75f524f48b55763ef0af7168314e1c9dbff62c62 (diff)
Merge pull request #28422 from vespa-engine/mpolden/hide-weighted
Filter generated endpoints before other filters MERGEOK
-rw-r--r--controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java10
1 files changed, 5 insertions, 5 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java
index 080ab46f0fa..30f3f74c400 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/restapi/application/ApplicationApiHandler.java
@@ -2063,13 +2063,13 @@ public class ApplicationApiHandler extends AuditLoggingRequestHandler {
EndpointList endpoints = zoneEndpoints.and(declaredEndpoints);
EndpointList generatedEndpoints = endpoints.generated();
if (!includeHidden) {
- // Hide legacy and weighted endpoints by default
- endpoints = endpoints.not().legacy()
- .not().scope(Endpoint.Scope.weighted);
- // Hide non-generated if we have any
+ // If we have generated endpoints, hide non-generated
if (!generatedEndpoints.isEmpty()) {
- endpoints = generatedEndpoints;
+ endpoints = endpoints.generated();
}
+ // Hide legacy and weighted endpoints
+ endpoints = endpoints.not().legacy()
+ .not().scope(Endpoint.Scope.weighted);
}
return endpoints;
}