Merge pull request #11411 from vespa-engine/mpolden/log-removal

Log routing policy removal

1 files changed, 5 insertions, 0 deletions

diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/RoutingPolicies.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/RoutingPolicies.java
index 4e70ebb971b..12108b9d29a 100644
--- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/RoutingPolicies.java
+++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/RoutingPolicies.java
@@ -149,6 +149,10 @@ public class RoutingPolicies {
         // Remove active load balancers and irrelevant zones from candidates
         removalCandidates.removeIf(policy -> activeLoadBalancers.contains(policy.canonicalName()) ||
                                              !policy.zone().equals(loadBalancers.zone));
+        if (!removalCandidates.isEmpty()) {
+            LOGGER.log(LogLevel.WARNING, "Removing " + removalCandidates + ". Active load balancers " +
+                                         activeLoadBalancers);
+        }
         for (var policy : removalCandidates) {
             var dnsName = policy.endpointIn(controller.system()).dnsName();
             controller.nameServiceForwarder().removeRecords(Record.Type.CNAME, RecordName.from(dnsName), Priority.normal);
@@ -235,6 +239,7 @@ public class RoutingPolicies {
                     return true;
             }
             // Any other state, such as inactive, is ignored.
+            LOGGER.log(LogLevel.WARNING, "Ignoring load balancer " + loadBalancer.hostname() + " in state " + loadBalancer.state());
             return false;
         }