Feature #7864
closed
Exposed ports support
Description
The configuration or environment part of the wizard should show a way of adding exposed ports. The Docker Remote API supports it but Fog doesn't yet. Assume Fog does support it as I have a patch pending review.
The format for exposed ports goes like
{ "ExposedPorts" : [ "22/tcp": {}, "80/tcp": {}] }
Yes, key is the port/protocol, value is.. nothing?
Updated by Vanya Jauhal over 9 years ago
I guess we will have to let the user configure a list of exposed ports in the wizard, and apart from that we will also have to ask the users to map the ports every time they start a container ?
Because, as is apparent in the docker api documentation here(https://docs.docker.com/reference/api/docker_remote_api_v1.9/#start-a-container) , we will have to submit a port-binding every time we start a container ?
Updated by Daniel Lobato Garcia over 9 years ago
Right, probably the wizard can show this in the 'environment' step. We save that information in the db, so every time you start the container you can reutilize the ports information without prompting the user.
Updated by The Foreman Bot over 9 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman-docker/pull/64 added
- Pull request deleted (
)
Updated by David Davis over 9 years ago
Wouldn't support for PortBindings be much more useful? Unless I am understanding it incorrectly, it seems like you can do the same thing with PortBindings that you can do with ExposedPorts.
Updated by Vanya Jauhal about 9 years ago
- Status changed from Ready For Testing to Closed
- % Done changed from 0 to 100
Applied in changeset foreman-docker|8e2848d58b12a71a89f19e6eb98be790d5f19067.
Updated by