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

Galaxy can't handle services with multiple ports #226

Open
jbardin opened this issue Apr 17, 2015 · 3 comments
Open

Galaxy can't handle services with multiple ports #226

jbardin opened this issue Apr 17, 2015 · 3 comments
Labels

Comments

@jbardin
Copy link
Contributor

jbardin commented Apr 17, 2015

from registry/registry.go:

//FIXME: We're using the first found port and assuming it's tcp.
//How should we handle a service that exposes multiple ports
//as well as tcp vs udp ports
@jbardin
Copy link
Contributor Author

jbardin commented Apr 20, 2015

The listening port is also controlled by the single config variable GALAXY_PORT, which is what is mapped to the first port listed by docker

@jbardin jbardin added the bug label Apr 20, 2015
@jbardin
Copy link
Contributor Author

jbardin commented Apr 20, 2015

The ports definitions are stored in a map, and therefor have no order defined. A service with multiple ports will have it's backend redefined randomly every time commander checks its registration.

@jbardin
Copy link
Contributor Author

jbardin commented Apr 21, 2015

Partially fixed by pull #227. Services still can't expose multiple ports through shuttle, but they will at least be able to work at all.

Services can set GALAXY_PORT to match the internal port assignment that they wish to expose in shuttle.

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

No branches or pull requests

1 participant