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