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 no ports for GameServer #749

Closed
ktong opened this issue Apr 28, 2019 · 8 comments · Fixed by #2006
Closed

Allow no ports for GameServer #749

ktong opened this issue Apr 28, 2019 · 8 comments · Fixed by #2006
Labels
good first issue These are great first issues. If you are looking for a place to start, start here! help wanted We would love help on these issues. Please come help us! kind/design Proposal discussing new features / fixes and how they should be implemented kind/feature New features for Agones
Milestone

Comments

@ktong
Copy link

ktong commented Apr 28, 2019

We would like to leverage company's API gateway to provide https termination, authentication, rate limit, ip filter and more in front of game server which uses websocket (probably quic later) for communication. Our K8s cluster is private GKE, and has not public ip for nodes, and all traffic will route from API gateway.

Currently, ports expose game server using node ip and ports. The K8s cluster works well but as it does not use node's port for traffic, ports configuration is a little annoying. It would be better if game server can config without ports (no hostport too).

@markmandel markmandel added kind/design Proposal discussing new features / fixes and how they should be implemented kind/feature New features for Agones labels Apr 28, 2019
@roberthbailey
Copy link
Member

This has some overlap with what is discussed in #668 for bare metal. The deployment environment is different, but the feature ask sounds similar.

@theminecoder
Copy link

@markmandel So hey, I've finally had a chance to look at this and the pr again and it seems that Agones does require at least 1 port to be specified in the game server template.

The Fleet "event-server" is invalid: spec.template.spec.ports: Invalid value: "": spec.template.spec.ports in body should have at least 1 items

The current workaround that we are using is exposing a port that isn't being listened on in the container however I really don't want to keep that going long term. My take is that not allowing this is kind of a bug and the fix needs to be that we can not specify ports or that we allow Internal ports to become a thing. If we go the Internal port route I am happy to reopen the pr (I just need to refork the repo & do the commits :P)

@markmandel
Copy link
Member

I feel like we could likely just change the validation so that it allows 0, and it would probably work just fine?

(Seems like the easiest place to start!)

@theminecoder
Copy link

Yeh that's likely the way to go, however I do think we should have a test case for this just to make sure it doesn't lock up the port assigner like it was when I was testing with Internal ports (which is why https://github.com/googleforgames/agones/pull/1576/files#diff-310ad1666561c0a99ea0effb347aea2bR124 existed)

@markmandel
Copy link
Member

Ah yep! Testing always good 👍

@ItsKev
Copy link
Contributor

ItsKev commented Feb 23, 2021

Any news on this issue? We want to use Agones but would like to handle the routing to the gameservers by ourselves.

@markmandel markmandel added help wanted We would love help on these issues. Please come help us! good first issue These are great first issues. If you are looking for a place to start, start here! labels Feb 23, 2021
@markmandel
Copy link
Member

@ItsKev sounds like you are volunteering to work on this feature 😊

@ItsKev
Copy link
Contributor

ItsKev commented Feb 24, 2021

@markmandel Sure, I'll try my best.

@markmandel markmandel added this to the 1.13.0 milestone Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue These are great first issues. If you are looking for a place to start, start here! help wanted We would love help on these issues. Please come help us! kind/design Proposal discussing new features / fixes and how they should be implemented kind/feature New features for Agones
Projects
None yet
5 participants