updated service.yaml

HariSekhon · May 7, 2023 · 8b63cdc · 8b63cdc
1 parent 0ee92ec
commit 8b63cdc
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/service.yaml b/service.yaml
@@ -129,6 +129,11 @@ spec:
   # - cons:
   #   - can cause load imbalances between pods (which is why this is not the default)
   # - not needed for HTTP(s) services using Ingress -> default ClusterIP services
+  #
+  # XXX: beware externalTrafficPolicy: Local may skip a hop on GKE going directly to a node with a pod via a health check, but if the healthcheck fails and it goes to a random node this could cause timeouts:
+  #
+  #   https://cloud.google.com/knowledge/kb/externaltrafficpolicy-cluster-works-but-local-doesn-t-000004708
+  #
   #externalTrafficPolicy: Local
   externalTrafficPolicy: Cluster  # default, distribute to all service endpoints (pods) inside cluster