Examples: replace service.beta.kubernetes.io/external-traffic with service.spec.externalTrafficPolicy #1924
Comments
Merged
bklang
added a commit
to bklang/helm-docker-mailserver
that referenced
this issue
May 5, 2020
This makes two changes: First, it upgrades the syntax to use newer version of the external traffic policy. This is based on testing on my Kubernetes 1.15 cluster, and was discovered from [this reference](kubernetes/ingress-nginx#1924) Second, it makes the non-load-balancer case switch to Cluster policy, which is necessary to allow the cluster to route NodePort traffic without a Kubernetes load balancer
bklang
added a commit
to bklang/helm-docker-mailserver
that referenced
this issue
May 6, 2020
This makes two changes: First, it upgrades the syntax to use newer version of the external traffic policy. This is based on testing on my Kubernetes 1.15 cluster, and was discovered from [this reference](kubernetes/ingress-nginx#1924) Second, it makes the non-load-balancer case switch to Cluster policy, which is necessary to allow the cluster to route NodePort traffic without a Kubernetes load balancer Also, fixed tiny whitespace nit
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Kubernetes 1.7+ does not support
service.beta.kubernetes.io/external-traffic: OnlyLocalannotation anymore. Insteadservice.spec.externalTrafficPolicyshould be used.static-ip-svc.yamlin the examples should be changed to use the new configuration.Had a hard time troubleshooting the provided examples, until I came around this piece of documentation:
https://kubernetes.io/docs/tasks/access-application-cluster/create-external-load-balancer/#feature-availability
The text was updated successfully, but these errors were encountered: