You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by "birke-kb (via GitHub)" <gi...@apache.org> on 2026/03/11 15:43:55 UTC
[PR] Remove externalTrafficPolicy from template for service type ClusterIP [apisix-helm-chart]
birke-kb opened a new pull request, #950:
URL: https://github.com/apache/apisix-helm-chart/pull/950
Add a condition to specify `externalTrafficPolicy` in the apisix-gateway service spec only for service types LoadBalancer and NodePort. Adding the field for service type ClusterIP leads to configuration errors in Kubernetes.
--
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: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Remove externalTrafficPolicy from template for service type ClusterIP [apisix-helm-chart]
Posted by "AlinsRan (via GitHub)" <gi...@apache.org>.
AlinsRan closed pull request #950: Remove externalTrafficPolicy from template for service type ClusterIP
URL: https://github.com/apache/apisix-helm-chart/pull/950
--
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: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Remove externalTrafficPolicy from template for service type ClusterIP [apisix-helm-chart]
Posted by "nic-6443 (via GitHub)" <gi...@apache.org>.
nic-6443 merged PR #950:
URL: https://github.com/apache/apisix-helm-chart/pull/950
--
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: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Remove externalTrafficPolicy from template for service type ClusterIP [apisix-helm-chart]
Posted by "eedeebee (via GitHub)" <gi...@apache.org>.
eedeebee commented on PR #950:
URL: https://github.com/apache/apisix-helm-chart/pull/950#issuecomment-4987464706
Hi apisix folks - thanks for making and maintaining this chart!
Is there anything else needed for this PR to get reviewed?
It addresses https://github.com/apache/apisix-helm-chart/issues/755 and https://github.com/apache/apisix-helm-chart/issues/780 which have been open for 2 years. I'm happy to help or add whatever it takes to get it in.
--
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: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Remove externalTrafficPolicy from template for service type ClusterIP [apisix-helm-chart]
Posted by "martin-mat (via GitHub)" <gi...@apache.org>.
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: notifications-unsubscribe@apisix.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org