added log for empty load assignment

allegro · Dec 16, 2024 · 46a320b · 46a320b
1 parent 866410f
commit 46a320b
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/...src/main/kotlin/pl/allegro/tech/servicemesh/envoycontrol/snapshot/EnvoySnapshotFactory.kt b/...src/main/kotlin/pl/allegro/tech/servicemesh/envoycontrol/snapshot/EnvoySnapshotFactory.kt
@@ -281,7 +281,14 @@ class EnvoySnapshotFactory(
                 //    ClusterLoadAssignments many times - it may reduce MEM, CPU and latency if some serviceTags are
                 //    commonly used
                 endpointsFactory.filterEndpoints(endpoints, routeSpec.settings.routingPolicy).let {
-                    endpointsFactory.assignLocalityWeights(routeSpec, it)
+                    endpointsFactory.assignLocalityWeights(routeSpec, it).also { la ->
+                        if (la.endpointsList.any { el -> el.lbEndpointsList.isEmpty() }) {
+                            logger.warn(
+                                "ClusterLoadAssignment for ${routeSpec.clusterName} has empty lbEndpoints, " +
+                                    "original clusterLoadAssignment: ${endpoints.toString()}"
+                            )
+                        }
+                    }
                 }
             }
         }