docker_container: add default_host_ip option

[felixfontein]

SUMMARY

Fixes #70.

ISSUE TYPE

• Feature Pull Request

COMPONENT NAME

docker_container

[felixfontein]

@seblu if you have time, feel free to test this :) I tested it locally with Docker CE 20.10.2, and for my simple test case it worked.

[seblu]

Yes, this will make my ansible collection fu be better ! I needed to learn how to overrides collection.

So, it looks like it works as expected.

TASK [community.docker.docker_container] *****************************************************************************************************************************************************
task path: /home/seblu/scm/infra/playbooks/test.yml:6
--- before
+++ after
@@ -2,11 +2,7 @@
     "expected_ports": {
         "25826/udp": [
             {
-                "HostIp": "::",
-                "HostPort": "25826"
-            },
-            {
-                "HostIp": "0.0.0.0",
+                "HostIp": "",
                 "HostPort": "25826"
             }
         ]

[felixfontein]

@WojciechowskiPiotr can you take a quick look? If we can get this merged and a new version released by next ~Tuesday, this should get included in the next Ansible 2.10.6 release (which will probably be the last 2.10.x release).

[WojciechowskiPiotr]

@WojciechowskiPiotr can you take a quick look? If we can get this merged and a new version released by next ~Tuesday, this should get included in the next Ansible 2.10.6 release (which will probably be the last 2.10.x release).

[WojciechowskiPiotr]

Merged, closed by accident, sorry.

Shouldn't we include this option in examples? I mean this is quite a significant change that needs to be made in playbooks when migrating to 20.10 so maybe some example(s) should help the users to not make mistakes? I know they should read the documentation, but still...

[felixfontein]

@WojciechowskiPiotr makes sense. Feel free to create a PR for that!