mirror of
https://github.com/ansible-collections/community.general.git
synced 2025-04-26 12:21:26 -07:00
docker_container: improve publish all ports functionality (#46594)
* Improve handling of published_ports: all. * Add changelog.
This commit is contained in:
parent
d69700b236
commit
8afe46dc02
3 changed files with 101 additions and 2 deletions
|
@ -302,7 +302,7 @@ options:
|
|||
- "Use docker CLI syntax: C(8000), C(9000:8000), or C(0.0.0.0:9000:8000), where 8000 is a
|
||||
container port, 9000 is a host port, and 0.0.0.0 is a host interface."
|
||||
- Container ports must be exposed either in the Dockerfile or via the C(expose) option.
|
||||
- A value of all will publish all exposed container ports to random host ports, ignoring
|
||||
- A value of C(all) will publish all exposed container ports to random host ports, ignoring
|
||||
any other mappings.
|
||||
- If C(networks) parameter is provided, will inspect each network to see if there exists
|
||||
a bridge network with optional parameter com.docker.network.bridge.host_binding_ipv4.
|
||||
|
@ -1500,7 +1500,8 @@ class Container(DockerBaseClass):
|
|||
expected_volumes=config.get('Volumes'),
|
||||
expected_binds=host_config.get('Binds'),
|
||||
volumes_from=host_config.get('VolumesFrom'),
|
||||
working_dir=config.get('WorkingDir')
|
||||
working_dir=config.get('WorkingDir'),
|
||||
publish_all_ports=host_config.get('PublishAllPorts'),
|
||||
)
|
||||
if self.parameters.restart_policy:
|
||||
config_mapping['restart_retries'] = restart_policy.get('MaximumRetryCount')
|
||||
|
@ -2291,6 +2292,8 @@ class AnsibleDockerClientContainer(AnsibleDockerClient):
|
|||
comparisons[key_main]['comparison'] = value
|
||||
else:
|
||||
self.fail("Unknown comparison mode '%s'!" % value)
|
||||
# Add implicit options
|
||||
comparisons['publish_all_ports'] = dict(type='value', comparison='strict', name='published_ports')
|
||||
# Check legacy values
|
||||
if self.module.params['ignore_image'] and comparisons['image']['comparison'] != 'ignore':
|
||||
self.module.warn('The ignore_image option has been overridden by the comparisons option!')
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue