Project

General

Profile

Feature #7864

Exposed ports support

Added by Daniel Lobato Garcia almost 6 years ago. Updated about 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:

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?

Associated revisions

Revision 8e2848d5 (diff)
Added by Vanya Jauhal about 5 years ago

Fixes #7864 - Adds support for exposed_ports

Revision 8e2848d5 (diff)
Added by Vanya Jauhal about 5 years ago

Fixes #7864 - Adds support for exposed_ports

History

#1 Updated by Vanya Jauhal over 5 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 ?

#2 Updated by Daniel Lobato Garcia over 5 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.

#3 Updated by Vanya Jauhal over 5 years ago

  • Assignee set to Vanya Jauhal

#4 Updated by The Foreman Bot over 5 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-docker/pull/64 added
  • Pull request deleted ()

#5 Updated by David Davis over 5 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.

#6 Updated by Vanya Jauhal about 5 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#7 Updated by Partha Aji about 5 years ago

  • Bugzilla link set to 1231362

Also available in: Atom PDF