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' |