83a23cd56a
When a subset of cinder services is enabled, we don't expect all interfaces to exist and unexpected errors/warnings to be reported in workload-status. Change-Id: I9d5b6149c847c972d6fb0cfaf697d2885bbbf5c1 Closes-Bug: 1606559
1025 lines
44 KiB
Python
1025 lines
44 KiB
Python
# Copyright 2016 Canonical Ltd
|
|
#
|
|
# Licensed under the Apache License, Version 2.0 (the "License");
|
|
# you may not use this file except in compliance with the License.
|
|
# You may obtain a copy of the License at
|
|
#
|
|
# http://www.apache.org/licenses/LICENSE-2.0
|
|
#
|
|
# Unless required by applicable law or agreed to in writing, software
|
|
# distributed under the License is distributed on an "AS IS" BASIS,
|
|
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
# See the License for the specific language governing permissions and
|
|
# limitations under the License.
|
|
|
|
import os
|
|
import subprocess
|
|
|
|
from collections import OrderedDict
|
|
from mock import patch, call, MagicMock, Mock
|
|
|
|
os.environ['JUJU_UNIT_NAME'] = 'cinder'
|
|
import cinder_utils as cinder_utils
|
|
|
|
from test_utils import CharmTestCase
|
|
|
|
TO_PATCH = [
|
|
# helpers.core.hookenv
|
|
'config',
|
|
'log',
|
|
'juju_log',
|
|
'relation_get',
|
|
'relation_set',
|
|
'local_unit',
|
|
# helpers.core.host
|
|
'lsb_release',
|
|
'mounts',
|
|
'umount',
|
|
'mkdir',
|
|
'service_restart',
|
|
# helpers.core.templating
|
|
'render',
|
|
# helpers.contrib.openstack.utils
|
|
'git_generate_systemd_init_files',
|
|
'git_src_dir',
|
|
# helpers.contrib.python.packages
|
|
'pip_install',
|
|
# ceph utils
|
|
# storage_utils
|
|
'create_lvm_physical_volume',
|
|
'create_lvm_volume_group',
|
|
'deactivate_lvm_volume_group',
|
|
'is_lvm_physical_volume',
|
|
'list_lvm_volume_group',
|
|
'relation_ids',
|
|
'relation_set',
|
|
'remove_lvm_physical_volume',
|
|
'ensure_loopback_device',
|
|
'is_block_device',
|
|
'zap_disk',
|
|
'os_release',
|
|
'get_os_codename_install_source',
|
|
'configure_installation_source',
|
|
'is_elected_leader',
|
|
'templating',
|
|
'install_alternative',
|
|
# fetch
|
|
'apt_update',
|
|
'apt_upgrade',
|
|
'apt_install',
|
|
'service_stop',
|
|
'service_start',
|
|
# cinder
|
|
'ceph_config_file',
|
|
]
|
|
|
|
|
|
MOUNTS = [
|
|
['/mnt', '/dev/fakevbd']
|
|
]
|
|
|
|
DPKG_OPTIONS = [
|
|
'--option', 'Dpkg::Options::=--force-confnew',
|
|
'--option', 'Dpkg::Options::=--force-confdef',
|
|
]
|
|
|
|
FDISKDISPLAY = """
|
|
Disk /dev/fakevbd doesn't contain a valid partition table
|
|
|
|
Disk /dev/fakevbd: 21.5 GB, 21474836480 bytes
|
|
16 heads, 63 sectors/track, 41610 cylinders, total 41943040 sectors
|
|
Units = sectors of 1 * 512 = 512 bytes
|
|
Sector size (logical/physical): 512 bytes / 512 bytes
|
|
I/O size (minimum/optimal): 512 bytes / 512 bytes
|
|
Disk identifier: 0x00000000
|
|
|
|
"""
|
|
|
|
openstack_origin_git = \
|
|
"""repositories:
|
|
- {name: requirements,
|
|
repository: 'git://git.openstack.org/openstack/requirements',
|
|
branch: stable/juno}
|
|
- {name: cinder,
|
|
repository: 'git://git.openstack.org/openstack/cinder',
|
|
branch: stable/juno}"""
|
|
|
|
|
|
class TestCinderUtils(CharmTestCase):
|
|
|
|
def setUp(self):
|
|
super(TestCinderUtils, self).setUp(cinder_utils, TO_PATCH)
|
|
self.config.side_effect = self.test_config.get_all
|
|
self.apache24_conf_dir = '/etc/apache2/conf-available'
|
|
self.charm_ceph_conf = '/var/lib/charm/cinder/ceph.conf'
|
|
self.ceph_conf = '/etc/ceph/ceph.conf'
|
|
self.cinder_conf = '/etc/cinder/cinder.conf'
|
|
|
|
def svc_enabled(self, svc):
|
|
return svc in self.test_config.get('enabled-services')
|
|
|
|
def test_all_services_enabled(self):
|
|
'It determines all services are enabled based on config'
|
|
self.test_config.set('enabled-services', 'all')
|
|
enabled = []
|
|
for s in ['volume', 'api', 'scheduler']:
|
|
enabled.append(cinder_utils.service_enabled(s))
|
|
self.assertEquals(enabled, [True, True, True])
|
|
|
|
def test_service_enabled(self):
|
|
'It determines services are enabled based on config'
|
|
self.test_config.set('enabled-services', 'api,volume,scheduler')
|
|
self.assertTrue(cinder_utils.service_enabled('volume'))
|
|
|
|
def test_service_not_enabled(self):
|
|
'It determines services are not enabled based on config'
|
|
self.test_config.set('enabled-services', 'api,scheduler')
|
|
self.assertFalse(cinder_utils.service_enabled('volume'))
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
@patch('cinder_utils.git_install_requested')
|
|
def test_determine_packages_all(self, git_requested, service_enabled):
|
|
'It determines all packages required when all services enabled'
|
|
git_requested.return_value = False
|
|
service_enabled.return_value = True
|
|
pkgs = cinder_utils.determine_packages()
|
|
self.assertEquals(sorted(pkgs),
|
|
sorted(cinder_utils.COMMON_PACKAGES +
|
|
cinder_utils.VOLUME_PACKAGES +
|
|
cinder_utils.API_PACKAGES +
|
|
cinder_utils.SCHEDULER_PACKAGES))
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
@patch('cinder_utils.git_install_requested')
|
|
def test_determine_packages_subset(self, git_requested, service_enabled):
|
|
'It determines packages required for a subset of enabled services'
|
|
git_requested.return_value = False
|
|
service_enabled.side_effect = self.svc_enabled
|
|
|
|
self.test_config.set('enabled-services', 'api')
|
|
pkgs = cinder_utils.determine_packages()
|
|
common = cinder_utils.COMMON_PACKAGES
|
|
self.assertEquals(sorted(pkgs),
|
|
sorted(common + cinder_utils.API_PACKAGES))
|
|
self.test_config.set('enabled-services', 'volume')
|
|
pkgs = cinder_utils.determine_packages()
|
|
common = cinder_utils.COMMON_PACKAGES
|
|
self.assertEquals(sorted(pkgs),
|
|
sorted(common + cinder_utils.VOLUME_PACKAGES))
|
|
self.test_config.set('enabled-services', 'api,scheduler')
|
|
pkgs = cinder_utils.determine_packages()
|
|
common = cinder_utils.COMMON_PACKAGES
|
|
self.assertEquals(sorted(pkgs),
|
|
sorted(common + cinder_utils.API_PACKAGES +
|
|
cinder_utils.SCHEDULER_PACKAGES))
|
|
|
|
@patch('cinder_utils.restart_map')
|
|
def test_services(self, restart_map):
|
|
restart_map.return_value = OrderedDict([
|
|
('test_conf1', ['svc1']),
|
|
('test_conf2', ['svc2', 'svc3', 'svc1']),
|
|
])
|
|
self.assertEquals(cinder_utils.services(), ['svc2', 'svc3', 'svc1'])
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
@patch('os.path.exists')
|
|
def test_creates_resource_map_all_enabled(self, path_exists,
|
|
service_enabled):
|
|
service_enabled.return_value = True
|
|
path_exists.return_value = True
|
|
self.ceph_config_file.return_value = self.charm_ceph_conf
|
|
self.relation_ids.return_value = []
|
|
ex_map = OrderedDict([
|
|
('/etc/cinder/cinder.conf', ['cinder-api', 'cinder-volume',
|
|
'cinder-scheduler', 'haproxy']),
|
|
('/etc/cinder/api-paste.ini', ['cinder-api']),
|
|
('/etc/haproxy/haproxy.cfg', ['haproxy']),
|
|
('/etc/apache2/sites-available/openstack_https_frontend.conf',
|
|
['apache2']),
|
|
])
|
|
for cfg in ex_map.keys():
|
|
self.assertEquals(cinder_utils.resource_map()[cfg]['services'],
|
|
ex_map[cfg])
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
@patch('os.path.exists')
|
|
def test_creates_resource_map_no_api(self, path_exists,
|
|
service_enabled):
|
|
service_enabled.side_effect = self.svc_enabled
|
|
self.test_config.set('enabled-services', 'scheduler,volume')
|
|
path_exists.return_value = True
|
|
self.ceph_config_file.return_value = self.charm_ceph_conf
|
|
self.relation_ids.return_value = []
|
|
ex_map = OrderedDict([
|
|
('/etc/cinder/cinder.conf', ['cinder-volume', 'cinder-scheduler',
|
|
'haproxy']),
|
|
('/etc/cinder/api-paste.ini', []),
|
|
('/etc/haproxy/haproxy.cfg', ['haproxy']),
|
|
('/etc/apache2/sites-available/openstack_https_frontend.conf',
|
|
['apache2']),
|
|
])
|
|
for cfg in ex_map.keys():
|
|
self.assertEquals(cinder_utils.resource_map()[cfg]['services'],
|
|
ex_map[cfg])
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
@patch('os.path.exists')
|
|
def test_creates_resource_map_backup_backend(self, path_exists,
|
|
service_enabled):
|
|
service_enabled.return_value = True
|
|
path_exists.return_value = True
|
|
self.ceph_config_file.return_value = self.charm_ceph_conf
|
|
self.relation_ids.side_effect = lambda x: {
|
|
'storage-backend': [],
|
|
'backup-backend': ['rid1'],
|
|
'ceph': []}[x]
|
|
self.assertTrue(
|
|
'cinder-backup' in
|
|
cinder_utils.resource_map()[self.cinder_conf]['services'])
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
@patch('os.path.exists')
|
|
def test_creates_resource_map_no_backup(self, path_exists,
|
|
service_enabled):
|
|
service_enabled.return_value = True
|
|
path_exists.return_value = True
|
|
self.ceph_config_file.return_value = self.charm_ceph_conf
|
|
self.relation_ids.side_effect = lambda x: {
|
|
'storage-backend': [],
|
|
'backup-backend': [],
|
|
'ceph': []}[x]
|
|
self.assertFalse(
|
|
'cinder-backup' in
|
|
cinder_utils.resource_map()[self.cinder_conf]['services'])
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
@patch('os.path.exists')
|
|
def test_creates_resource_map_no_ceph_conf(self, path_exists,
|
|
service_enabled):
|
|
service_enabled.return_value = True
|
|
path_exists.return_value = True
|
|
self.ceph_config_file.return_value = self.charm_ceph_conf
|
|
self.relation_ids.side_effect = lambda x: {
|
|
'storage-backend': [],
|
|
'backup-backend': [],
|
|
'ceph': []}[x]
|
|
self.assertFalse(self.charm_ceph_conf in
|
|
cinder_utils.resource_map().keys())
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
@patch('os.path.exists')
|
|
def test_creates_resource_map_ceph_conf(self, path_exists,
|
|
service_enabled):
|
|
service_enabled.return_value = True
|
|
path_exists.return_value = True
|
|
self.ceph_config_file.return_value = self.charm_ceph_conf
|
|
self.relation_ids.side_effect = lambda x: {
|
|
'storage-backend': [],
|
|
'backup-backend': [],
|
|
'ceph': ['rid1']}[x]
|
|
self.assertTrue(self.charm_ceph_conf in
|
|
cinder_utils.resource_map().keys())
|
|
self.mkdir.assert_has_calls(
|
|
[call('/etc/ceph'),
|
|
call('/var/lib/charm/cinder')]
|
|
)
|
|
self.install_alternative.assert_called_with(
|
|
'ceph.conf',
|
|
'/etc/ceph/ceph.conf',
|
|
self.charm_ceph_conf)
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
@patch('os.path.exists')
|
|
def test_creates_resource_map_old_apache(self, path_exists,
|
|
service_enabled):
|
|
service_enabled.return_value = True
|
|
path_exists.side_effect = lambda x: x not in [self.apache24_conf_dir]
|
|
self.ceph_config_file.return_value = self.charm_ceph_conf
|
|
self.relation_ids.side_effect = lambda x: {
|
|
'storage-backend': [],
|
|
'backup-backend': [],
|
|
'ceph': []}[x]
|
|
self.assertTrue(
|
|
'/etc/apache2/sites-available/openstack_https_frontend' in
|
|
cinder_utils.resource_map().keys())
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
@patch('os.path.exists')
|
|
def test_creates_resource_map_apache24(self, path_exists, service_enabled):
|
|
service_enabled.return_value = True
|
|
path_exists.side_effect = lambda x: x in [self.apache24_conf_dir]
|
|
self.ceph_config_file.return_value = self.charm_ceph_conf
|
|
self.relation_ids.side_effect = lambda x: {
|
|
'storage-backend': [],
|
|
'backup-backend': [],
|
|
'ceph': []}[x]
|
|
self.assertTrue(
|
|
'/etc/apache2/sites-available/openstack_https_frontend.conf' in
|
|
cinder_utils.resource_map().keys())
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
def test_filter_services_selective(self, service_enabled):
|
|
service_enabled.side_effect = self.svc_enabled
|
|
self.test_config.set('enabled-services', 'scheduler,volume')
|
|
self.assertEqual(
|
|
cinder_utils.filter_services(['cinder-api', 'cinder-volume',
|
|
'haproxy']),
|
|
['cinder-volume', 'haproxy']
|
|
)
|
|
|
|
@patch('cinder_utils.service_enabled')
|
|
def test_filter_services_all(self, service_enabled):
|
|
service_enabled.return_value = True
|
|
self.test_config.set('enabled-services', 'scheduler,volume')
|
|
self.assertEqual(
|
|
cinder_utils.filter_services(['cinder-api', 'cinder-volume',
|
|
'haproxy']),
|
|
['cinder-api', 'cinder-volume', 'haproxy']
|
|
)
|
|
|
|
@patch('cinder_utils.resource_map')
|
|
def test_restart_map(self, resource_map):
|
|
resource_map.return_value = OrderedDict([
|
|
('/etc/testfile1.conf', {
|
|
'hook_contexts': ['dummyctxt1', 'dummyctxt2'],
|
|
'services': ['svc1'],
|
|
}),
|
|
('/etc/testfile2.conf', {
|
|
'hook_contexts': ['dummyctxt1', 'dummyctxt3'],
|
|
'services': [],
|
|
}),
|
|
])
|
|
ex_map = OrderedDict([
|
|
('/etc/testfile1.conf', ['svc1']),
|
|
])
|
|
self.assertEquals(cinder_utils.restart_map(), ex_map)
|
|
|
|
def test_clean_storage_unmount(self):
|
|
'It unmounts block device when cleaning storage'
|
|
self.is_lvm_physical_volume.return_value = False
|
|
self.zap_disk.return_value = True
|
|
self.mounts.return_value = MOUNTS
|
|
cinder_utils.clean_storage('/dev/fakevbd')
|
|
self.umount.called_with('/dev/fakevbd', True)
|
|
|
|
def test_clean_storage_lvm_wipe(self):
|
|
'It removes traces of LVM when cleaning storage'
|
|
self.mounts.return_value = []
|
|
self.is_lvm_physical_volume.return_value = True
|
|
cinder_utils.clean_storage('/dev/fakevbd')
|
|
self.remove_lvm_physical_volume.assert_called_with('/dev/fakevbd')
|
|
self.deactivate_lvm_volume_group.assert_called_with('/dev/fakevbd')
|
|
self.zap_disk.assert_called_with('/dev/fakevbd')
|
|
|
|
def test_clean_storage_zap_disk(self):
|
|
'It removes traces of LVM when cleaning storage'
|
|
self.mounts.return_value = []
|
|
self.is_lvm_physical_volume.return_value = False
|
|
cinder_utils.clean_storage('/dev/fakevbd')
|
|
self.zap_disk.assert_called_with('/dev/fakevbd')
|
|
|
|
def test_parse_block_device(self):
|
|
self.assertTrue(cinder_utils._parse_block_device(None),
|
|
(None, 0))
|
|
self.assertTrue(cinder_utils._parse_block_device('fakevdc'),
|
|
('/dev/fakevdc', 0))
|
|
self.assertTrue(cinder_utils._parse_block_device('/dev/fakevdc'),
|
|
('/dev/fakevdc', 0))
|
|
self.assertTrue(cinder_utils._parse_block_device('/dev/fakevdc'),
|
|
('/dev/fakevdc', 0))
|
|
self.assertTrue(cinder_utils._parse_block_device('/mnt/loop0|10'),
|
|
('/mnt/loop0', 10))
|
|
self.assertTrue(cinder_utils._parse_block_device('/mnt/loop0'),
|
|
('/mnt/loop0', cinder_utils.DEFAULT_LOOPBACK_SIZE))
|
|
|
|
@patch('subprocess.check_output')
|
|
def test_has_partition_table(self, _check):
|
|
_check.return_value = FDISKDISPLAY
|
|
block_device = '/dev/fakevbd'
|
|
cinder_utils.has_partition_table(block_device)
|
|
_check.assert_called_with(['fdisk', '-l', '/dev/fakevbd'], stderr=-2)
|
|
|
|
@patch('cinder_utils.log_lvm_info', Mock())
|
|
@patch.object(cinder_utils, 'ensure_lvm_volume_group_non_existent')
|
|
@patch.object(cinder_utils, 'clean_storage')
|
|
@patch.object(cinder_utils, 'reduce_lvm_volume_group_missing')
|
|
@patch.object(cinder_utils, 'extend_lvm_volume_group')
|
|
def test_configure_lvm_storage(self, extend_lvm, reduce_lvm,
|
|
clean_storage, ensure_non_existent):
|
|
devices = ['/dev/fakevbd', '/dev/fakevdc']
|
|
self.is_lvm_physical_volume.return_value = False
|
|
cinder_utils.configure_lvm_storage(devices, 'test', True, True)
|
|
clean_storage.assert_has_calls(
|
|
[call('/dev/fakevbd'),
|
|
call('/dev/fakevdc')]
|
|
)
|
|
self.create_lvm_physical_volume.assert_has_calls(
|
|
[call('/dev/fakevbd'),
|
|
call('/dev/fakevdc')]
|
|
)
|
|
self.create_lvm_volume_group.assert_called_with('test', '/dev/fakevbd')
|
|
reduce_lvm.assert_called_with('test')
|
|
extend_lvm.assert_called_with('test', '/dev/fakevdc')
|
|
ensure_non_existent.assert_called_with('test')
|
|
|
|
@patch('cinder_utils.log_lvm_info', Mock())
|
|
@patch.object(cinder_utils, 'has_partition_table')
|
|
@patch.object(cinder_utils, 'clean_storage')
|
|
@patch.object(cinder_utils, 'reduce_lvm_volume_group_missing')
|
|
@patch.object(cinder_utils, 'extend_lvm_volume_group')
|
|
def test_configure_lvm_storage_unused_dev(self, extend_lvm, reduce_lvm,
|
|
clean_storage, has_part):
|
|
devices = ['/dev/fakevbd', '/dev/fakevdc']
|
|
self.is_lvm_physical_volume.return_value = False
|
|
has_part.return_value = False
|
|
cinder_utils.configure_lvm_storage(devices, 'test', False, True)
|
|
clean_storage.assert_has_calls(
|
|
[call('/dev/fakevbd'),
|
|
call('/dev/fakevdc')]
|
|
)
|
|
self.create_lvm_physical_volume.assert_has_calls(
|
|
[call('/dev/fakevbd'),
|
|
call('/dev/fakevdc')]
|
|
)
|
|
self.create_lvm_volume_group.assert_called_with('test', '/dev/fakevbd')
|
|
reduce_lvm.assert_called_with('test')
|
|
extend_lvm.assert_called_with('test', '/dev/fakevdc')
|
|
|
|
@patch('cinder_utils.log_lvm_info', Mock())
|
|
@patch.object(cinder_utils, 'has_partition_table')
|
|
@patch.object(cinder_utils, 'reduce_lvm_volume_group_missing')
|
|
def test_configure_lvm_storage_used_dev(self, reduce_lvm, has_part):
|
|
devices = ['/dev/fakevbd', '/dev/fakevdc']
|
|
self.is_lvm_physical_volume.return_value = False
|
|
has_part.return_value = True
|
|
cinder_utils.configure_lvm_storage(devices, 'test', False, True)
|
|
reduce_lvm.assert_called_with('test')
|
|
|
|
@patch('cinder_utils.log_lvm_info', Mock())
|
|
@patch.object(cinder_utils, 'ensure_lvm_volume_group_non_existent')
|
|
@patch.object(cinder_utils, 'clean_storage')
|
|
@patch.object(cinder_utils, 'reduce_lvm_volume_group_missing')
|
|
@patch.object(cinder_utils, 'extend_lvm_volume_group')
|
|
def test_configure_lvm_storage_loopback(self, extend_lvm, reduce_lvm,
|
|
clean_storage,
|
|
ensure_non_existent):
|
|
devices = ['/mnt/loop0|10']
|
|
self.ensure_loopback_device.return_value = '/dev/loop0'
|
|
self.is_lvm_physical_volume.return_value = False
|
|
cinder_utils.configure_lvm_storage(devices, 'test', True, True)
|
|
clean_storage.assert_called_with('/dev/loop0')
|
|
self.ensure_loopback_device.assert_called_with('/mnt/loop0', '10')
|
|
self.create_lvm_physical_volume.assert_called_with('/dev/loop0')
|
|
self.create_lvm_volume_group.assert_called_with('test', '/dev/loop0')
|
|
reduce_lvm.assert_called_with('test')
|
|
self.assertFalse(extend_lvm.called)
|
|
ensure_non_existent.assert_called_with('test')
|
|
|
|
@patch.object(cinder_utils, 'lvm_volume_group_exists')
|
|
@patch('cinder_utils.log_lvm_info', Mock())
|
|
@patch.object(cinder_utils, 'clean_storage')
|
|
@patch.object(cinder_utils, 'reduce_lvm_volume_group_missing')
|
|
@patch.object(cinder_utils, 'extend_lvm_volume_group')
|
|
def test_configure_lvm_storage_existing_vg(self, extend_lvm, reduce_lvm,
|
|
clean_storage, lvm_exists):
|
|
def pv_lookup(device):
|
|
devices = {
|
|
'/dev/fakevbd': True,
|
|
'/dev/fakevdc': False
|
|
}
|
|
return devices[device]
|
|
|
|
def vg_lookup(device):
|
|
devices = {
|
|
'/dev/fakevbd': 'test',
|
|
'/dev/fakevdc': None
|
|
}
|
|
return devices[device]
|
|
devices = ['/dev/fakevbd', '/dev/fakevdc']
|
|
lvm_exists.return_value = False
|
|
self.is_lvm_physical_volume.side_effect = pv_lookup
|
|
self.list_lvm_volume_group.side_effect = vg_lookup
|
|
cinder_utils.configure_lvm_storage(devices, 'test', True, True)
|
|
clean_storage.assert_has_calls(
|
|
[call('/dev/fakevdc')]
|
|
)
|
|
self.create_lvm_physical_volume.assert_has_calls(
|
|
[call('/dev/fakevdc')]
|
|
)
|
|
reduce_lvm.assert_called_with('test')
|
|
extend_lvm.assert_called_with('test', '/dev/fakevdc')
|
|
self.assertFalse(self.create_lvm_volume_group.called)
|
|
|
|
@patch.object(cinder_utils, 'lvm_volume_group_exists')
|
|
@patch('cinder_utils.log_lvm_info', Mock())
|
|
@patch.object(cinder_utils, 'clean_storage')
|
|
@patch.object(cinder_utils, 'reduce_lvm_volume_group_missing')
|
|
@patch.object(cinder_utils, 'extend_lvm_volume_group')
|
|
def test_configure_lvm_storage_different_vg(self, extend_lvm, reduce_lvm,
|
|
clean_storage, lvm_exists):
|
|
def pv_lookup(device):
|
|
devices = {
|
|
'/dev/fakevbd': True,
|
|
'/dev/fakevdc': True
|
|
}
|
|
return devices[device]
|
|
|
|
def vg_lookup(device):
|
|
devices = {
|
|
'/dev/fakevbd': 'test',
|
|
'/dev/fakevdc': 'another'
|
|
}
|
|
return devices[device]
|
|
devices = ['/dev/fakevbd', '/dev/fakevdc']
|
|
self.is_lvm_physical_volume.side_effect = pv_lookup
|
|
self.list_lvm_volume_group.side_effect = vg_lookup
|
|
lvm_exists.return_value = False
|
|
cinder_utils.configure_lvm_storage(devices, 'test', True, True)
|
|
clean_storage.assert_called_with('/dev/fakevdc')
|
|
self.create_lvm_physical_volume.assert_called_with('/dev/fakevdc')
|
|
reduce_lvm.assert_called_with('test')
|
|
extend_lvm.assert_called_with('test', '/dev/fakevdc')
|
|
self.assertFalse(self.create_lvm_volume_group.called)
|
|
|
|
@patch('cinder_utils.log_lvm_info', Mock())
|
|
@patch.object(cinder_utils, 'clean_storage')
|
|
@patch.object(cinder_utils, 'reduce_lvm_volume_group_missing')
|
|
@patch.object(cinder_utils, 'extend_lvm_volume_group')
|
|
def test_configure_lvm_storage_different_vg_ignore(self, extend_lvm,
|
|
reduce_lvm,
|
|
clean_storage):
|
|
def pv_lookup(device):
|
|
devices = {
|
|
'/dev/fakevbd': True,
|
|
'/dev/fakevdc': True
|
|
}
|
|
return devices[device]
|
|
|
|
def vg_lookup(device):
|
|
devices = {
|
|
'/dev/fakevbd': 'test',
|
|
'/dev/fakevdc': 'another'
|
|
}
|
|
return devices[device]
|
|
devices = ['/dev/fakevbd', '/dev/fakevdc']
|
|
self.is_lvm_physical_volume.side_effect = pv_lookup
|
|
self.list_lvm_volume_group.side_effect = vg_lookup
|
|
cinder_utils.configure_lvm_storage(devices, 'test', False, False)
|
|
self.assertFalse(clean_storage.called)
|
|
self.assertFalse(self.create_lvm_physical_volume.called)
|
|
self.assertFalse(reduce_lvm.called)
|
|
self.assertFalse(extend_lvm.called)
|
|
self.assertFalse(self.create_lvm_volume_group.called)
|
|
|
|
@patch('cinder_utils.log_lvm_info', Mock())
|
|
@patch.object(cinder_utils, 'reduce_lvm_volume_group_missing')
|
|
def test_configure_lvm_storage_unforced_remove_default(self, reduce_lvm):
|
|
"""It doesn't force remove missing by default."""
|
|
devices = ['/dev/fakevbd']
|
|
cinder_utils.configure_lvm_storage(devices, 'test', False, True)
|
|
reduce_lvm.assert_called_with('test')
|
|
|
|
@patch('cinder_utils.log_lvm_info', Mock())
|
|
@patch.object(cinder_utils, 'reduce_lvm_volume_group_missing')
|
|
def test_configure_lvm_storage_force_removemissing(self, reduce_lvm):
|
|
"""It forces remove missing when asked to."""
|
|
devices = ['/dev/fakevbd']
|
|
cinder_utils.configure_lvm_storage(
|
|
devices, 'test', False, True, remove_missing_force=True)
|
|
reduce_lvm.assert_called_with('test', extra_args=['--force'])
|
|
|
|
@patch('subprocess.check_call')
|
|
def test_reduce_lvm_volume_group_missing(self, _call):
|
|
cinder_utils.reduce_lvm_volume_group_missing('test')
|
|
_call.assert_called_with(['vgreduce', '--removemissing', 'test'])
|
|
|
|
@patch('subprocess.check_call')
|
|
def test_reduce_lvm_volume_group_missing_extra_args(self, _call):
|
|
cinder_utils.reduce_lvm_volume_group_missing(
|
|
'test', extra_args=['--arg'])
|
|
expected_call_args = ['vgreduce', '--removemissing', '--arg', 'test']
|
|
_call.assert_called_with(expected_call_args)
|
|
|
|
@patch('subprocess.check_call')
|
|
def test_extend_lvm_volume_group(self, _call):
|
|
cinder_utils.extend_lvm_volume_group('test', '/dev/sdb')
|
|
_call.assert_called_with(['vgextend', 'test', '/dev/sdb'])
|
|
|
|
@patch.object(cinder_utils, 'enabled_services')
|
|
@patch.object(cinder_utils, 'local_unit', lambda *args: 'unit/0')
|
|
@patch.object(cinder_utils, 'uuid')
|
|
def test_migrate_database(self, mock_uuid, mock_enabled_services):
|
|
'It migrates database with cinder-manage'
|
|
uuid = 'a-great-uuid'
|
|
mock_uuid.uuid4.return_value = uuid
|
|
mock_enabled_services.return_value = ['svc1']
|
|
rid = 'cluster:0'
|
|
self.relation_ids.return_value = [rid]
|
|
args = {'cinder-db-initialised': "unit/0-%s" % uuid}
|
|
with patch('subprocess.check_call') as check_call:
|
|
cinder_utils.migrate_database()
|
|
check_call.assert_called_with(['cinder-manage', 'db', 'sync'])
|
|
self.relation_set.assert_called_with(relation_id=rid, **args)
|
|
self.service_restart.assert_called_with('svc1')
|
|
|
|
@patch.object(cinder_utils, 'resource_map')
|
|
def test_register_configs(self, resource_map):
|
|
resource_map.return_value = OrderedDict([
|
|
('/etc/testfile1.conf', {
|
|
'contexts': ['dummyctxt1', 'dummyctxt2'],
|
|
'services': ['svc1'],
|
|
}),
|
|
('/etc/testfile2.conf', {
|
|
'contexts': ['dummyctxt1', 'dummyctxt3'],
|
|
'services': [],
|
|
}),
|
|
])
|
|
configs = cinder_utils.register_configs()
|
|
calls = [
|
|
call('/etc/testfile1.conf', ['dummyctxt1', 'dummyctxt2']),
|
|
call('/etc/testfile2.conf', ['dummyctxt1', 'dummyctxt3']),
|
|
]
|
|
configs.register.assert_has_calls(calls)
|
|
|
|
def test_set_ceph_kludge(self):
|
|
pass
|
|
"""
|
|
def set_ceph_env_variables(service):
|
|
# XXX: Horrid kludge to make cinder-volume use
|
|
# a different ceph username than admin
|
|
env = open('/etc/environment', 'r').read()
|
|
if 'CEPH_ARGS' not in env:
|
|
with open('/etc/environment', 'a') as out:
|
|
out.write('CEPH_ARGS="--id %s"\n' % service)
|
|
with open('/etc/init/cinder-volume.override', 'w') as out:
|
|
out.write('env CEPH_ARGS="--id %s"\n' % service)
|
|
"""
|
|
|
|
@patch.object(cinder_utils, 'services')
|
|
@patch.object(cinder_utils, 'migrate_database')
|
|
@patch.object(cinder_utils, 'determine_packages')
|
|
def test_openstack_upgrade_leader(self, pkgs, migrate, services):
|
|
pkgs.return_value = ['mypackage']
|
|
self.config.side_effect = None
|
|
self.config.return_value = 'cloud:precise-havana'
|
|
services.return_value = ['cinder-api', 'cinder-volume']
|
|
self.is_elected_leader.return_value = True
|
|
self.get_os_codename_install_source.return_value = 'havana'
|
|
configs = MagicMock()
|
|
cinder_utils.do_openstack_upgrade(configs)
|
|
self.assertTrue(configs.write_all.called)
|
|
self.apt_upgrade.assert_called_with(options=DPKG_OPTIONS,
|
|
fatal=True, dist=True)
|
|
self.apt_install.assert_called_with(['mypackage'], fatal=True)
|
|
configs.set_release.assert_called_with(openstack_release='havana')
|
|
self.assertTrue(migrate.called)
|
|
|
|
@patch.object(cinder_utils, 'services')
|
|
@patch.object(cinder_utils, 'migrate_database')
|
|
@patch.object(cinder_utils, 'determine_packages')
|
|
def test_openstack_upgrade_not_leader(self, pkgs, migrate, services):
|
|
pkgs.return_value = ['mypackage']
|
|
self.config.side_effect = None
|
|
self.config.return_value = 'cloud:precise-havana'
|
|
services.return_value = ['cinder-api', 'cinder-volume']
|
|
self.is_elected_leader.return_value = False
|
|
self.get_os_codename_install_source.return_value = 'havana'
|
|
configs = MagicMock()
|
|
cinder_utils.do_openstack_upgrade(configs)
|
|
self.assertTrue(configs.write_all.called)
|
|
self.apt_upgrade.assert_called_with(options=DPKG_OPTIONS,
|
|
fatal=True, dist=True)
|
|
self.apt_install.assert_called_with(['mypackage'], fatal=True)
|
|
configs.set_release.assert_called_with(openstack_release='havana')
|
|
self.assertFalse(migrate.called)
|
|
|
|
@patch.object(cinder_utils, 'git_install_requested')
|
|
@patch.object(cinder_utils, 'git_clone_and_install')
|
|
@patch.object(cinder_utils, 'git_post_install')
|
|
@patch.object(cinder_utils, 'git_pre_install')
|
|
def test_git_install(self, git_pre, git_post, git_clone_and_install,
|
|
git_requested):
|
|
projects_yaml = openstack_origin_git
|
|
git_requested.return_value = True
|
|
cinder_utils.git_install(projects_yaml)
|
|
self.assertTrue(git_pre.called)
|
|
git_clone_and_install.assert_called_with(openstack_origin_git,
|
|
core_project='cinder')
|
|
self.assertTrue(git_post.called)
|
|
|
|
@patch.object(cinder_utils, 'mkdir')
|
|
@patch.object(cinder_utils, 'write_file')
|
|
@patch.object(cinder_utils, 'add_user_to_group')
|
|
@patch.object(cinder_utils, 'add_group')
|
|
@patch.object(cinder_utils, 'adduser')
|
|
def test_git_pre_install(self, adduser, add_group, add_user_to_group,
|
|
write_file, mkdir):
|
|
cinder_utils.git_pre_install()
|
|
adduser.assert_called_with('cinder', shell='/bin/bash',
|
|
system_user=True)
|
|
add_group.assert_called_with('cinder', system_group=True)
|
|
add_user_to_group.assert_called_with('cinder', 'cinder')
|
|
expected = [
|
|
call('/etc/tgt', owner='cinder', perms=488, force=False,
|
|
group='cinder'),
|
|
call('/var/lib/cinder', owner='cinder', perms=493, force=False,
|
|
group='cinder'),
|
|
call('/var/lib/cinder/volumes', owner='cinder', perms=488,
|
|
force=False, group='cinder'),
|
|
call('/var/lock/cinder', owner='cinder', perms=488, force=False,
|
|
group='root'),
|
|
call('/var/log/cinder', owner='cinder', perms=488, force=False,
|
|
group='cinder'),
|
|
]
|
|
self.assertEquals(mkdir.call_args_list, expected)
|
|
expected = [
|
|
call('/var/log/cinder/cinder-api.log', '', perms=0600,
|
|
owner='cinder', group='cinder'),
|
|
call('/var/log/cinder/cinder-backup.log', '', perms=0600,
|
|
owner='cinder', group='cinder'),
|
|
call('/var/log/cinder/cinder-scheduler.log', '', perms=0600,
|
|
owner='cinder', group='cinder'),
|
|
call('/var/log/cinder/cinder-volume.log', '', perms=0600,
|
|
owner='cinder', group='cinder'),
|
|
]
|
|
self.assertEquals(write_file.call_args_list, expected)
|
|
|
|
@patch.object(cinder_utils, 'services')
|
|
@patch('os.path.join')
|
|
@patch('os.path.exists')
|
|
@patch('shutil.copytree')
|
|
@patch('shutil.rmtree')
|
|
@patch('os.chown')
|
|
@patch('os.chmod')
|
|
@patch('os.symlink')
|
|
def test_git_post_install_upstart(self, symlink, chmod, chown, rmtree,
|
|
copytree, exists, join, services):
|
|
services.return_value = ['svc1']
|
|
projects_yaml = openstack_origin_git
|
|
join.return_value = 'joined-string'
|
|
self.lsb_release.return_value = {'DISTRIB_RELEASE': '15.04'}
|
|
cinder_utils.git_post_install(projects_yaml)
|
|
self.pip_install('mysql-python', venv='joined-string')
|
|
expected = [
|
|
call('joined-string', '/etc/cinder'),
|
|
]
|
|
copytree.assert_has_calls(expected)
|
|
|
|
expected = [
|
|
call('joined-string', '/usr/local/bin/cinder-manage'),
|
|
call('/usr/lib/python2.7/dist-packages/rbd.py', 'joined-string'),
|
|
call('/usr/lib/python2.7/dist-packages/rados.py', 'joined-string'),
|
|
]
|
|
symlink.assert_has_calls(expected, any_order=True)
|
|
|
|
cinder_api_context = {
|
|
'service_description': 'Cinder API server',
|
|
'service_name': 'Cinder',
|
|
'user_name': 'cinder',
|
|
'start_dir': '/var/lib/cinder',
|
|
'process_name': 'cinder-api',
|
|
'executable_name': 'joined-string',
|
|
'config_files': ['/etc/cinder/cinder.conf'],
|
|
'log_file': '/var/log/cinder/cinder-api.log',
|
|
}
|
|
|
|
cinder_backup_context = {
|
|
'service_description': 'Cinder backup server',
|
|
'service_name': 'Cinder',
|
|
'user_name': 'cinder',
|
|
'start_dir': '/var/lib/cinder',
|
|
'process_name': 'cinder-backup',
|
|
'executable_name': 'joined-string',
|
|
'config_files': ['/etc/cinder/cinder.conf'],
|
|
'log_file': '/var/log/cinder/cinder-backup.log',
|
|
}
|
|
|
|
cinder_scheduler_context = {
|
|
'service_description': 'Cinder scheduler server',
|
|
'service_name': 'Cinder',
|
|
'user_name': 'cinder',
|
|
'start_dir': '/var/lib/cinder',
|
|
'process_name': 'cinder-scheduler',
|
|
'executable_name': 'joined-string',
|
|
'config_files': ['/etc/cinder/cinder.conf'],
|
|
'log_file': '/var/log/cinder/cinder-scheduler.log',
|
|
}
|
|
|
|
cinder_volume_context = {
|
|
'service_description': 'Cinder volume server',
|
|
'service_name': 'Cinder',
|
|
'user_name': 'cinder',
|
|
'start_dir': '/var/lib/cinder',
|
|
'process_name': 'cinder-volume',
|
|
'executable_name': 'joined-string',
|
|
'config_files': ['/etc/cinder/cinder.conf'],
|
|
'log_file': '/var/log/cinder/cinder-volume.log',
|
|
}
|
|
expected = [
|
|
call('git/cinder_tgt.conf', '/etc/tgt/conf.d', {}, owner='cinder',
|
|
group='cinder', perms=0o644),
|
|
call('git/logging.conf', '/etc/cinder/logging.conf', {},
|
|
owner='cinder', group='cinder', perms=0o644),
|
|
call('git/cinder_sudoers', '/etc/sudoers.d/cinder_sudoers', {},
|
|
owner='root', group='root', perms=0o440),
|
|
call('git.upstart', '/etc/init/cinder-api.conf',
|
|
cinder_api_context, perms=0o644,
|
|
templates_dir='joined-string'),
|
|
call('git.upstart', '/etc/init/cinder-backup.conf',
|
|
cinder_backup_context, perms=0o644,
|
|
templates_dir='joined-string'),
|
|
call('git.upstart', '/etc/init/cinder-scheduler.conf',
|
|
cinder_scheduler_context, perms=0o644,
|
|
templates_dir='joined-string'),
|
|
call('git.upstart', '/etc/init/cinder-volume.conf',
|
|
cinder_volume_context, perms=0o644,
|
|
templates_dir='joined-string'),
|
|
]
|
|
self.assertEquals(self.render.call_args_list, expected)
|
|
expected = [call('tgtd'), call('svc1')]
|
|
self.assertEquals(self.service_restart.call_args_list, expected)
|
|
|
|
@patch.object(cinder_utils, 'services')
|
|
@patch('os.path.join')
|
|
@patch('shutil.copytree')
|
|
@patch('shutil.rmtree')
|
|
@patch('pwd.getpwnam')
|
|
@patch('grp.getgrnam')
|
|
@patch('os.chown')
|
|
@patch('os.chmod')
|
|
@patch('os.symlink')
|
|
def test_git_post_install_systemd(self, symlink, chmod, chown, grp, pwd,
|
|
rmtree, copytree, join, services):
|
|
projects_yaml = openstack_origin_git
|
|
join.return_value = 'joined-string'
|
|
self.lsb_release.return_value = {'DISTRIB_RELEASE': '15.10'}
|
|
cinder_utils.git_post_install(projects_yaml)
|
|
|
|
expected = [
|
|
call('git/cinder_tgt.conf', '/etc/tgt/conf.d', {},
|
|
group='cinder', owner='cinder', perms=420),
|
|
call('git/logging.conf', '/etc/cinder/logging.conf', {},
|
|
group='cinder', owner='cinder', perms=420),
|
|
call('git/cinder_sudoers', '/etc/sudoers.d/cinder_sudoers', {},
|
|
group='root', owner='root', perms=288),
|
|
call('git/cinder-api.init.in.template', 'joined-string',
|
|
{'daemon_path': 'joined-string'}, perms=420),
|
|
call('git/cinder-backup.init.in.template', 'joined-string',
|
|
{'daemon_path': 'joined-string'}, perms=420),
|
|
call('git/cinder-scheduler.init.in.template', 'joined-string',
|
|
{'daemon_path': 'joined-string'}, perms=420),
|
|
call('git/cinder-volume.init.in.template', 'joined-string',
|
|
{'daemon_path': 'joined-string'}, perms=420),
|
|
]
|
|
self.assertEquals(self.render.call_args_list, expected)
|
|
|
|
@patch.object(cinder_utils, 'local_unit', lambda *args: 'unit/0')
|
|
def test_check_db_initialised_by_self(self):
|
|
self.relation_get.return_value = {}
|
|
cinder_utils.check_db_initialised()
|
|
self.assertFalse(self.relation_set.called)
|
|
|
|
self.relation_get.return_value = {'cinder-db-initialised':
|
|
'unit/0-1234'}
|
|
cinder_utils.check_db_initialised()
|
|
self.assertFalse(self.relation_set.called)
|
|
|
|
@patch.object(cinder_utils, 'enabled_services')
|
|
@patch.object(cinder_utils, 'local_unit', lambda *args: 'unit/0')
|
|
def test_check_db_initialised(self, enabled_services):
|
|
enabled_services.return_value = ['svc1']
|
|
self.relation_get.return_value = {}
|
|
cinder_utils.check_db_initialised()
|
|
self.assertFalse(self.relation_set.called)
|
|
|
|
self.relation_get.return_value = {'cinder-db-initialised':
|
|
'unit/1-1234'}
|
|
cinder_utils.check_db_initialised()
|
|
calls = [call(**{'cinder-db-initialised-echo': 'unit/1-1234'})]
|
|
self.relation_set.assert_has_calls(calls)
|
|
self.service_restart.assert_called_with('svc1')
|
|
|
|
@patch('subprocess.check_output')
|
|
def test_log_lvm_info(self, _check):
|
|
output = "some output"
|
|
_check.return_value = output
|
|
cinder_utils.log_lvm_info()
|
|
_check.assert_called_with(['pvscan'])
|
|
self.juju_log.assert_called_with("pvscan: %s" % output)
|
|
|
|
@patch.object(cinder_utils, 'lvm_volume_group_exists')
|
|
@patch.object(cinder_utils, 'remove_lvm_volume_group')
|
|
def test_ensure_non_existent_removes_if_present(self,
|
|
remove_lvm_volume_group,
|
|
volume_group_exists):
|
|
volume_group = "test"
|
|
volume_group_exists.return_value = True
|
|
cinder_utils.ensure_lvm_volume_group_non_existent(volume_group)
|
|
remove_lvm_volume_group.assert_called_with(volume_group)
|
|
|
|
@patch.object(cinder_utils, 'lvm_volume_group_exists')
|
|
@patch.object(cinder_utils, 'remove_lvm_volume_group')
|
|
def test_ensure_non_existent_not_present(self, remove_lvm_volume_group,
|
|
volume_group_exists):
|
|
volume_group = "test"
|
|
volume_group_exists.return_value = False
|
|
cinder_utils.ensure_lvm_volume_group_non_existent(volume_group)
|
|
self.assertFalse(remove_lvm_volume_group.called)
|
|
|
|
@patch('subprocess.check_call')
|
|
def test_lvm_volume_group_exists_finds_volume_group(self, _check):
|
|
volume_group = "test"
|
|
_check.return_value = True
|
|
result = cinder_utils.lvm_volume_group_exists(volume_group)
|
|
self.assertTrue(result)
|
|
_check.assert_called_with(['vgdisplay', volume_group])
|
|
|
|
@patch('subprocess.check_call')
|
|
def test_lvm_volume_group_exists_finds_no_volume_group(self, _check):
|
|
volume_group = "test"
|
|
|
|
def raise_error(x):
|
|
raise subprocess.CalledProcessError(1, x)
|
|
_check.side_effect = raise_error
|
|
result = cinder_utils.lvm_volume_group_exists(volume_group)
|
|
self.assertFalse(result)
|
|
_check.assert_called_with(['vgdisplay', volume_group])
|
|
|
|
@patch('subprocess.check_call')
|
|
def test_remove_lvm_volume_group(self, _check):
|
|
volume_group = "test"
|
|
cinder_utils.remove_lvm_volume_group(volume_group)
|
|
_check.assert_called_with(['vgremove', '--force', volume_group])
|
|
|
|
def test_required_interfaces_api(self):
|
|
'''identity-service interface required for api service'''
|
|
expected = {
|
|
'database': ('shared-db', 'pgsql-db'),
|
|
'messaging': ('amqp',),
|
|
'identity': ('identity-service',),
|
|
}
|
|
self.assertEqual(cinder_utils.required_interfaces(), expected)
|
|
|
|
def test_required_interfaces_no_api(self):
|
|
'''
|
|
identity-service interface not required for volume
|
|
or scheduler service
|
|
'''
|
|
self.test_config.set('enabled-services', 'volume,scheduler')
|
|
expected = {
|
|
'database': ('shared-db', 'pgsql-db'),
|
|
'messaging': ('amqp',),
|
|
}
|
|
self.assertEqual(cinder_utils.required_interfaces(), expected)
|
|
|
|
def test_assess_status(self):
|
|
with patch.object(cinder_utils, 'assess_status_func') as asf:
|
|
callee = MagicMock()
|
|
asf.return_value = callee
|
|
cinder_utils.assess_status('test-config')
|
|
asf.assert_called_once_with('test-config')
|
|
callee.assert_called_once_with()
|
|
|
|
@patch.object(cinder_utils, 'get_optional_interfaces')
|
|
@patch.object(cinder_utils, 'check_optional_relations')
|
|
@patch.object(cinder_utils, 'required_interfaces')
|
|
@patch.object(cinder_utils, 'services')
|
|
@patch.object(cinder_utils, 'make_assess_status_func')
|
|
def test_assess_status_func(self,
|
|
make_assess_status_func,
|
|
services,
|
|
required_interfaces,
|
|
check_optional_relations,
|
|
get_optional_interfaces):
|
|
services.return_value = 's1'
|
|
required_interfaces.return_value = {'int': ['test 1']}
|
|
get_optional_interfaces.return_value = {'opt': ['test 2']}
|
|
cinder_utils.assess_status_func('test-config')
|
|
# ports=None whilst port checks are disabled.
|
|
make_assess_status_func.assert_called_once_with(
|
|
'test-config',
|
|
{'int': ['test 1'], 'opt': ['test 2']},
|
|
charm_func=check_optional_relations,
|
|
services='s1', ports=None)
|
|
|
|
def test_pause_unit_helper(self):
|
|
with patch.object(cinder_utils, '_pause_resume_helper') as prh:
|
|
cinder_utils.pause_unit_helper('random-config')
|
|
prh.assert_called_once_with(cinder_utils.pause_unit,
|
|
'random-config')
|
|
with patch.object(cinder_utils, '_pause_resume_helper') as prh:
|
|
cinder_utils.resume_unit_helper('random-config')
|
|
prh.assert_called_once_with(cinder_utils.resume_unit,
|
|
'random-config')
|
|
|
|
@patch.object(cinder_utils, 'services')
|
|
def test_pause_resume_helper(self, services):
|
|
f = MagicMock()
|
|
services.return_value = 's1'
|
|
with patch.object(cinder_utils, 'assess_status_func') as asf:
|
|
asf.return_value = 'assessor'
|
|
cinder_utils._pause_resume_helper(f, 'some-config')
|
|
asf.assert_called_once_with('some-config')
|
|
# ports=None whilst port checks are disabled.
|
|
f.assert_called_once_with('assessor', services='s1', ports=None)
|