607 lines
25 KiB
Python
607 lines
25 KiB
Python
# Copyright 2014 NEC Corporation. All rights reserved.
|
|
#
|
|
# 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.
|
|
|
|
from heatclient import exc
|
|
|
|
from magnum.common import exception
|
|
from magnum.conductor.handlers import bay_k8s_heat
|
|
from magnum import objects
|
|
from magnum.openstack.common import loopingcall
|
|
from magnum.tests import base
|
|
from magnum.tests.unit.db import base as db_base
|
|
from magnum.tests.unit.db import utils
|
|
|
|
import mock
|
|
from mock import patch
|
|
from oslo_config import cfg
|
|
|
|
|
|
class TestBayK8sHeat(base.TestCase):
|
|
def setUp(self):
|
|
super(TestBayK8sHeat, self).setUp()
|
|
self.baymodel_dict = {
|
|
'image_id': 'image_id',
|
|
'flavor_id': 'flavor_id',
|
|
'master_flavor_id': 'master_flavor_id',
|
|
'keypair_id': 'keypair_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'external_network_id': 'external_network_id',
|
|
'fixed_network': 'private',
|
|
'docker_volume_size': 20,
|
|
'ssh_authorized_key': 'ssh_authorized_key',
|
|
'token': None,
|
|
}
|
|
self.bay_dict = {
|
|
'baymodel_id': 'xx-xx-xx-xx',
|
|
'name': 'bay1',
|
|
'stack_id': 'xx-xx-xx-xx',
|
|
'api_address': '172.17.2.3',
|
|
'node_addresses': ['172.17.2.4'],
|
|
'node_count': 1,
|
|
}
|
|
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition(self,
|
|
mock_objects_baymodel_get_by_uuid):
|
|
baymodel = objects.BayModel(self.context, **self.baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_image': 'image_id',
|
|
'server_flavor': 'flavor_id',
|
|
'master_flavor': 'master_flavor_id',
|
|
'number_of_minions': '1',
|
|
'fixed_network': 'private',
|
|
'docker_volume_size': 20,
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('requests.get')
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_coreos_with_disovery(self,
|
|
mock_objects_baymodel_get_by_uuid,
|
|
reqget):
|
|
cfg.CONF.set_override('cluster_type',
|
|
'coreos',
|
|
group='k8s_heat')
|
|
cfg.CONF.set_override('coreos_discovery_token_url',
|
|
'http://tokentest',
|
|
group='bay')
|
|
mock_req = mock.MagicMock(text='/h1/h2/h3')
|
|
reqget.return_value = mock_req
|
|
baymodel = objects.BayModel(self.context, **self.baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_image': 'image_id',
|
|
'server_flavor': 'flavor_id',
|
|
'master_flavor': 'master_flavor_id',
|
|
'number_of_minions': '1',
|
|
'fixed_network': 'private',
|
|
'docker_volume_size': 20,
|
|
'ssh_authorized_key': 'ssh_authorized_key',
|
|
'token': 'h3'
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('uuid.uuid4')
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_coreos_no_discoveryurl(self,
|
|
mock_objects_baymodel_get_by_uuid,
|
|
mock_uuid):
|
|
cfg.CONF.set_override('cluster_type',
|
|
'coreos',
|
|
group='k8s_heat')
|
|
cfg.CONF.set_override('coreos_discovery_token_url',
|
|
None,
|
|
group='bay')
|
|
mock_uuid.return_value = mock.MagicMock(
|
|
hex='ba3d1866282848ddbedc76112110c208')
|
|
baymodel = objects.BayModel(self.context, **self.baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_image': 'image_id',
|
|
'server_flavor': 'flavor_id',
|
|
'master_flavor': 'master_flavor_id',
|
|
'number_of_minions': '1',
|
|
'fixed_network': 'private',
|
|
'docker_volume_size': 20,
|
|
'ssh_authorized_key': 'ssh_authorized_key',
|
|
'token': 'ba3d1866282848ddbedc76112110c208'
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_without_dns(self,
|
|
mock_objects_baymodel_get_by_uuid):
|
|
baymodel_dict = self.baymodel_dict
|
|
baymodel_dict['dns_nameserver'] = None
|
|
baymodel = objects.BayModel(self.context, **baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'server_image': 'image_id',
|
|
'server_flavor': 'flavor_id',
|
|
'master_flavor': 'master_flavor_id',
|
|
'number_of_minions': '1',
|
|
'fixed_network': 'private',
|
|
'docker_volume_size': 20
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_without_server_image(self,
|
|
mock_objects_baymodel_get_by_uuid):
|
|
baymodel_dict = self.baymodel_dict
|
|
baymodel_dict['image_id'] = None
|
|
baymodel = objects.BayModel(self.context, **baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_flavor': 'flavor_id',
|
|
'master_flavor': 'master_flavor_id',
|
|
'number_of_minions': '1',
|
|
'fixed_network': 'private',
|
|
'docker_volume_size': 20
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_without_server_flavor(self,
|
|
mock_objects_baymodel_get_by_uuid):
|
|
baymodel_dict = self.baymodel_dict
|
|
baymodel_dict['flavor_id'] = None
|
|
baymodel = objects.BayModel(self.context, **baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_image': 'image_id',
|
|
'master_flavor': 'master_flavor_id',
|
|
'number_of_minions': '1',
|
|
'fixed_network': 'private',
|
|
'docker_volume_size': 20,
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_without_docker_volume_size(self,
|
|
mock_objects_baymodel_get_by_uuid):
|
|
baymodel_dict = self.baymodel_dict
|
|
baymodel_dict['docker_volume_size'] = None
|
|
baymodel = objects.BayModel(self.context, **baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_image': 'image_id',
|
|
'server_flavor': 'flavor_id',
|
|
'fixed_network': 'private',
|
|
'master_flavor': 'master_flavor_id',
|
|
'number_of_minions': '1',
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_without_fixed_network(self,
|
|
mock_objects_baymodel_get_by_uuid):
|
|
baymodel_dict = self.baymodel_dict
|
|
baymodel_dict['fixed_network'] = None
|
|
baymodel = objects.BayModel(self.context, **baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_image': 'image_id',
|
|
'master_flavor': 'master_flavor_id',
|
|
'server_flavor': 'flavor_id',
|
|
'number_of_minions': '1',
|
|
'docker_volume_size': 20,
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_without_master_flavor(self,
|
|
mock_objects_baymodel_get_by_uuid):
|
|
baymodel_dict = self.baymodel_dict
|
|
baymodel_dict['master_flavor_id'] = None
|
|
baymodel = objects.BayModel(self.context, **baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_image': 'image_id',
|
|
'server_flavor': 'flavor_id',
|
|
'number_of_minions': '1',
|
|
'fixed_network': 'private',
|
|
'docker_volume_size': 20,
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_without_ssh_authorized_key(self,
|
|
mock_objects_baymodel_get_by_uuid):
|
|
cfg.CONF.set_override('cluster_type',
|
|
'coreos',
|
|
group='k8s_heat')
|
|
baymodel_dict = self.baymodel_dict
|
|
baymodel_dict['ssh_authorized_key'] = None
|
|
baymodel = objects.BayModel(self.context, **baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_image': 'image_id',
|
|
'master_flavor': 'master_flavor_id',
|
|
'server_flavor': 'flavor_id',
|
|
'number_of_minions': '1',
|
|
'fixed_network': 'private',
|
|
'docker_volume_size': 20,
|
|
}
|
|
self.assertIn('token', definition)
|
|
del definition['token']
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_without_apiserver_port(self,
|
|
mock_objects_baymodel_get_by_uuid):
|
|
baymodel_dict = self.baymodel_dict
|
|
baymodel_dict['apiserver_port'] = None
|
|
baymodel = objects.BayModel(self.context, **baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **self.bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_image': 'image_id',
|
|
'server_flavor': 'flavor_id',
|
|
'master_flavor': 'master_flavor_id',
|
|
'number_of_minions': '1',
|
|
'fixed_network': 'private',
|
|
'docker_volume_size': 20,
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
@patch('magnum.objects.BayModel.get_by_uuid')
|
|
def test_extract_template_definition_without_node_count(self,
|
|
mock_objects_baymodel_get_by_uuid):
|
|
bay_dict = self.bay_dict
|
|
bay_dict['node_count'] = None
|
|
baymodel = objects.BayModel(self.context, **self.baymodel_dict)
|
|
mock_objects_baymodel_get_by_uuid.return_value = baymodel
|
|
bay = objects.Bay(self.context, **bay_dict)
|
|
|
|
(template_path,
|
|
definition) = bay_k8s_heat._extract_template_definition(self.context,
|
|
bay)
|
|
|
|
expected = {
|
|
'ssh_key_name': 'keypair_id',
|
|
'external_network_id': 'external_network_id',
|
|
'dns_nameserver': 'dns_nameserver',
|
|
'server_image': 'image_id',
|
|
'server_flavor': 'flavor_id',
|
|
'fixed_network': 'private',
|
|
'master_flavor': 'master_flavor_id',
|
|
'docker_volume_size': 20,
|
|
}
|
|
self.assertEqual(expected, definition)
|
|
|
|
def test_update_stack_outputs(self):
|
|
expected_api_address = 'api_address'
|
|
expected_node_addresses = ['ex_minion', 'address']
|
|
|
|
outputs = [
|
|
{
|
|
"output_value": expected_node_addresses,
|
|
"description": "No description given",
|
|
"output_key": "kube_minions_external"
|
|
},
|
|
{
|
|
"output_value": expected_api_address,
|
|
"description": "No description given",
|
|
"output_key": "kube_master"
|
|
}
|
|
]
|
|
mock_stack = mock.MagicMock()
|
|
mock_stack.outputs = outputs
|
|
mock_bay = mock.MagicMock()
|
|
|
|
bay_k8s_heat._update_stack_outputs(mock_stack, mock_bay)
|
|
|
|
self.assertEqual(mock_bay.api_address, expected_api_address)
|
|
self.assertEqual(mock_bay.node_addresses, expected_node_addresses)
|
|
|
|
@patch('magnum.common.short_id.generate_id')
|
|
@patch('heatclient.common.template_utils.get_template_contents')
|
|
@patch('magnum.conductor.handlers.bay_k8s_heat'
|
|
'._extract_template_definition')
|
|
def test_create_stack(self,
|
|
mock_extract_template_definition,
|
|
mock_get_template_contents,
|
|
mock_generate_id):
|
|
|
|
mock_generate_id.return_value = 'xx-xx-xx-xx'
|
|
expected_stack_name = 'expected_stack_name-xx-xx-xx-xx'
|
|
expected_template_contents = 'template_contents'
|
|
exptected_files = []
|
|
dummy_bay_name = 'expected_stack_name'
|
|
|
|
mock_tpl_files = mock.MagicMock()
|
|
mock_tpl_files.items.return_value = exptected_files
|
|
mock_get_template_contents.return_value = [
|
|
mock_tpl_files, expected_template_contents]
|
|
mock_extract_template_definition.return_value = ('template/path',
|
|
{})
|
|
mock_heat_client = mock.MagicMock()
|
|
mock_osc = mock.MagicMock()
|
|
mock_osc.heat.return_value = mock_heat_client
|
|
mock_bay = mock.MagicMock()
|
|
mock_bay.name = dummy_bay_name
|
|
|
|
bay_k8s_heat._create_stack(self.context, mock_osc, mock_bay)
|
|
|
|
expected_args = {
|
|
'stack_name': expected_stack_name,
|
|
'parameters': {},
|
|
'template': expected_template_contents,
|
|
'files': dict(exptected_files)
|
|
}
|
|
mock_heat_client.stacks.create.assert_called_once_with(**expected_args)
|
|
|
|
@patch('heatclient.common.template_utils.get_template_contents')
|
|
@patch('magnum.conductor.handlers.bay_k8s_heat'
|
|
'._extract_template_definition')
|
|
def test_update_stack(self,
|
|
mock_extract_template_definition,
|
|
mock_get_template_contents):
|
|
|
|
mock_stack_id = 'xx-xx-xx-xx'
|
|
expected_template_contents = 'template_contents'
|
|
exptected_files = []
|
|
|
|
mock_tpl_files = mock.MagicMock()
|
|
mock_tpl_files.items.return_value = exptected_files
|
|
mock_get_template_contents.return_value = [
|
|
mock_tpl_files, expected_template_contents]
|
|
mock_extract_template_definition.return_value = ('template/path',
|
|
{})
|
|
mock_heat_client = mock.MagicMock()
|
|
mock_osc = mock.MagicMock()
|
|
mock_osc.heat.return_value = mock_heat_client
|
|
mock_bay = mock.MagicMock()
|
|
mock_bay.stack_id = mock_stack_id
|
|
|
|
bay_k8s_heat._update_stack({}, mock_osc, mock_bay)
|
|
|
|
expected_args = {
|
|
'parameters': {},
|
|
'template': expected_template_contents,
|
|
'files': dict(exptected_files)
|
|
}
|
|
mock_heat_client.stacks.update.assert_called_once_with(mock_stack_id,
|
|
**expected_args)
|
|
|
|
@patch('oslo_config.cfg')
|
|
@patch('magnum.common.clients.OpenStackClients')
|
|
def setup_poll_test(self, mock_openstack_client, cfg):
|
|
cfg.CONF.k8s_heat.max_attempts = 10
|
|
bay = mock.MagicMock()
|
|
mock_heat_stack = mock.MagicMock()
|
|
mock_heat_client = mock.MagicMock()
|
|
mock_heat_client.stacks.get.return_value = mock_heat_stack
|
|
mock_openstack_client.heat.return_value = mock_heat_client
|
|
poller = bay_k8s_heat.HeatPoller(mock_openstack_client, bay)
|
|
return (mock_heat_stack, bay, poller)
|
|
|
|
def test_poll_no_save(self):
|
|
mock_heat_stack, bay, poller = self.setup_poll_test()
|
|
|
|
bay.status = 'CREATE_IN_PROGRESS'
|
|
mock_heat_stack.stack_status = 'CREATE_IN_PROGRESS'
|
|
poller.poll_and_check()
|
|
|
|
self.assertEqual(bay.save.call_count, 0)
|
|
self.assertEqual(poller.attempts, 1)
|
|
|
|
def test_poll_save(self):
|
|
mock_heat_stack, bay, poller = self.setup_poll_test()
|
|
|
|
bay.status = 'CREATE_IN_PROGRESS'
|
|
mock_heat_stack.stack_status = 'CREATE_FAILED'
|
|
self.assertRaises(loopingcall.LoopingCallDone, poller.poll_and_check)
|
|
|
|
self.assertEqual(bay.save.call_count, 1)
|
|
self.assertEqual(bay.status, 'CREATE_FAILED')
|
|
self.assertEqual(poller.attempts, 1)
|
|
|
|
def test_poll_done(self):
|
|
mock_heat_stack, bay, poller = self.setup_poll_test()
|
|
|
|
mock_heat_stack.stack_status = 'DELETE_COMPLETE'
|
|
self.assertRaises(loopingcall.LoopingCallDone, poller.poll_and_check)
|
|
|
|
mock_heat_stack.stack_status = 'CREATE_FAILED'
|
|
self.assertRaises(loopingcall.LoopingCallDone, poller.poll_and_check)
|
|
self.assertEqual(poller.attempts, 2)
|
|
|
|
def test_poll_destroy(self):
|
|
mock_heat_stack, bay, poller = self.setup_poll_test()
|
|
|
|
mock_heat_stack.stack_status = 'DELETE_FAILED'
|
|
self.assertRaises(loopingcall.LoopingCallDone, poller.poll_and_check)
|
|
# Destroy method is not called when stack delete failed
|
|
self.assertEqual(bay.destroy.call_count, 0)
|
|
|
|
mock_heat_stack.stack_status = 'DELETE_IN_PROGRESS'
|
|
poller.poll_and_check()
|
|
self.assertEqual(bay.destroy.call_count, 0)
|
|
self.assertEqual(bay.status, 'DELETE_IN_PROGRESS')
|
|
|
|
mock_heat_stack.stack_status = 'DELETE_COMPLETE'
|
|
self.assertRaises(loopingcall.LoopingCallDone, poller.poll_and_check)
|
|
# The bay status should still be DELETE_IN_PROGRESS, because
|
|
# the destroy() method may be failed. If success, this bay record
|
|
# will delete directly, change status is meaningless.
|
|
self.assertEqual(bay.status, 'DELETE_IN_PROGRESS')
|
|
self.assertEqual(bay.destroy.call_count, 1)
|
|
|
|
|
|
class TestHandler(db_base.DbTestCase):
|
|
|
|
def setUp(self):
|
|
super(TestHandler, self).setUp()
|
|
self.handler = bay_k8s_heat.Handler()
|
|
bay_dict = utils.get_test_bay(node_count=1)
|
|
self.bay = objects.Bay(self.context, **bay_dict)
|
|
self.bay.create()
|
|
|
|
@patch('magnum.conductor.handlers.bay_k8s_heat.Handler._poll_and_check')
|
|
@patch('magnum.conductor.handlers.bay_k8s_heat._update_stack')
|
|
@patch('magnum.common.clients.OpenStackClients')
|
|
def test_update_node_count_success(self, mock_openstack_client_class,
|
|
mock_update_stack, mock_poll_and_check):
|
|
mock_heat_stack = mock.MagicMock()
|
|
mock_heat_stack.stack_status = 'CREATE_COMPLETE'
|
|
mock_heat_client = mock.MagicMock()
|
|
mock_heat_client.stacks.get.return_value = mock_heat_stack
|
|
mock_openstack_client = mock_openstack_client_class.return_value
|
|
mock_openstack_client.heat.return_value = mock_heat_client
|
|
|
|
self.bay.node_count = 2
|
|
self.handler.bay_update(self.context, self.bay)
|
|
|
|
mock_update_stack.assert_called_once_with(self.context,
|
|
mock_openstack_client,
|
|
self.bay)
|
|
bay = objects.Bay.get(self.context, self.bay.uuid)
|
|
self.assertEqual(bay.node_count, 2)
|
|
|
|
@patch('magnum.conductor.handlers.bay_k8s_heat.Handler._poll_and_check')
|
|
@patch('magnum.conductor.handlers.bay_k8s_heat._update_stack')
|
|
@patch('magnum.common.clients.OpenStackClients')
|
|
def test_update_node_count_failure(self, mock_openstack_client_class,
|
|
mock_update_stack, mock_poll_and_check):
|
|
mock_heat_stack = mock.MagicMock()
|
|
mock_heat_stack.stack_status = 'CREATE_FAILED'
|
|
mock_heat_client = mock.MagicMock()
|
|
mock_heat_client.stacks.get.return_value = mock_heat_stack
|
|
mock_openstack_client = mock_openstack_client_class.return_value
|
|
mock_openstack_client.heat.return_value = mock_heat_client
|
|
|
|
self.bay.node_count = 2
|
|
self.assertRaises(exception.NotSupported, self.handler.bay_update,
|
|
self.context, self.bay)
|
|
|
|
bay = objects.Bay.get(self.context, self.bay.uuid)
|
|
self.assertEqual(bay.node_count, 1)
|
|
|
|
@patch('magnum.conductor.handlers.bay_k8s_heat._create_stack')
|
|
@patch('magnum.common.clients.OpenStackClients')
|
|
def test_create(self, mock_openstack_client_class, mock_create_stack):
|
|
mock_create_stack.side_effect = exc.HTTPBadRequest
|
|
self.assertRaises(exception.InvalidParameterValue,
|
|
self.handler.bay_create, self.context, self.bay)
|
|
|
|
@patch('magnum.common.clients.OpenStackClients')
|
|
def test_bay_delete(self, mock_openstack_client_class):
|
|
osc = mock.MagicMock()
|
|
mock_openstack_client_class.return_value = osc
|
|
osc.heat.side_effect = exc.HTTPNotFound
|
|
self.handler.bay_delete(self.context, self.bay.uuid)
|
|
# The bay has been destroyed
|
|
self.assertRaises(exception.BayNotFound,
|
|
objects.Bay.get, self.context, self.bay.uuid)
|