Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

syncer: cannot change healthCheckNodePort on loadBalancer #240

Closed
olljanat opened this issue Dec 9, 2021 · 2 comments
Closed

syncer: cannot change healthCheckNodePort on loadBalancer #240

olljanat opened this issue Dec 9, 2021 · 2 comments

Comments

@olljanat
Copy link
Contributor

olljanat commented Dec 9, 2021

Found from v0.5.0-alpha.7 when publishing LoadBalancer service with externalTrafficPolicy: Local that syncer log was full errors like these:

I1209 16:43:44.702759       1 logr.go:249] service: traefik: updating physical vcluster-test/traefik-x-kube-system-x-test, because virtual service have changed
E1209 16:43:44.721183       1 logr.go:265] reconciler group  reconciler kind Service: controller: service: name traefik namespace kube-system: Reconciler error Service "traefik-x-kube-system-x-test" is invalid: spec.healthCheckNodePort: Invalid value: 30395: cannot change healthCheckNodePort on loadBalancer service with externalTraffic=Local during update

As workaround it can be specified manually like described on https://kubernetes.io/docs/tasks/access-application-cluster/create-external-load-balancer/#preserving-the-client-source-ip but would be nice to get it fixed on way that dynamic port can be used.

@FabianKramm
Copy link
Member

@olljanat thanks for creating this issue! You are correct, seems like this is currently a problem, we'll add a fix and new e2e test for this.

@olljanat
Copy link
Contributor Author

This works correctly on v0.5.0-beta.0

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants