-
Notifications
You must be signed in to change notification settings - Fork 5
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
Bug: Gateway listener names are not used properly #19
Comments
I'm having potentially related problems as well:
Happy to open 4 different issues, if these are really issues. Maybe I'm doing something wrong. |
Huh, that's a lot of issues in one go. Can you please reorg this dump (or at least the ones that we can confirm) to separate issues and close this one? Let see:
This is confirmed. Here is the offending line: indeed, we should include the listener name (or use only the listener name, it is supposed to be unique), something like the bwlow:
Do you feel like sending a PR? Don't forget to add a test pls.
Does this have something to do with mixed-protocol LB support? This is only a bug if mixed-protocol LB support is enabled. You should know better than me, the PR came from you proper...:-D
This one seems particularly nasty: I've seen this issue a couple of times in the tests but I could never quite reproduce it. The problem is that every time we update the service we have to explicitly specify the nodePort otherwise Kubernetes will choose one randomly and we end up rewriting the nodeport every time we run the render cycle. So we have to make sure during service updates that if a service-port already contains a valid nodeport then we leave that unchanged, and only add a new nodeport when we add a new service-port. The only place we have access to both the existing Service and the one we are about to install is inside
Hmmm, we don't even touch the
UDP + TLS = DTLS. Still, I don't get this, can you please elaborate? |
The gateway-operator does not use the configured names of the listeners in the Gateway resource.
In case of applying the following manifest:
The following error shows up in the gateway-operator's logs:
It seems the Services
spec.ports.name
field is filled with the gateway's name + the port's protocol type, instead of the name configured in the Gateway'sspec.listeners.name
.This does not cause any problems if only one listener is configured. As soon as two or more listeners are configured their names will bump into each other causing an error.
The text was updated successfully, but these errors were encountered: