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

Allow configuration of proxy_responses value for tcp/udp configmaps #1731

Closed
NinjaBanjo opened this issue Nov 20, 2017 · 2 comments · Fixed by #1777
Closed

Allow configuration of proxy_responses value for tcp/udp configmaps #1731

NinjaBanjo opened this issue Nov 20, 2017 · 2 comments · Fixed by #1777

Comments

@NinjaBanjo
Copy link

Is this a BUG REPORT or FEATURE REQUEST? (choose one): FEATURE REQUEST

If this is a FEATURE REQUEST, please:

  • "proxy_responses" value is configurable with udp/tcp configmaps (annotation?)

Currently this value is hard-coded in the nginx.conf template file

Reasoning:
I'm currently trying to use ingress-nginx to expose a udp application (teamspeak) with a long-living udp connection cycle. I can use nc to see the service is exposed

$ nc -v -u -z -w 3 10.1.101.10 9987
$ Connection to 10.1.101.10 9987 port [udp/*] succeeded!

Trying to use the exposed port to reach the application acces_logs show a udp entry, but the application doesn't "connect":

[20/Nov/2017:05:20:23 +0000]UDP20032340.002
[20/Nov/2017:05:20:23 +0000]UDP200244380.094
@piercefoospacenug
Copy link

i see that the feature request has been solved. However i am currently trying to accomplish the same as you, @NinjaBanjo . Could you please let me know if you managed to get teamspeak working with this method? Even though the server "works", I am experiencing a lot of packetloss when using the nginx udp ingress for it which makes it unusable. thank you

@githubstoreforme
Copy link

Is this packet loss been solved?
I am experiencing the same on latest also when I tried the example from

https://github.com/nginxinc/kubernetes-ingress/blob/main/examples/tcp-udp

Let If I need to do any additional configuration here?

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

Successfully merging a pull request may close this issue.

3 participants