|
Packit |
bc9a3a |
# This file is part of cloud-init. See LICENSE file for license information.
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
from cloudinit.config import cc_lxd
|
|
Packit |
bc9a3a |
from cloudinit.sources import DataSourceNoCloud
|
|
Packit |
bc9a3a |
from cloudinit import (distros, helpers, cloud)
|
|
Packit |
bc9a3a |
from cloudinit.tests import helpers as t_help
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
try:
|
|
Packit |
bc9a3a |
from unittest import mock
|
|
Packit |
bc9a3a |
except ImportError:
|
|
Packit |
bc9a3a |
import mock
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
class TestLxd(t_help.CiTestCase):
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
with_logs = True
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
lxd_cfg = {
|
|
Packit |
bc9a3a |
'lxd': {
|
|
Packit |
bc9a3a |
'init': {
|
|
Packit |
bc9a3a |
'network_address': '0.0.0.0',
|
|
Packit |
bc9a3a |
'storage_backend': 'zfs',
|
|
Packit |
bc9a3a |
'storage_pool': 'poolname',
|
|
Packit |
bc9a3a |
}
|
|
Packit |
bc9a3a |
}
|
|
Packit |
bc9a3a |
}
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
def _get_cloud(self, distro):
|
|
Packit |
bc9a3a |
cls = distros.fetch(distro)
|
|
Packit |
bc9a3a |
paths = helpers.Paths({})
|
|
Packit |
bc9a3a |
d = cls(distro, {}, paths)
|
|
Packit |
bc9a3a |
ds = DataSourceNoCloud.DataSourceNoCloud({}, d, paths)
|
|
Packit |
bc9a3a |
cc = cloud.Cloud(ds, paths, {}, d, None)
|
|
Packit |
bc9a3a |
return cc
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd.maybe_cleanup_default")
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd.util")
|
|
Packit |
bc9a3a |
def test_lxd_init(self, mock_util, m_maybe_clean):
|
|
Packit |
bc9a3a |
cc = self._get_cloud('ubuntu')
|
|
Packit |
bc9a3a |
mock_util.which.return_value = True
|
|
Packit |
bc9a3a |
m_maybe_clean.return_value = None
|
|
Packit |
bc9a3a |
cc_lxd.handle('cc_lxd', self.lxd_cfg, cc, self.logger, [])
|
|
Packit |
bc9a3a |
self.assertTrue(mock_util.which.called)
|
|
Packit |
bc9a3a |
# no bridge config, so maybe_cleanup should not be called.
|
|
Packit |
bc9a3a |
self.assertFalse(m_maybe_clean.called)
|
|
Packit |
bc9a3a |
self.assertEqual(
|
|
Packit |
bc9a3a |
[mock.call(['lxd', 'waitready', '--timeout=300']),
|
|
Packit |
bc9a3a |
mock.call(
|
|
Packit |
bc9a3a |
['lxd', 'init', '--auto', '--network-address=0.0.0.0',
|
|
Packit |
bc9a3a |
'--storage-backend=zfs', '--storage-pool=poolname'])],
|
|
Packit |
bc9a3a |
mock_util.subp.call_args_list)
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd.maybe_cleanup_default")
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd.util")
|
|
Packit |
bc9a3a |
def test_lxd_install(self, mock_util, m_maybe_clean):
|
|
Packit |
bc9a3a |
cc = self._get_cloud('ubuntu')
|
|
Packit |
bc9a3a |
cc.distro = mock.MagicMock()
|
|
Packit |
bc9a3a |
mock_util.which.return_value = None
|
|
Packit |
bc9a3a |
cc_lxd.handle('cc_lxd', self.lxd_cfg, cc, self.logger, [])
|
|
Packit |
bc9a3a |
self.assertNotIn('WARN', self.logs.getvalue())
|
|
Packit |
bc9a3a |
self.assertTrue(cc.distro.install_packages.called)
|
|
Packit |
bc9a3a |
cc_lxd.handle('cc_lxd', self.lxd_cfg, cc, self.logger, [])
|
|
Packit |
bc9a3a |
self.assertFalse(m_maybe_clean.called)
|
|
Packit |
bc9a3a |
install_pkg = cc.distro.install_packages.call_args_list[0][0][0]
|
|
Packit |
bc9a3a |
self.assertEqual(sorted(install_pkg), ['lxd', 'zfsutils-linux'])
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd.maybe_cleanup_default")
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd.util")
|
|
Packit |
bc9a3a |
def test_no_init_does_nothing(self, mock_util, m_maybe_clean):
|
|
Packit |
bc9a3a |
cc = self._get_cloud('ubuntu')
|
|
Packit |
bc9a3a |
cc.distro = mock.MagicMock()
|
|
Packit |
bc9a3a |
cc_lxd.handle('cc_lxd', {'lxd': {}}, cc, self.logger, [])
|
|
Packit |
bc9a3a |
self.assertFalse(cc.distro.install_packages.called)
|
|
Packit |
bc9a3a |
self.assertFalse(mock_util.subp.called)
|
|
Packit |
bc9a3a |
self.assertFalse(m_maybe_clean.called)
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd.maybe_cleanup_default")
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd.util")
|
|
Packit |
bc9a3a |
def test_no_lxd_does_nothing(self, mock_util, m_maybe_clean):
|
|
Packit |
bc9a3a |
cc = self._get_cloud('ubuntu')
|
|
Packit |
bc9a3a |
cc.distro = mock.MagicMock()
|
|
Packit |
bc9a3a |
cc_lxd.handle('cc_lxd', {'package_update': True}, cc, self.logger, [])
|
|
Packit |
bc9a3a |
self.assertFalse(cc.distro.install_packages.called)
|
|
Packit |
bc9a3a |
self.assertFalse(mock_util.subp.called)
|
|
Packit |
bc9a3a |
self.assertFalse(m_maybe_clean.called)
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
def test_lxd_debconf_new_full(self):
|
|
Packit |
bc9a3a |
data = {"mode": "new",
|
|
Packit |
bc9a3a |
"name": "testbr0",
|
|
Packit |
bc9a3a |
"ipv4_address": "10.0.8.1",
|
|
Packit |
bc9a3a |
"ipv4_netmask": "24",
|
|
Packit |
bc9a3a |
"ipv4_dhcp_first": "10.0.8.2",
|
|
Packit |
bc9a3a |
"ipv4_dhcp_last": "10.0.8.254",
|
|
Packit |
bc9a3a |
"ipv4_dhcp_leases": "250",
|
|
Packit |
bc9a3a |
"ipv4_nat": "true",
|
|
Packit |
bc9a3a |
"ipv6_address": "fd98:9e0:3744::1",
|
|
Packit |
bc9a3a |
"ipv6_netmask": "64",
|
|
Packit |
bc9a3a |
"ipv6_nat": "true",
|
|
Packit |
bc9a3a |
"domain": "lxd"}
|
|
Packit |
bc9a3a |
self.assertEqual(
|
|
Packit |
bc9a3a |
cc_lxd.bridge_to_debconf(data),
|
|
Packit |
bc9a3a |
{"lxd/setup-bridge": "true",
|
|
Packit |
bc9a3a |
"lxd/bridge-name": "testbr0",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv4": "true",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv4-address": "10.0.8.1",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv4-netmask": "24",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv4-dhcp-first": "10.0.8.2",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv4-dhcp-last": "10.0.8.254",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv4-dhcp-leases": "250",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv4-nat": "true",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv6": "true",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv6-address": "fd98:9e0:3744::1",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv6-netmask": "64",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv6-nat": "true",
|
|
Packit |
bc9a3a |
"lxd/bridge-domain": "lxd"})
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
def test_lxd_debconf_new_partial(self):
|
|
Packit |
bc9a3a |
data = {"mode": "new",
|
|
Packit |
bc9a3a |
"ipv6_address": "fd98:9e0:3744::1",
|
|
Packit |
bc9a3a |
"ipv6_netmask": "64",
|
|
Packit |
bc9a3a |
"ipv6_nat": "true"}
|
|
Packit |
bc9a3a |
self.assertEqual(
|
|
Packit |
bc9a3a |
cc_lxd.bridge_to_debconf(data),
|
|
Packit |
bc9a3a |
{"lxd/setup-bridge": "true",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv6": "true",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv6-address": "fd98:9e0:3744::1",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv6-netmask": "64",
|
|
Packit |
bc9a3a |
"lxd/bridge-ipv6-nat": "true"})
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
def test_lxd_debconf_existing(self):
|
|
Packit |
bc9a3a |
data = {"mode": "existing",
|
|
Packit |
bc9a3a |
"name": "testbr0"}
|
|
Packit |
bc9a3a |
self.assertEqual(
|
|
Packit |
bc9a3a |
cc_lxd.bridge_to_debconf(data),
|
|
Packit |
bc9a3a |
{"lxd/setup-bridge": "false",
|
|
Packit |
bc9a3a |
"lxd/use-existing-bridge": "true",
|
|
Packit |
bc9a3a |
"lxd/bridge-name": "testbr0"})
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
def test_lxd_debconf_none(self):
|
|
Packit |
bc9a3a |
data = {"mode": "none"}
|
|
Packit |
bc9a3a |
self.assertEqual(
|
|
Packit |
bc9a3a |
cc_lxd.bridge_to_debconf(data),
|
|
Packit |
bc9a3a |
{"lxd/setup-bridge": "false",
|
|
Packit |
bc9a3a |
"lxd/bridge-name": ""})
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
def test_lxd_cmd_new_full(self):
|
|
Packit |
bc9a3a |
data = {"mode": "new",
|
|
Packit |
bc9a3a |
"name": "testbr0",
|
|
Packit |
bc9a3a |
"ipv4_address": "10.0.8.1",
|
|
Packit |
bc9a3a |
"ipv4_netmask": "24",
|
|
Packit |
bc9a3a |
"ipv4_dhcp_first": "10.0.8.2",
|
|
Packit |
bc9a3a |
"ipv4_dhcp_last": "10.0.8.254",
|
|
Packit |
bc9a3a |
"ipv4_dhcp_leases": "250",
|
|
Packit |
bc9a3a |
"ipv4_nat": "true",
|
|
Packit |
bc9a3a |
"ipv6_address": "fd98:9e0:3744::1",
|
|
Packit |
bc9a3a |
"ipv6_netmask": "64",
|
|
Packit |
bc9a3a |
"ipv6_nat": "true",
|
|
Packit |
bc9a3a |
"domain": "lxd"}
|
|
Packit |
bc9a3a |
self.assertEqual(
|
|
Packit |
bc9a3a |
cc_lxd.bridge_to_cmd(data),
|
|
Packit |
bc9a3a |
(["network", "create", "testbr0",
|
|
Packit |
bc9a3a |
"ipv4.address=10.0.8.1/24", "ipv4.nat=true",
|
|
Packit |
bc9a3a |
"ipv4.dhcp.ranges=10.0.8.2-10.0.8.254",
|
|
Packit |
bc9a3a |
"ipv6.address=fd98:9e0:3744::1/64",
|
|
Packit |
bc9a3a |
"ipv6.nat=true", "dns.domain=lxd"],
|
|
Packit |
bc9a3a |
["network", "attach-profile",
|
|
Packit |
bc9a3a |
"testbr0", "default", "eth0"]))
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
def test_lxd_cmd_new_partial(self):
|
|
Packit |
bc9a3a |
data = {"mode": "new",
|
|
Packit |
bc9a3a |
"ipv6_address": "fd98:9e0:3744::1",
|
|
Packit |
bc9a3a |
"ipv6_netmask": "64",
|
|
Packit |
bc9a3a |
"ipv6_nat": "true"}
|
|
Packit |
bc9a3a |
self.assertEqual(
|
|
Packit |
bc9a3a |
cc_lxd.bridge_to_cmd(data),
|
|
Packit |
bc9a3a |
(["network", "create", "lxdbr0", "ipv4.address=none",
|
|
Packit |
bc9a3a |
"ipv6.address=fd98:9e0:3744::1/64", "ipv6.nat=true"],
|
|
Packit |
bc9a3a |
["network", "attach-profile",
|
|
Packit |
bc9a3a |
"lxdbr0", "default", "eth0"]))
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
def test_lxd_cmd_existing(self):
|
|
Packit |
bc9a3a |
data = {"mode": "existing",
|
|
Packit |
bc9a3a |
"name": "testbr0"}
|
|
Packit |
bc9a3a |
self.assertEqual(
|
|
Packit |
bc9a3a |
cc_lxd.bridge_to_cmd(data),
|
|
Packit |
bc9a3a |
(None, ["network", "attach-profile",
|
|
Packit |
bc9a3a |
"testbr0", "default", "eth0"]))
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
def test_lxd_cmd_none(self):
|
|
Packit |
bc9a3a |
data = {"mode": "none"}
|
|
Packit |
bc9a3a |
self.assertEqual(
|
|
Packit |
bc9a3a |
cc_lxd.bridge_to_cmd(data),
|
|
Packit |
bc9a3a |
(None, None))
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
class TestLxdMaybeCleanupDefault(t_help.CiTestCase):
|
|
Packit |
bc9a3a |
"""Test the implementation of maybe_cleanup_default."""
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
defnet = cc_lxd._DEFAULT_NETWORK_NAME
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd._lxc")
|
|
Packit |
bc9a3a |
def test_network_other_than_default_not_deleted(self, m_lxc):
|
|
Packit |
bc9a3a |
"""deletion or removal should only occur if bridge is default."""
|
|
Packit |
bc9a3a |
cc_lxd.maybe_cleanup_default(
|
|
Packit |
bc9a3a |
net_name="lxdbr1", did_init=True, create=True, attach=True)
|
|
Packit |
bc9a3a |
m_lxc.assert_not_called()
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd._lxc")
|
|
Packit |
bc9a3a |
def test_did_init_false_does_not_delete(self, m_lxc):
|
|
Packit |
bc9a3a |
"""deletion or removal should only occur if did_init is True."""
|
|
Packit |
bc9a3a |
cc_lxd.maybe_cleanup_default(
|
|
Packit |
bc9a3a |
net_name=self.defnet, did_init=False, create=True, attach=True)
|
|
Packit |
bc9a3a |
m_lxc.assert_not_called()
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd._lxc")
|
|
Packit |
bc9a3a |
def test_network_deleted_if_create_true(self, m_lxc):
|
|
Packit |
bc9a3a |
"""deletion of network should occur if create is True."""
|
|
Packit |
bc9a3a |
cc_lxd.maybe_cleanup_default(
|
|
Packit |
bc9a3a |
net_name=self.defnet, did_init=True, create=True, attach=False)
|
|
Packit |
bc9a3a |
m_lxc.assert_called_once_with(["network", "delete", self.defnet])
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
@mock.patch("cloudinit.config.cc_lxd._lxc")
|
|
Packit |
bc9a3a |
def test_device_removed_if_attach_true(self, m_lxc):
|
|
Packit |
bc9a3a |
"""deletion of network should occur if create is True."""
|
|
Packit |
bc9a3a |
nic_name = "my_nic"
|
|
Packit |
bc9a3a |
profile = "my_profile"
|
|
Packit |
bc9a3a |
cc_lxd.maybe_cleanup_default(
|
|
Packit |
bc9a3a |
net_name=self.defnet, did_init=True, create=False, attach=True,
|
|
Packit |
bc9a3a |
profile=profile, nic_name=nic_name)
|
|
Packit |
bc9a3a |
m_lxc.assert_called_once_with(
|
|
Packit |
bc9a3a |
["profile", "device", "remove", profile, nic_name])
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
|
|
Packit |
bc9a3a |
# vi: ts=4 expandtab
|