ansible-test - allow for the same listening port on multiple interfac…

…es (#75387)
ansible · Aug 3, 2021 · 1d2af0b · 1d2af0b
1 parent c8d4131
commit 1d2af0b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/changelogs/fragments/ansible-test_inspect-ports-on-all-interfaces.yml b/changelogs/fragments/ansible-test_inspect-ports-on-all-interfaces.yml
@@ -0,0 +1,2 @@
+bugfixes:
+  - ansible-test - allow the same listening port on all container interfaces
diff --git a/test/lib/ansible_test/_internal/containers.py b/test/lib/ansible_test/_internal/containers.py
@@ -524,7 +524,7 @@ def register(self, args):  # type: (EnvironmentConfig) -> SupportContainer
             # inspect the support container to locate the published ports
             tcp_ports = dict((port, container.get_tcp_port(port)) for port in self.ports)
 
-            if any(not config or len(config) != 1 for config in tcp_ports.values()):
+            if any(not config or len(set(conf['HostPort'] for conf in config)) != 1 for config in tcp_ports.values()):
                 raise ApplicationError('Unexpected `docker inspect` results for published TCP ports:\n%s' % json.dumps(tcp_ports, indent=4, sort_keys=True))
 
             published_ports = dict((port, int(config[0]['HostPort'])) for port, config in tcp_ports.items())