From d7252ba5829f005915ce8e7efc9ec252c06d8bc0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20Nasiadka?= Date: Mon, 7 Jun 2021 13:56:48 +0200 Subject: [PATCH] Change network_mode default to host Kolla-Ansible deploys Docker without bridge by default [1]. [1]: https://review.opendev.org/c/openstack/kolla-ansible/+/751795 Change-Id: I7430b72f5d8abe59a7151df73e6d59898d942636 --- kolla/common/config.py | 2 +- kolla/tests/test_build.py | 20 +++++++++---------- .../network-mode-host-2aca256e2a1fbf68.yaml | 6 ++++++ 3 files changed, 17 insertions(+), 11 deletions(-) create mode 100644 releasenotes/notes/network-mode-host-2aca256e2a1fbf68.yaml diff --git a/kolla/common/config.py b/kolla/common/config.py index 64affd5f49..07136d896f 100755 --- a/kolla/common/config.py +++ b/kolla/common/config.py @@ -174,7 +174,7 @@ _CLI_OPTS = [ help='Show all available images (filtering supported)'), cfg.StrOpt('namespace', short='n', default='kolla', help='The Docker namespace name'), - cfg.StrOpt('network_mode', default=None, + cfg.StrOpt('network_mode', default='host', help='The network mode for Docker build. Example: host'), cfg.BoolOpt('cache', default=True, help='Use the Docker cache when building'), diff --git a/kolla/tests/test_build.py b/kolla/tests/test_build.py index 3e9dd06341..422dfcd73a 100644 --- a/kolla/tests/test_build.py +++ b/kolla/tests/test_build.py @@ -166,8 +166,8 @@ class TasksTest(base.TestCase): mock_client().build.assert_called_once_with( path=self.image.path, tag=self.image.canonical_name, decode=True, - network_mode=None, nocache=False, rm=True, pull=True, forcerm=True, - buildargs=None) + network_mode='host', nocache=False, rm=True, pull=True, + forcerm=True, buildargs=None) self.assertTrue(builder.success) @@ -176,14 +176,14 @@ class TasksTest(base.TestCase): def test_build_image_with_network_mode(self, mock_client): self.dc = mock_client push_queue = mock.Mock() - self.conf.set_override('network_mode', 'host') + self.conf.set_override('network_mode', 'bridge') builder = build.BuildTask(self.conf, self.image, push_queue) builder.run() mock_client().build.assert_called_once_with( path=self.image.path, tag=self.image.canonical_name, decode=True, - network_mode='host', nocache=False, rm=True, pull=True, + network_mode='bridge', nocache=False, rm=True, pull=True, forcerm=True, buildargs=None) self.assertTrue(builder.success) @@ -203,8 +203,8 @@ class TasksTest(base.TestCase): mock_client().build.assert_called_once_with( path=self.image.path, tag=self.image.canonical_name, decode=True, - network_mode=None, nocache=False, rm=True, pull=True, forcerm=True, - buildargs=build_args) + network_mode='host', nocache=False, rm=True, pull=True, + forcerm=True, buildargs=build_args) self.assertTrue(builder.success) @@ -222,8 +222,8 @@ class TasksTest(base.TestCase): mock_client().build.assert_called_once_with( path=self.image.path, tag=self.image.canonical_name, decode=True, - network_mode=None, nocache=False, rm=True, pull=True, forcerm=True, - buildargs=build_args) + network_mode='host', nocache=False, rm=True, pull=True, + forcerm=True, buildargs=build_args) self.assertTrue(builder.success) @@ -243,8 +243,8 @@ class TasksTest(base.TestCase): mock_client().build.assert_called_once_with( path=self.image.path, tag=self.image.canonical_name, decode=True, - network_mode=None, nocache=False, rm=True, pull=True, forcerm=True, - buildargs=build_args) + network_mode='host', nocache=False, rm=True, pull=True, + forcerm=True, buildargs=build_args) self.assertTrue(builder.success) diff --git a/releasenotes/notes/network-mode-host-2aca256e2a1fbf68.yaml b/releasenotes/notes/network-mode-host-2aca256e2a1fbf68.yaml new file mode 100644 index 0000000000..2183791dce --- /dev/null +++ b/releasenotes/notes/network-mode-host-2aca256e2a1fbf68.yaml @@ -0,0 +1,6 @@ +--- +upgrade: + - | + Changed default of ``network_mode`` to ``host`` since Kolla-Ansible + ``bootstrap-servers`` is deploying Docker without a bridge by default + since `Wallaby `__