97d5e6ebe7
Following up cluster drivers implementation, move the scale managers at driver level. This change is needed to add the driver field properly. Change-Id: Ia854f2354c51b5fa47095bb4cb118416f3f01a33 Implements: blueprint bay-drivers
223 lines
9.3 KiB
Python
223 lines
9.3 KiB
Python
# Copyright 2015 Huawei Technologies Co.,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 mock
|
|
|
|
from magnum.common import exception
|
|
from magnum.conductor import scale_manager
|
|
from magnum.drivers.common.k8s_scale_manager import K8sScaleManager
|
|
from magnum.drivers.mesos_ubuntu_v1.scale_manager import MesosScaleManager
|
|
from magnum.tests import base
|
|
|
|
|
|
class TestScaleManager(base.TestCase):
|
|
|
|
def _test_get_removal_nodes(
|
|
self, mock_get_hosts, mock_get_num_of_removal,
|
|
mock_is_scale_down, mock_get_by_uuid, is_scale_down,
|
|
num_of_removal, all_hosts, container_hosts,
|
|
expected_removal_hosts):
|
|
|
|
mock_is_scale_down.return_value = is_scale_down
|
|
mock_get_num_of_removal.return_value = num_of_removal
|
|
|
|
mock_get_hosts.return_value = container_hosts
|
|
|
|
mock_heat_output = mock.MagicMock()
|
|
mock_heat_output.get_output_value.return_value = all_hosts
|
|
|
|
mock_stack = mock.MagicMock()
|
|
mock_heat_client = mock.MagicMock()
|
|
mock_osc = mock.MagicMock()
|
|
mock_heat_client.stacks.get.return_value = mock_stack
|
|
mock_osc.heat.return_value = mock_heat_client
|
|
|
|
mock_context = mock.MagicMock()
|
|
mock_cluster = mock.MagicMock()
|
|
scale_mgr = scale_manager.ScaleManager(mock_context, mock_osc,
|
|
mock_cluster)
|
|
|
|
if expected_removal_hosts is None:
|
|
self.assertRaises(exception.MagnumException,
|
|
scale_mgr.get_removal_nodes, mock_heat_output)
|
|
else:
|
|
removal_hosts = scale_mgr.get_removal_nodes(mock_heat_output)
|
|
self.assertEqual(expected_removal_hosts, removal_hosts)
|
|
if num_of_removal > 0:
|
|
mock_get_hosts.assert_called_once_with(mock_context,
|
|
mock_cluster)
|
|
|
|
@mock.patch('magnum.objects.Cluster.get_by_uuid')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager._is_scale_down')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_num_of_removal')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_hosts_with_container')
|
|
def test_get_removal_nodes_no_container_host(
|
|
self, mock_get_hosts, mock_get_num_of_removal,
|
|
mock_is_scale_down, mock_get_by_uuid):
|
|
|
|
is_scale_down = True
|
|
num_of_removal = 1
|
|
all_hosts = ['10.0.0.3']
|
|
container_hosts = set()
|
|
expected_removal_hosts = ['10.0.0.3']
|
|
self._test_get_removal_nodes(
|
|
mock_get_hosts, mock_get_num_of_removal, mock_is_scale_down,
|
|
mock_get_by_uuid, is_scale_down, num_of_removal, all_hosts,
|
|
container_hosts, expected_removal_hosts)
|
|
|
|
@mock.patch('magnum.objects.Cluster.get_by_uuid')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager._is_scale_down')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_num_of_removal')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_hosts_with_container')
|
|
def test_get_removal_nodes_one_container_host(
|
|
self, mock_get_hosts, mock_get_num_of_removal,
|
|
mock_is_scale_down, mock_get_by_uuid):
|
|
|
|
is_scale_down = True
|
|
num_of_removal = 1
|
|
all_hosts = ['10.0.0.3', '10.0.0.4']
|
|
container_hosts = set(['10.0.0.3'])
|
|
expected_removal_hosts = ['10.0.0.4']
|
|
self._test_get_removal_nodes(
|
|
mock_get_hosts, mock_get_num_of_removal, mock_is_scale_down,
|
|
mock_get_by_uuid, is_scale_down, num_of_removal, all_hosts,
|
|
container_hosts, expected_removal_hosts)
|
|
|
|
@mock.patch('magnum.objects.Cluster.get_by_uuid')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager._is_scale_down')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_num_of_removal')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_hosts_with_container')
|
|
def test_get_removal_nodes_two_container_hosts(
|
|
self, mock_get_hosts, mock_get_num_of_removal,
|
|
mock_is_scale_down, mock_get_by_uuid):
|
|
|
|
is_scale_down = True
|
|
num_of_removal = 1
|
|
all_hosts = ['10.0.0.3', '10.0.0.4']
|
|
container_hosts = set(['10.0.0.3', '10.0.0.4'])
|
|
expected_removal_hosts = []
|
|
self._test_get_removal_nodes(
|
|
mock_get_hosts, mock_get_num_of_removal, mock_is_scale_down,
|
|
mock_get_by_uuid, is_scale_down, num_of_removal, all_hosts,
|
|
container_hosts, expected_removal_hosts)
|
|
|
|
@mock.patch('magnum.objects.Cluster.get_by_uuid')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager._is_scale_down')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_num_of_removal')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_hosts_with_container')
|
|
def test_get_removal_nodes_three_container_hosts(
|
|
self, mock_get_hosts, mock_get_num_of_removal,
|
|
mock_is_scale_down, mock_get_by_uuid):
|
|
|
|
is_scale_down = True
|
|
num_of_removal = 1
|
|
all_hosts = ['10.0.0.3', '10.0.0.4']
|
|
container_hosts = set(['10.0.0.3', '10.0.0.4', '10.0.0.5'])
|
|
expected_removal_hosts = []
|
|
self._test_get_removal_nodes(
|
|
mock_get_hosts, mock_get_num_of_removal, mock_is_scale_down,
|
|
mock_get_by_uuid, is_scale_down, num_of_removal, all_hosts,
|
|
container_hosts, expected_removal_hosts)
|
|
|
|
@mock.patch('magnum.objects.Cluster.get_by_uuid')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager._is_scale_down')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_num_of_removal')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_hosts_with_container')
|
|
def test_get_removal_nodes_scale_up(
|
|
self, mock_get_hosts, mock_get_num_of_removal,
|
|
mock_is_scale_down, mock_get_by_uuid):
|
|
|
|
is_scale_down = False
|
|
num_of_removal = -1
|
|
all_hosts = ['10.0.0.3', '10.0.0.4']
|
|
container_hosts = set()
|
|
expected_removal_hosts = []
|
|
self._test_get_removal_nodes(
|
|
mock_get_hosts, mock_get_num_of_removal, mock_is_scale_down,
|
|
mock_get_by_uuid, is_scale_down, num_of_removal, all_hosts,
|
|
container_hosts, expected_removal_hosts)
|
|
|
|
@mock.patch('magnum.objects.Cluster.get_by_uuid')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager._is_scale_down')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_num_of_removal')
|
|
@mock.patch('magnum.conductor.scale_manager.ScaleManager.'
|
|
'_get_hosts_with_container')
|
|
def test_get_removal_nodes_with_none_hosts(
|
|
self, mock_get_hosts, mock_get_num_of_removal,
|
|
mock_is_scale_down, mock_get_by_uuid):
|
|
|
|
is_scale_down = True
|
|
num_of_removal = 1
|
|
all_hosts = None
|
|
container_hosts = set()
|
|
expected_removal_hosts = None
|
|
self._test_get_removal_nodes(
|
|
mock_get_hosts, mock_get_num_of_removal, mock_is_scale_down,
|
|
mock_get_by_uuid, is_scale_down, num_of_removal, all_hosts,
|
|
container_hosts, expected_removal_hosts)
|
|
|
|
|
|
class TestK8sScaleManager(base.TestCase):
|
|
|
|
@mock.patch('magnum.objects.Cluster.get_by_uuid')
|
|
@mock.patch('magnum.conductor.k8s_api.create_k8s_api')
|
|
def test_get_hosts_with_container(self, mock_create_api, mock_get):
|
|
pods = mock.MagicMock()
|
|
pod_1 = mock.MagicMock()
|
|
pod_1.spec.node_name = 'node1'
|
|
pod_2 = mock.MagicMock()
|
|
pod_2.spec.node_name = 'node2'
|
|
pods.items = [pod_1, pod_2]
|
|
mock_api = mock.MagicMock()
|
|
mock_api.list_namespaced_pod.return_value = pods
|
|
mock_create_api.return_value = mock_api
|
|
|
|
mgr = K8sScaleManager(
|
|
mock.MagicMock(), mock.MagicMock(), mock.MagicMock())
|
|
hosts = mgr._get_hosts_with_container(
|
|
mock.MagicMock(), mock.MagicMock())
|
|
self.assertEqual(hosts, {'node1', 'node2'})
|
|
|
|
|
|
class TestMesosScaleManager(base.TestCase):
|
|
|
|
@mock.patch('magnum.objects.Cluster.get_by_uuid')
|
|
@mock.patch('marathon.MarathonClient')
|
|
@mock.patch('marathon.MarathonClient.list_tasks')
|
|
def test_get_hosts_with_container(self, mock_list_tasks,
|
|
mock_client, mock_get):
|
|
task_1 = mock.MagicMock()
|
|
task_1.host = 'node1'
|
|
task_2 = mock.MagicMock()
|
|
task_2.host = 'node2'
|
|
tasks = [task_1, task_2]
|
|
mock_list_tasks.return_value = tasks
|
|
|
|
mgr = MesosScaleManager(
|
|
mock.MagicMock(), mock.MagicMock(), mock.MagicMock())
|
|
hosts = mgr._get_hosts_with_container(
|
|
mock.MagicMock(), mock.MagicMock())
|
|
self.assertEqual(hosts, {'node1', 'node2'})
|