| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 1 | # vim: tabstop=4 shiftwidth=4 softtabstop=4 | 
|  | 2 |  | 
|  | 3 | # Copyright 2013 NEC Corporation. | 
|  | 4 | # All Rights Reserved. | 
|  | 5 | # | 
|  | 6 | #    Licensed under the Apache License, Version 2.0 (the "License"); you may | 
|  | 7 | #    not use this file except in compliance with the License. You may obtain | 
|  | 8 | #    a copy of the License at | 
|  | 9 | # | 
|  | 10 | #         http://www.apache.org/licenses/LICENSE-2.0 | 
|  | 11 | # | 
|  | 12 | #    Unless required by applicable law or agreed to in writing, software | 
|  | 13 | #    distributed under the License is distributed on an "AS IS" BASIS, WITHOUT | 
|  | 14 | #    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the | 
|  | 15 | #    License for the specific language governing permissions and limitations | 
|  | 16 | #    under the License. | 
|  | 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 | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 20 | from tempest.common.utils.data_utils import rand_name | 
|  | 21 | from tempest import exceptions | 
|  | 22 | from tempest.test import attr | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 23 |  | 
|  | 24 |  | 
| ivan-zhu | f2b0050 | 2013-10-18 10:06:52 +0800 | [diff] [blame] | 25 | class AggregatesAdminTestJSON(base.BaseV2ComputeAdminTest): | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 26 |  | 
|  | 27 | """ | 
|  | 28 | Tests Aggregates API that require admin privileges | 
|  | 29 | """ | 
|  | 30 |  | 
| Mitsuhiko Yamazaki | ae8fc53 | 2013-04-22 11:17:35 +0900 | [diff] [blame] | 31 | _host_key = 'OS-EXT-SRV-ATTR:host' | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 32 | _interface = 'json' | 
|  | 33 |  | 
|  | 34 | @classmethod | 
|  | 35 | def setUpClass(cls): | 
|  | 36 | super(AggregatesAdminTestJSON, cls).setUpClass() | 
|  | 37 | cls.client = cls.os_adm.aggregates_client | 
| Mitsuhiko Yamazaki | ae8fc53 | 2013-04-22 11:17:35 +0900 | [diff] [blame] | 38 | cls.user_client = cls.aggregates_client | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 39 | cls.aggregate_name_prefix = 'test_aggregate_' | 
|  | 40 | cls.az_name_prefix = 'test_az_' | 
|  | 41 |  | 
|  | 42 | resp, hosts_all = cls.os_adm.hosts_client.list_hosts() | 
|  | 43 | hosts = map(lambda x: x['host_name'], | 
|  | 44 | filter(lambda y: y['service'] == 'compute', hosts_all)) | 
|  | 45 | cls.host = hosts[0] | 
|  | 46 |  | 
| Giulio Fidente | ba3985a | 2013-05-29 01:46:36 +0200 | [diff] [blame] | 47 | @attr(type='gate') | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 48 | def test_aggregate_create_delete(self): | 
|  | 49 | # Create and delete an aggregate. | 
|  | 50 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 51 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 52 | self.assertEqual(200, resp.status) | 
|  | 53 | self.assertEqual(aggregate_name, aggregate['name']) | 
|  | 54 | self.assertEqual(None, aggregate['availability_zone']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 55 |  | 
|  | 56 | resp, _ = self.client.delete_aggregate(aggregate['id']) | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 57 | self.assertEqual(200, resp.status) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 58 | self.client.wait_for_resource_deletion(aggregate['id']) | 
|  | 59 |  | 
| Giulio Fidente | ba3985a | 2013-05-29 01:46:36 +0200 | [diff] [blame] | 60 | @attr(type='gate') | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 61 | def test_aggregate_create_delete_with_az(self): | 
|  | 62 | # Create and delete an aggregate. | 
|  | 63 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 64 | az_name = rand_name(self.az_name_prefix) | 
|  | 65 | resp, aggregate = self.client.create_aggregate(aggregate_name, az_name) | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 66 | self.assertEqual(200, resp.status) | 
|  | 67 | self.assertEqual(aggregate_name, aggregate['name']) | 
|  | 68 | self.assertEqual(az_name, aggregate['availability_zone']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 69 |  | 
|  | 70 | resp, _ = self.client.delete_aggregate(aggregate['id']) | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 71 | self.assertEqual(200, resp.status) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 72 | self.client.wait_for_resource_deletion(aggregate['id']) | 
|  | 73 |  | 
| Giulio Fidente | ba3985a | 2013-05-29 01:46:36 +0200 | [diff] [blame] | 74 | @attr(type='gate') | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 75 | def test_aggregate_create_verify_entry_in_list(self): | 
|  | 76 | # Create an aggregate and ensure it is listed. | 
|  | 77 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 78 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
|  | 79 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 80 |  | 
|  | 81 | resp, aggregates = self.client.list_aggregates() | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 82 | self.assertEqual(200, resp.status) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 83 | self.assertIn((aggregate['id'], aggregate['availability_zone']), | 
|  | 84 | map(lambda x: (x['id'], x['availability_zone']), | 
|  | 85 | aggregates)) | 
|  | 86 |  | 
| Giulio Fidente | ba3985a | 2013-05-29 01:46:36 +0200 | [diff] [blame] | 87 | @attr(type='gate') | 
| ivan-zhu | 35e1f8e | 2013-10-18 15:51:16 +0800 | [diff] [blame] | 88 | def test_aggregate_create_update_metadata_get_details(self): | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 89 | # Create an aggregate and ensure its details are returned. | 
|  | 90 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 91 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
|  | 92 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 93 |  | 
|  | 94 | resp, body = self.client.get_aggregate(aggregate['id']) | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 95 | self.assertEqual(200, resp.status) | 
|  | 96 | self.assertEqual(aggregate['name'], body['name']) | 
|  | 97 | self.assertEqual(aggregate['availability_zone'], | 
|  | 98 | body['availability_zone']) | 
| ivan-zhu | 35e1f8e | 2013-10-18 15:51:16 +0800 | [diff] [blame] | 99 | self.assertEqual({}, body["metadata"]) | 
|  | 100 |  | 
|  | 101 | # set the metadata of the aggregate | 
|  | 102 | meta = {"key": "value"} | 
|  | 103 | resp, body = self.client.set_metadata(aggregate['id'], meta) | 
|  | 104 | self.assertEqual(200, resp.status) | 
|  | 105 | self.assertEqual(meta, body["metadata"]) | 
|  | 106 |  | 
|  | 107 | # verify the metadata has been set | 
|  | 108 | resp, body = self.client.get_aggregate(aggregate['id']) | 
|  | 109 | self.assertEqual(200, resp.status) | 
|  | 110 | self.assertEqual(meta, body["metadata"]) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 111 |  | 
| Zhu Zhu | 7b5f629 | 2013-09-22 15:47:54 +0800 | [diff] [blame] | 112 | @attr(type='gate') | 
|  | 113 | def test_aggregate_create_update_with_az(self): | 
|  | 114 | # Update an aggregate and ensure properties are updated correctly | 
|  | 115 | self.useFixture(fixtures.LockFixture('availability_zone')) | 
|  | 116 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 117 | az_name = rand_name(self.az_name_prefix) | 
|  | 118 | resp, aggregate = self.client.create_aggregate(aggregate_name, az_name) | 
|  | 119 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 120 |  | 
|  | 121 | self.assertEqual(200, resp.status) | 
|  | 122 | self.assertEqual(aggregate_name, aggregate['name']) | 
|  | 123 | self.assertEqual(az_name, aggregate['availability_zone']) | 
|  | 124 | self.assertIsNotNone(aggregate['id']) | 
|  | 125 |  | 
|  | 126 | aggregate_id = aggregate['id'] | 
|  | 127 | new_aggregate_name = aggregate_name + '_new' | 
|  | 128 | new_az_name = az_name + '_new' | 
|  | 129 |  | 
|  | 130 | resp, resp_aggregate = self.client.update_aggregate(aggregate_id, | 
|  | 131 | new_aggregate_name, | 
|  | 132 | new_az_name) | 
|  | 133 | self.assertEqual(200, resp.status) | 
|  | 134 | self.assertEqual(new_aggregate_name, resp_aggregate['name']) | 
|  | 135 | self.assertEqual(new_az_name, resp_aggregate['availability_zone']) | 
|  | 136 |  | 
|  | 137 | resp, aggregates = self.client.list_aggregates() | 
|  | 138 | self.assertEqual(200, resp.status) | 
|  | 139 | self.assertIn((aggregate_id, new_aggregate_name, new_az_name), | 
|  | 140 | map(lambda x: | 
|  | 141 | (x['id'], x['name'], x['availability_zone']), | 
|  | 142 | aggregates)) | 
|  | 143 |  | 
| Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 144 | @attr(type=['negative', 'gate']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 145 | def test_aggregate_create_as_user(self): | 
|  | 146 | # Regular user is not allowed to create an aggregate. | 
|  | 147 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 148 | self.assertRaises(exceptions.Unauthorized, | 
|  | 149 | self.user_client.create_aggregate, | 
|  | 150 | aggregate_name) | 
|  | 151 |  | 
| Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 152 | @attr(type=['negative', 'gate']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 153 | def test_aggregate_delete_as_user(self): | 
|  | 154 | # Regular user is not allowed to delete an aggregate. | 
|  | 155 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 156 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
|  | 157 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 158 |  | 
|  | 159 | self.assertRaises(exceptions.Unauthorized, | 
|  | 160 | self.user_client.delete_aggregate, | 
|  | 161 | aggregate['id']) | 
|  | 162 |  | 
| Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 163 | @attr(type=['negative', 'gate']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 164 | def test_aggregate_list_as_user(self): | 
|  | 165 | # Regular user is not allowed to list aggregates. | 
|  | 166 | self.assertRaises(exceptions.Unauthorized, | 
|  | 167 | self.user_client.list_aggregates) | 
|  | 168 |  | 
| Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 169 | @attr(type=['negative', 'gate']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 170 | def test_aggregate_get_details_as_user(self): | 
|  | 171 | # Regular user is not allowed to get aggregate details. | 
|  | 172 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 173 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
|  | 174 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 175 |  | 
|  | 176 | self.assertRaises(exceptions.Unauthorized, | 
|  | 177 | self.user_client.get_aggregate, | 
|  | 178 | aggregate['id']) | 
|  | 179 |  | 
| Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 180 | @attr(type=['negative', 'gate']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 181 | def test_aggregate_delete_with_invalid_id(self): | 
|  | 182 | # Delete an aggregate with invalid id should raise exceptions. | 
|  | 183 | self.assertRaises(exceptions.NotFound, | 
|  | 184 | self.client.delete_aggregate, -1) | 
|  | 185 |  | 
| Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 186 | @attr(type=['negative', 'gate']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 187 | def test_aggregate_get_details_with_invalid_id(self): | 
|  | 188 | # Get aggregate details with invalid id should raise exceptions. | 
|  | 189 | self.assertRaises(exceptions.NotFound, | 
|  | 190 | self.client.get_aggregate, -1) | 
|  | 191 |  | 
| Giulio Fidente | ba3985a | 2013-05-29 01:46:36 +0200 | [diff] [blame] | 192 | @attr(type='gate') | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 193 | def test_aggregate_add_remove_host(self): | 
|  | 194 | # Add an host to the given aggregate and remove. | 
| Matthew Treinish | a03ed79 | 2013-09-23 21:38:15 +0000 | [diff] [blame] | 195 | self.useFixture(fixtures.LockFixture('availability_zone')) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 196 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 197 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
|  | 198 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 199 |  | 
|  | 200 | resp, body = self.client.add_host(aggregate['id'], self.host) | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 201 | self.assertEqual(200, resp.status) | 
|  | 202 | self.assertEqual(aggregate_name, body['name']) | 
|  | 203 | self.assertEqual(aggregate['availability_zone'], | 
|  | 204 | body['availability_zone']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 205 | self.assertIn(self.host, body['hosts']) | 
|  | 206 |  | 
|  | 207 | resp, body = self.client.remove_host(aggregate['id'], self.host) | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 208 | self.assertEqual(200, resp.status) | 
|  | 209 | self.assertEqual(aggregate_name, body['name']) | 
|  | 210 | self.assertEqual(aggregate['availability_zone'], | 
|  | 211 | body['availability_zone']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 212 | self.assertNotIn(self.host, body['hosts']) | 
|  | 213 |  | 
| Giulio Fidente | ba3985a | 2013-05-29 01:46:36 +0200 | [diff] [blame] | 214 | @attr(type='gate') | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 215 | def test_aggregate_add_host_list(self): | 
|  | 216 | # Add an host to the given aggregate and list. | 
| Matthew Treinish | a03ed79 | 2013-09-23 21:38:15 +0000 | [diff] [blame] | 217 | self.useFixture(fixtures.LockFixture('availability_zone')) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 218 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 219 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
|  | 220 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 221 | self.client.add_host(aggregate['id'], self.host) | 
|  | 222 | self.addCleanup(self.client.remove_host, aggregate['id'], self.host) | 
|  | 223 |  | 
|  | 224 | resp, aggregates = self.client.list_aggregates() | 
|  | 225 | aggs = filter(lambda x: x['id'] == aggregate['id'], aggregates) | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 226 | self.assertEqual(1, len(aggs)) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 227 | agg = aggs[0] | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 228 | self.assertEqual(aggregate_name, agg['name']) | 
|  | 229 | self.assertEqual(None, agg['availability_zone']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 230 | self.assertIn(self.host, agg['hosts']) | 
|  | 231 |  | 
| Giulio Fidente | ba3985a | 2013-05-29 01:46:36 +0200 | [diff] [blame] | 232 | @attr(type='gate') | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 233 | def test_aggregate_add_host_get_details(self): | 
|  | 234 | # Add an host to the given aggregate and get details. | 
| Matthew Treinish | a03ed79 | 2013-09-23 21:38:15 +0000 | [diff] [blame] | 235 | self.useFixture(fixtures.LockFixture('availability_zone')) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 236 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 237 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
|  | 238 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 239 | self.client.add_host(aggregate['id'], self.host) | 
|  | 240 | self.addCleanup(self.client.remove_host, aggregate['id'], self.host) | 
|  | 241 |  | 
|  | 242 | resp, body = self.client.get_aggregate(aggregate['id']) | 
| Chang Bo Guo | fc77e93 | 2013-09-16 17:38:26 -0700 | [diff] [blame] | 243 | self.assertEqual(aggregate_name, body['name']) | 
|  | 244 | self.assertEqual(None, body['availability_zone']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 245 | self.assertIn(self.host, body['hosts']) | 
|  | 246 |  | 
| Giulio Fidente | ba3985a | 2013-05-29 01:46:36 +0200 | [diff] [blame] | 247 | @attr(type='gate') | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 248 | def test_aggregate_add_host_create_server_with_az(self): | 
|  | 249 | # Add an host to the given aggregate and create a server. | 
| Matthew Treinish | a03ed79 | 2013-09-23 21:38:15 +0000 | [diff] [blame] | 250 | self.useFixture(fixtures.LockFixture('availability_zone')) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 251 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 252 | az_name = rand_name(self.az_name_prefix) | 
|  | 253 | resp, aggregate = self.client.create_aggregate(aggregate_name, az_name) | 
|  | 254 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 255 | self.client.add_host(aggregate['id'], self.host) | 
|  | 256 | self.addCleanup(self.client.remove_host, aggregate['id'], self.host) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 257 | server_name = rand_name('test_server_') | 
|  | 258 | servers_client = self.servers_client | 
|  | 259 | admin_servers_client = self.os_adm.servers_client | 
|  | 260 | resp, server = self.create_server(name=server_name, | 
|  | 261 | availability_zone=az_name) | 
|  | 262 | servers_client.wait_for_server_status(server['id'], 'ACTIVE') | 
|  | 263 | resp, body = admin_servers_client.get_server(server['id']) | 
| Mitsuhiko Yamazaki | ae8fc53 | 2013-04-22 11:17:35 +0900 | [diff] [blame] | 264 | self.assertEqual(self.host, body[self._host_key]) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 265 |  | 
| Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 266 | @attr(type=['negative', 'gate']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 267 | def test_aggregate_add_non_exist_host(self): | 
|  | 268 | # Adding a non-exist host to an aggregate should raise exceptions. | 
|  | 269 | resp, hosts_all = self.os_adm.hosts_client.list_hosts() | 
|  | 270 | hosts = map(lambda x: x['host_name'], hosts_all) | 
|  | 271 | while True: | 
|  | 272 | non_exist_host = rand_name('nonexist_host_') | 
|  | 273 | if non_exist_host not in hosts: | 
|  | 274 | break | 
|  | 275 |  | 
|  | 276 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 277 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
|  | 278 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 279 |  | 
|  | 280 | self.assertRaises(exceptions.NotFound, self.client.add_host, | 
|  | 281 | aggregate['id'], non_exist_host) | 
|  | 282 |  | 
| Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 283 | @attr(type=['negative', 'gate']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 284 | def test_aggregate_add_host_as_user(self): | 
|  | 285 | # Regular user is not allowed to add a host to an aggregate. | 
|  | 286 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 287 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
|  | 288 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 289 |  | 
|  | 290 | self.assertRaises(exceptions.Unauthorized, | 
|  | 291 | self.user_client.add_host, | 
|  | 292 | aggregate['id'], self.host) | 
|  | 293 |  | 
| Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 294 | @attr(type=['negative', 'gate']) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 295 | def test_aggregate_remove_host_as_user(self): | 
|  | 296 | # Regular user is not allowed to remove a host from an aggregate. | 
| Matthew Treinish | a03ed79 | 2013-09-23 21:38:15 +0000 | [diff] [blame] | 297 | self.useFixture(fixtures.LockFixture('availability_zone')) | 
| Mitsuhiko Yamazaki | 74f0707 | 2013-04-02 11:52:31 +0900 | [diff] [blame] | 298 | aggregate_name = rand_name(self.aggregate_name_prefix) | 
|  | 299 | resp, aggregate = self.client.create_aggregate(aggregate_name) | 
|  | 300 | self.addCleanup(self.client.delete_aggregate, aggregate['id']) | 
|  | 301 | self.client.add_host(aggregate['id'], self.host) | 
|  | 302 | self.addCleanup(self.client.remove_host, aggregate['id'], self.host) | 
|  | 303 |  | 
|  | 304 | self.assertRaises(exceptions.Unauthorized, | 
|  | 305 | self.user_client.remove_host, | 
|  | 306 | aggregate['id'], self.host) | 
| Mitsuhiko Yamazaki | ae8fc53 | 2013-04-22 11:17:35 +0900 | [diff] [blame] | 307 |  | 
|  | 308 |  | 
|  | 309 | class AggregatesAdminTestXML(AggregatesAdminTestJSON): | 
|  | 310 | _host_key = ( | 
|  | 311 | '{http://docs.openstack.org/compute/ext/extended_status/api/v1.1}host') | 
|  | 312 | _interface = 'xml' |