Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 1 | # Copyright 2013 NEC Corporation. |
| 2 | # All Rights Reserved. |
| 3 | # |
| 4 | # Licensed under the Apache License, Version 2.0 (the "License"); you may |
| 5 | # not use this file except in compliance with the License. You may obtain |
| 6 | # a copy of the License at |
| 7 | # |
| 8 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | # |
| 10 | # Unless required by applicable law or agreed to in writing, software |
| 11 | # distributed under the License is distributed on an "AS IS" BASIS, WITHOUT |
| 12 | # WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the |
| 13 | # License for the specific language governing permissions and limitations |
| 14 | # under the License. |
| 15 | |
zhufl | c2c2f01 | 2016-05-16 17:34:18 +0800 | [diff] [blame] | 16 | import testtools |
| 17 | |
Sean Dague | 1937d09 | 2013-05-17 16:36:38 -0400 | [diff] [blame] | 18 | from tempest.api.compute import base |
Matthew Treinish | a03ed79 | 2013-09-23 21:38:15 +0000 | [diff] [blame] | 19 | from tempest.common import tempest_fixtures as fixtures |
Ken'ichi Ohmichi | 49db4fe | 2016-08-12 15:26:51 -0700 | [diff] [blame] | 20 | from tempest import config |
Ken'ichi Ohmichi | 757833a | 2017-03-10 10:30:30 -0800 | [diff] [blame] | 21 | from tempest.lib.common.utils import data_utils |
Jordan Pittier | 9e227c5 | 2016-02-09 14:35:18 +0100 | [diff] [blame] | 22 | from tempest.lib.common.utils import test_utils |
Ken'ichi Ohmichi | ebbfd1c | 2017-01-27 16:37:00 -0800 | [diff] [blame] | 23 | from tempest.lib import decorators |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 24 | |
Ken'ichi Ohmichi | 49db4fe | 2016-08-12 15:26:51 -0700 | [diff] [blame] | 25 | CONF = config.CONF |
| 26 | |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 27 | |
zhufl | 4af2c82 | 2018-08-06 14:38:53 +0800 | [diff] [blame] | 28 | class AggregatesAdminTestBase(base.BaseV2ComputeAdminTest): |
Ken'ichi Ohmichi | 88363cb | 2015-11-19 08:00:54 +0000 | [diff] [blame] | 29 | """Tests Aggregates API that require admin privileges""" |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 30 | |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 31 | @classmethod |
Rohan Kanade | 60b7309 | 2015-02-04 17:58:19 +0530 | [diff] [blame] | 32 | def setup_clients(cls): |
zhufl | 4af2c82 | 2018-08-06 14:38:53 +0800 | [diff] [blame] | 33 | super(AggregatesAdminTestBase, cls).setup_clients() |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 34 | cls.client = cls.os_admin.aggregates_client |
Rohan Kanade | 60b7309 | 2015-02-04 17:58:19 +0530 | [diff] [blame] | 35 | |
| 36 | @classmethod |
Andrea Frittoli | 50bb80d | 2014-09-15 12:34:27 +0100 | [diff] [blame] | 37 | def resource_setup(cls): |
zhufl | 4af2c82 | 2018-08-06 14:38:53 +0800 | [diff] [blame] | 38 | super(AggregatesAdminTestBase, cls).resource_setup() |
Ken'ichi Ohmichi | 4937f56 | 2015-03-23 00:15:01 +0000 | [diff] [blame] | 39 | cls.aggregate_name_prefix = 'test_aggregate' |
| 40 | cls.az_name_prefix = 'test_az' |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 41 | |
zhufl | c2c2f01 | 2016-05-16 17:34:18 +0800 | [diff] [blame] | 42 | cls.host = None |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 43 | hypers = cls.os_admin.hypervisor_client.list_hypervisors( |
Jianghua Wang | dbe76af | 2016-06-13 12:53:16 +0800 | [diff] [blame] | 44 | detail=True)['hypervisors'] |
Ken'ichi Ohmichi | 49db4fe | 2016-08-12 15:26:51 -0700 | [diff] [blame] | 45 | |
| 46 | if CONF.compute.hypervisor_type: |
| 47 | hypers = [hyper for hyper in hypers |
| 48 | if (hyper['hypervisor_type'] == |
| 49 | CONF.compute.hypervisor_type)] |
| 50 | |
zhufl | 34afeb2 | 2018-02-09 15:03:41 +0800 | [diff] [blame] | 51 | cls.hosts_available = [hyper['service']['host'] for hyper in hypers |
| 52 | if (hyper['state'] == 'up' and |
| 53 | hyper['status'] == 'enabled')] |
| 54 | if cls.hosts_available: |
| 55 | cls.host = cls.hosts_available[0] |
zhufl | c2c2f01 | 2016-05-16 17:34:18 +0800 | [diff] [blame] | 56 | else: |
Ken'ichi Ohmichi | 49db4fe | 2016-08-12 15:26:51 -0700 | [diff] [blame] | 57 | msg = "no available compute node found" |
| 58 | if CONF.compute.hypervisor_type: |
| 59 | msg += " for hypervisor_type %s" % CONF.compute.hypervisor_type |
| 60 | raise testtools.TestCase.failureException(msg) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 61 | |
zhufl | b926695 | 2017-05-04 15:41:57 +0800 | [diff] [blame] | 62 | def _create_test_aggregate(self, **kwargs): |
| 63 | if 'name' not in kwargs: |
| 64 | kwargs['name'] = data_utils.rand_name(self.aggregate_name_prefix) |
| 65 | aggregate = self.client.create_aggregate(**kwargs)['aggregate'] |
| 66 | self.addCleanup(test_utils.call_and_ignore_notfound_exc, |
| 67 | self.client.delete_aggregate, aggregate['id']) |
| 68 | self.assertEqual(kwargs['name'], aggregate['name']) |
| 69 | |
| 70 | return aggregate |
| 71 | |
zhufl | 4af2c82 | 2018-08-06 14:38:53 +0800 | [diff] [blame] | 72 | |
| 73 | class AggregatesAdminTestJSON(AggregatesAdminTestBase): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 74 | """Tests Aggregates API that require admin privileges""" |
zhufl | 4af2c82 | 2018-08-06 14:38:53 +0800 | [diff] [blame] | 75 | |
Ken'ichi Ohmichi | ebbfd1c | 2017-01-27 16:37:00 -0800 | [diff] [blame] | 76 | @decorators.idempotent_id('0d148aa3-d54c-4317-aa8d-42040a475e20') |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 77 | def test_aggregate_create_delete(self): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 78 | """Test create/delete aggregate""" |
zhufl | b926695 | 2017-05-04 15:41:57 +0800 | [diff] [blame] | 79 | aggregate = self._create_test_aggregate() |
llg8212 | e4cd392 | 2014-02-15 12:14:21 +0800 | [diff] [blame] | 80 | self.assertIsNone(aggregate['availability_zone']) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 81 | |
David Kranz | 0a73517 | 2015-01-16 10:51:18 -0500 | [diff] [blame] | 82 | self.client.delete_aggregate(aggregate['id']) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 83 | self.client.wait_for_resource_deletion(aggregate['id']) |
| 84 | |
Ken'ichi Ohmichi | ebbfd1c | 2017-01-27 16:37:00 -0800 | [diff] [blame] | 85 | @decorators.idempotent_id('5873a6f8-671a-43ff-8838-7ce430bb6d0b') |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 86 | def test_aggregate_create_delete_with_az(self): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 87 | """Test create/delete aggregate with availability_zone""" |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 88 | az_name = data_utils.rand_name(self.az_name_prefix) |
zhufl | b926695 | 2017-05-04 15:41:57 +0800 | [diff] [blame] | 89 | aggregate = self._create_test_aggregate(availability_zone=az_name) |
Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 90 | self.assertEqual(az_name, aggregate['availability_zone']) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 91 | |
David Kranz | 0a73517 | 2015-01-16 10:51:18 -0500 | [diff] [blame] | 92 | self.client.delete_aggregate(aggregate['id']) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 93 | self.client.wait_for_resource_deletion(aggregate['id']) |
| 94 | |
Ken'ichi Ohmichi | ebbfd1c | 2017-01-27 16:37:00 -0800 | [diff] [blame] | 95 | @decorators.idempotent_id('68089c38-04b1-4758-bdf0-cf0daec4defd') |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 96 | def test_aggregate_create_verify_entry_in_list(self): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 97 | """Test listing aggregate should contain the created aggregate""" |
zhufl | b926695 | 2017-05-04 15:41:57 +0800 | [diff] [blame] | 98 | aggregate = self._create_test_aggregate() |
ghanshyam | 2eb282d | 2015-08-04 15:05:19 +0900 | [diff] [blame] | 99 | aggregates = self.client.list_aggregates()['aggregates'] |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 100 | self.assertIn((aggregate['id'], aggregate['availability_zone']), |
| 101 | map(lambda x: (x['id'], x['availability_zone']), |
| 102 | aggregates)) |
| 103 | |
Ken'ichi Ohmichi | ebbfd1c | 2017-01-27 16:37:00 -0800 | [diff] [blame] | 104 | @decorators.idempotent_id('36ec92ca-7a73-43bc-b920-7531809e8540') |
ivan-zhu | 35e1f8e | 2013-10-18 15:51:16 +0800 | [diff] [blame] | 105 | def test_aggregate_create_update_metadata_get_details(self): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 106 | """Test set/get aggregate metadata""" |
zhufl | b926695 | 2017-05-04 15:41:57 +0800 | [diff] [blame] | 107 | aggregate = self._create_test_aggregate() |
ghanshyam | 2eb282d | 2015-08-04 15:05:19 +0900 | [diff] [blame] | 108 | body = self.client.show_aggregate(aggregate['id'])['aggregate'] |
Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 109 | self.assertEqual(aggregate['name'], body['name']) |
| 110 | self.assertEqual(aggregate['availability_zone'], |
| 111 | body['availability_zone']) |
ivan-zhu | 35e1f8e | 2013-10-18 15:51:16 +0800 | [diff] [blame] | 112 | self.assertEqual({}, body["metadata"]) |
| 113 | |
| 114 | # set the metadata of the aggregate |
| 115 | meta = {"key": "value"} |
Ken'ichi Ohmichi | ec452b8 | 2015-07-15 04:59:51 +0000 | [diff] [blame] | 116 | body = self.client.set_metadata(aggregate['id'], metadata=meta) |
ghanshyam | 2eb282d | 2015-08-04 15:05:19 +0900 | [diff] [blame] | 117 | self.assertEqual(meta, body['aggregate']["metadata"]) |
ivan-zhu | 35e1f8e | 2013-10-18 15:51:16 +0800 | [diff] [blame] | 118 | |
| 119 | # verify the metadata has been set |
ghanshyam | 2eb282d | 2015-08-04 15:05:19 +0900 | [diff] [blame] | 120 | body = self.client.show_aggregate(aggregate['id'])['aggregate'] |
ivan-zhu | 35e1f8e | 2013-10-18 15:51:16 +0800 | [diff] [blame] | 121 | self.assertEqual(meta, body["metadata"]) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 122 | |
Ken'ichi Ohmichi | ebbfd1c | 2017-01-27 16:37:00 -0800 | [diff] [blame] | 123 | @decorators.idempotent_id('4d2b2004-40fa-40a1-aab2-66f4dab81beb') |
Zhu Zhu | 7b5f629 | 2013-09-22 15:47:54 +0800 | [diff] [blame] | 124 | def test_aggregate_create_update_with_az(self): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 125 | """Test create/update aggregate with availability_zone""" |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 126 | aggregate_name = data_utils.rand_name(self.aggregate_name_prefix) |
| 127 | az_name = data_utils.rand_name(self.az_name_prefix) |
zhufl | b926695 | 2017-05-04 15:41:57 +0800 | [diff] [blame] | 128 | aggregate = self._create_test_aggregate( |
| 129 | name=aggregate_name, availability_zone=az_name) |
Zhu Zhu | 7b5f629 | 2013-09-22 15:47:54 +0800 | [diff] [blame] | 130 | |
Zhu Zhu | 7b5f629 | 2013-09-22 15:47:54 +0800 | [diff] [blame] | 131 | self.assertEqual(az_name, aggregate['availability_zone']) |
Zhu Zhu | 7b5f629 | 2013-09-22 15:47:54 +0800 | [diff] [blame] | 132 | |
| 133 | aggregate_id = aggregate['id'] |
| 134 | new_aggregate_name = aggregate_name + '_new' |
| 135 | new_az_name = az_name + '_new' |
| 136 | |
Ken'ichi Ohmichi | ec452b8 | 2015-07-15 04:59:51 +0000 | [diff] [blame] | 137 | resp_aggregate = self.client.update_aggregate( |
| 138 | aggregate_id, |
| 139 | name=new_aggregate_name, |
ghanshyam | 2eb282d | 2015-08-04 15:05:19 +0900 | [diff] [blame] | 140 | availability_zone=new_az_name)['aggregate'] |
Zhu Zhu | 7b5f629 | 2013-09-22 15:47:54 +0800 | [diff] [blame] | 141 | self.assertEqual(new_aggregate_name, resp_aggregate['name']) |
| 142 | self.assertEqual(new_az_name, resp_aggregate['availability_zone']) |
| 143 | |
ghanshyam | 2eb282d | 2015-08-04 15:05:19 +0900 | [diff] [blame] | 144 | aggregates = self.client.list_aggregates()['aggregates'] |
Zhu Zhu | 7b5f629 | 2013-09-22 15:47:54 +0800 | [diff] [blame] | 145 | self.assertIn((aggregate_id, new_aggregate_name, new_az_name), |
| 146 | map(lambda x: |
Matthew Treinish | 1d14c54 | 2014-06-17 20:25:40 -0400 | [diff] [blame] | 147 | (x['id'], x['name'], x['availability_zone']), |
Zhu Zhu | 7b5f629 | 2013-09-22 15:47:54 +0800 | [diff] [blame] | 148 | aggregates)) |
| 149 | |
Ken'ichi Ohmichi | ebbfd1c | 2017-01-27 16:37:00 -0800 | [diff] [blame] | 150 | @decorators.idempotent_id('c8e85064-e79b-4906-9931-c11c24294d02') |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 151 | def test_aggregate_add_remove_host(self): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 152 | """Test adding host to and removing host from aggregate""" |
Matthew Treinish | a03ed79 | 2013-09-23 21:38:15 +0000 | [diff] [blame] | 153 | self.useFixture(fixtures.LockFixture('availability_zone')) |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 154 | aggregate_name = data_utils.rand_name(self.aggregate_name_prefix) |
zhufl | b926695 | 2017-05-04 15:41:57 +0800 | [diff] [blame] | 155 | aggregate = self._create_test_aggregate(name=aggregate_name) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 156 | |
ghanshyam | 2eb282d | 2015-08-04 15:05:19 +0900 | [diff] [blame] | 157 | body = (self.client.add_host(aggregate['id'], host=self.host) |
| 158 | ['aggregate']) |
Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 159 | self.assertEqual(aggregate_name, body['name']) |
| 160 | self.assertEqual(aggregate['availability_zone'], |
| 161 | body['availability_zone']) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 162 | self.assertIn(self.host, body['hosts']) |
| 163 | |
ghanshyam | 2eb282d | 2015-08-04 15:05:19 +0900 | [diff] [blame] | 164 | body = (self.client.remove_host(aggregate['id'], host=self.host) |
| 165 | ['aggregate']) |
Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 166 | self.assertEqual(aggregate_name, body['name']) |
| 167 | self.assertEqual(aggregate['availability_zone'], |
| 168 | body['availability_zone']) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 169 | self.assertNotIn(self.host, body['hosts']) |
| 170 | |
Ken'ichi Ohmichi | ebbfd1c | 2017-01-27 16:37:00 -0800 | [diff] [blame] | 171 | @decorators.idempotent_id('7f6a1cc5-2446-4cdb-9baa-b6ae0a919b72') |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 172 | def test_aggregate_add_host_list(self): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 173 | """Test listing aggregate contains the host added to the aggregate |
| 174 | |
| 175 | Add a host to the given aggregate and list. |
| 176 | """ |
Matthew Treinish | a03ed79 | 2013-09-23 21:38:15 +0000 | [diff] [blame] | 177 | self.useFixture(fixtures.LockFixture('availability_zone')) |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 178 | aggregate_name = data_utils.rand_name(self.aggregate_name_prefix) |
zhufl | b926695 | 2017-05-04 15:41:57 +0800 | [diff] [blame] | 179 | aggregate = self._create_test_aggregate(name=aggregate_name) |
| 180 | |
Ken'ichi Ohmichi | ec452b8 | 2015-07-15 04:59:51 +0000 | [diff] [blame] | 181 | self.client.add_host(aggregate['id'], host=self.host) |
| 182 | self.addCleanup(self.client.remove_host, aggregate['id'], |
| 183 | host=self.host) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 184 | |
ghanshyam | 2eb282d | 2015-08-04 15:05:19 +0900 | [diff] [blame] | 185 | aggregates = self.client.list_aggregates()['aggregates'] |
Sirushti Murugesan | 935f2cc | 2016-07-12 19:48:24 +0530 | [diff] [blame] | 186 | aggs = [agg for agg in aggregates if agg['id'] == aggregate['id']] |
Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 187 | self.assertEqual(1, len(aggs)) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 188 | agg = aggs[0] |
Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 189 | self.assertEqual(aggregate_name, agg['name']) |
llg8212 | e4cd392 | 2014-02-15 12:14:21 +0800 | [diff] [blame] | 190 | self.assertIsNone(agg['availability_zone']) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 191 | self.assertIn(self.host, agg['hosts']) |
| 192 | |
Ken'ichi Ohmichi | ebbfd1c | 2017-01-27 16:37:00 -0800 | [diff] [blame] | 193 | @decorators.idempotent_id('eeef473c-7c52-494d-9f09-2ed7fc8fc036') |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 194 | def test_aggregate_add_host_get_details(self): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 195 | """Test showing aggregate contains the host added to the aggregate |
| 196 | |
| 197 | Add a host to the given aggregate and get details. |
| 198 | """ |
Matthew Treinish | a03ed79 | 2013-09-23 21:38:15 +0000 | [diff] [blame] | 199 | self.useFixture(fixtures.LockFixture('availability_zone')) |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 200 | aggregate_name = data_utils.rand_name(self.aggregate_name_prefix) |
zhufl | b926695 | 2017-05-04 15:41:57 +0800 | [diff] [blame] | 201 | aggregate = self._create_test_aggregate(name=aggregate_name) |
| 202 | |
Ken'ichi Ohmichi | ec452b8 | 2015-07-15 04:59:51 +0000 | [diff] [blame] | 203 | self.client.add_host(aggregate['id'], host=self.host) |
| 204 | self.addCleanup(self.client.remove_host, aggregate['id'], |
| 205 | host=self.host) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 206 | |
ghanshyam | 2eb282d | 2015-08-04 15:05:19 +0900 | [diff] [blame] | 207 | body = self.client.show_aggregate(aggregate['id'])['aggregate'] |
Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 208 | self.assertEqual(aggregate_name, body['name']) |
llg8212 | e4cd392 | 2014-02-15 12:14:21 +0800 | [diff] [blame] | 209 | self.assertIsNone(body['availability_zone']) |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 210 | self.assertIn(self.host, body['hosts']) |
| 211 | |
Ken'ichi Ohmichi | ebbfd1c | 2017-01-27 16:37:00 -0800 | [diff] [blame] | 212 | @decorators.idempotent_id('96be03c7-570d-409c-90f8-e4db3c646996') |
Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 213 | def test_aggregate_add_host_create_server_with_az(self): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 214 | """Test adding a host to the given aggregate and creating a server""" |
Matthew Treinish | a03ed79 | 2013-09-23 21:38:15 +0000 | [diff] [blame] | 215 | self.useFixture(fixtures.LockFixture('availability_zone')) |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 216 | az_name = data_utils.rand_name(self.az_name_prefix) |
zhufl | b926695 | 2017-05-04 15:41:57 +0800 | [diff] [blame] | 217 | aggregate = self._create_test_aggregate(availability_zone=az_name) |
| 218 | |
ghanshyam | 676492a | 2018-07-31 05:40:22 +0000 | [diff] [blame] | 219 | # Find a host that has not been added to other availability zone, |
| 220 | # for one host can't be added to different availability zones. |
zhufl | 34afeb2 | 2018-02-09 15:03:41 +0800 | [diff] [blame] | 221 | aggregates = self.client.list_aggregates()['aggregates'] |
| 222 | hosts_in_zone = [] |
ghanshyam | 676492a | 2018-07-31 05:40:22 +0000 | [diff] [blame] | 223 | for agg in aggregates: |
| 224 | if agg['availability_zone']: |
| 225 | hosts_in_zone.extend(agg['hosts']) |
zhufl | 34afeb2 | 2018-02-09 15:03:41 +0800 | [diff] [blame] | 226 | hosts = [v for v in self.hosts_available if v not in hosts_in_zone] |
| 227 | if not hosts: |
ghanshyam | 676492a | 2018-07-31 05:40:22 +0000 | [diff] [blame] | 228 | raise self.skipException("All hosts are already in other " |
| 229 | "availability zones, so can't add " |
| 230 | "host to aggregate. \nAggregates list: " |
| 231 | "%s" % aggregates) |
zhufl | 34afeb2 | 2018-02-09 15:03:41 +0800 | [diff] [blame] | 232 | host = hosts[0] |
| 233 | |
| 234 | self.client.add_host(aggregate['id'], host=host) |
| 235 | self.addCleanup(self.client.remove_host, aggregate['id'], host=host) |
zhufl | 7ae2268 | 2016-09-18 15:22:33 +0800 | [diff] [blame] | 236 | server = self.create_test_server(availability_zone=az_name, |
David Kranz | 0fb1429 | 2015-02-11 15:55:20 -0500 | [diff] [blame] | 237 | wait_until='ACTIVE') |
zhufl | 7bc916d | 2018-08-22 14:47:39 +0800 | [diff] [blame] | 238 | server_host = self.get_host_for_server(server['id']) |
| 239 | self.assertEqual(host, server_host) |
zhufl | 4af2c82 | 2018-08-06 14:38:53 +0800 | [diff] [blame] | 240 | |
| 241 | |
| 242 | class AggregatesAdminTestV241(AggregatesAdminTestBase): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 243 | """Tests Aggregates API that require admin privileges |
| 244 | |
| 245 | Tests Aggregates API that require admin privileges with compute |
| 246 | microversion greater than 2.40. |
| 247 | """ |
zhufl | 4af2c82 | 2018-08-06 14:38:53 +0800 | [diff] [blame] | 248 | min_microversion = '2.41' |
| 249 | |
| 250 | # NOTE(gmann): This test tests the Aggregate APIs response schema |
| 251 | # for 2.41 microversion. No specific assert or behaviour verification |
| 252 | # is needed. |
| 253 | |
| 254 | @decorators.idempotent_id('fdf24d9e-8afa-4700-b6aa-9c498351504f') |
| 255 | def test_create_update_show_aggregate_add_remove_host(self): |
zhufl | e354fa7 | 2020-08-12 17:07:50 +0800 | [diff] [blame] | 256 | """Test response schema of aggregates API |
| 257 | |
| 258 | Test response schema of aggregates API(create/update/show/add host/ |
| 259 | remove host) with compute microversion greater than 2.40. |
| 260 | """ |
zhufl | 4af2c82 | 2018-08-06 14:38:53 +0800 | [diff] [blame] | 261 | # Update and add a host to the given aggregate and get details. |
| 262 | self.useFixture(fixtures.LockFixture('availability_zone')) |
| 263 | # Checking create aggregate API response schema |
| 264 | aggregate = self._create_test_aggregate() |
| 265 | |
| 266 | new_aggregate_name = data_utils.rand_name(self.aggregate_name_prefix) |
| 267 | # Checking update aggregate API response schema |
| 268 | self.client.update_aggregate(aggregate['id'], name=new_aggregate_name) |
| 269 | # Checking show aggregate API response schema |
| 270 | self.client.show_aggregate(aggregate['id'])['aggregate'] |
| 271 | # Checking add host to aggregate API response schema |
| 272 | self.client.add_host(aggregate['id'], host=self.host) |
| 273 | # Checking rempve host from aggregate API response schema |
| 274 | self.client.remove_host(aggregate['id'], host=self.host) |