martin-mat commented on PR #950:
URL: 
https://github.com/apache/apisix-helm-chart/pull/950#issuecomment-5839900996

   Confirming this is still needed on the current chart (apisix 2.17.0, APISIX 
3.18.0). With `service.type: ClusterIP`, the install fails on Kubernetes 1.35:
   
   ```
   Error: INSTALLATION FAILED: 1 error occurred:
        * Service "apisix-gateway" is invalid: spec.externalTrafficPolicy: 
Invalid value: "Cluster": may only be set for externally-accessible services
   ```
   
   The template sets `externalTrafficPolicy` unconditionally, and the API 
server accepts it only on `LoadBalancer` and `NodePort` Services. This PR 
renders it for exactly those two types, so a ClusterIP gateway installs and the 
other types are unchanged. Fronting the gateway with an Ingress or a cloud load 
balancer is a common reason to want ClusterIP, as #755 and #780 describe.
   
   The workaround in the meantime, for anyone landing here, is 
`service.externalTrafficPolicy: null` in the values.
   
   Could a maintainer review this? It fixes #755, open since May 2024. Earlier 
attempts at the same fix (#792, #816) went stale without review.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to