Jay Pipes | 13b479b | 2012-06-11 14:52:27 -0400 | [diff] [blame] | 1 | # vim: tabstop=4 shiftwidth=4 softtabstop=4 |
| 2 | |
ZhiQiang Fan | 39f9722 | 2013-09-20 04:49:44 +0800 | [diff] [blame] | 3 | # Copyright 2012 OpenStack Foundation |
Jay Pipes | 13b479b | 2012-06-11 14:52:27 -0400 | [diff] [blame] | 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. |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 17 | |
Sean Dague | 1937d09 | 2013-05-17 16:36:38 -0400 | [diff] [blame] | 18 | from tempest.api.compute import base |
Matthew Treinish | 481466b | 2012-12-20 17:16:01 -0500 | [diff] [blame] | 19 | from tempest import clients |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 20 | from tempest.common.utils import data_utils |
Daryl Walleck | dc9e0c4 | 2012-04-02 16:51:26 -0500 | [diff] [blame] | 21 | from tempest import exceptions |
Giulio Fidente | 92f7719 | 2013-08-26 17:13:28 +0200 | [diff] [blame] | 22 | from tempest.openstack.common import log as logging |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 23 | from tempest.test import attr |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 24 | |
Giulio Fidente | 92f7719 | 2013-08-26 17:13:28 +0200 | [diff] [blame] | 25 | LOG = logging.getLogger(__name__) |
| 26 | |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 27 | |
ivan-zhu | f2b0050 | 2013-10-18 10:06:52 +0800 | [diff] [blame] | 28 | class AuthorizationTestJSON(base.BaseV2ComputeTest): |
Attila Fazekas | 19044d5 | 2013-02-16 07:35:06 +0100 | [diff] [blame] | 29 | _interface = 'json' |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 30 | |
| 31 | @classmethod |
| 32 | def setUpClass(cls): |
Salvatore Orlando | 5a33724 | 2014-01-15 22:49:22 +0000 | [diff] [blame^] | 33 | # No network resources required for this test |
| 34 | cls.set_network_resources() |
Matthew Treinish | f7fca6a | 2013-12-09 16:27:23 +0000 | [diff] [blame] | 35 | super(AuthorizationTestJSON, cls).setUpClass() |
| 36 | if not cls.multi_user: |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 37 | msg = "Need >1 user" |
ivan-zhu | 1feeb38 | 2013-01-24 10:14:39 +0800 | [diff] [blame] | 38 | raise cls.skipException(msg) |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 39 | cls.client = cls.os.servers_client |
| 40 | cls.images_client = cls.os.images_client |
rajalakshmi-ganesan | b74a11a | 2012-05-16 10:37:58 +0530 | [diff] [blame] | 41 | cls.keypairs_client = cls.os.keypairs_client |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 42 | cls.security_client = cls.os.security_groups_client |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 43 | |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 44 | if cls.config.compute.allow_tenant_isolation: |
Matthew Treinish | b86cda9 | 2013-07-29 11:22:23 -0400 | [diff] [blame] | 45 | creds = cls.isolated_creds.get_alt_creds() |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 46 | username, tenant_name, password = creds |
Matthew Treinish | 481466b | 2012-12-20 17:16:01 -0500 | [diff] [blame] | 47 | cls.alt_manager = clients.Manager(username=username, |
| 48 | password=password, |
| 49 | tenant_name=tenant_name) |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 50 | else: |
| 51 | # Use the alt_XXX credentials in the config file |
Matthew Treinish | 481466b | 2012-12-20 17:16:01 -0500 | [diff] [blame] | 52 | cls.alt_manager = clients.AltManager() |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 53 | |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 54 | cls.alt_client = cls.alt_manager.servers_client |
| 55 | cls.alt_images_client = cls.alt_manager.images_client |
| 56 | cls.alt_keypairs_client = cls.alt_manager.keypairs_client |
| 57 | cls.alt_security_client = cls.alt_manager.security_groups_client |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 58 | |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 59 | cls.alt_security_client._set_auth() |
Ken'ichi Ohmichi | cfc052e | 2013-10-23 11:50:04 +0900 | [diff] [blame] | 60 | resp, server = cls.create_test_server(wait_until='ACTIVE') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 61 | resp, cls.server = cls.client.get_server(server['id']) |
Jay Pipes | 3f981df | 2012-03-27 18:59:44 -0400 | [diff] [blame] | 62 | |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 63 | name = data_utils.rand_name('image') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 64 | resp, body = cls.client.create_image(server['id'], name) |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 65 | image_id = data_utils.parse_image_id(resp['location']) |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 66 | cls.images_client.wait_for_image_status(image_id, 'ACTIVE') |
| 67 | resp, cls.image = cls.images_client.get_image(image_id) |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 68 | |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 69 | cls.keypairname = data_utils.rand_name('keypair') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 70 | resp, keypair = \ |
| 71 | cls.keypairs_client.create_keypair(cls.keypairname) |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 72 | |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 73 | name = data_utils.rand_name('security') |
| 74 | description = data_utils.rand_name('description') |
nayna-patel | eda1d12 | 2013-03-20 14:44:31 +0000 | [diff] [blame] | 75 | resp, cls.security_group = cls.security_client.create_security_group( |
| 76 | name, description) |
rajalakshmi-ganesan | b74a11a | 2012-05-16 10:37:58 +0530 | [diff] [blame] | 77 | |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 78 | parent_group_id = cls.security_group['id'] |
| 79 | ip_protocol = 'tcp' |
| 80 | from_port = 22 |
| 81 | to_port = 22 |
nayna-patel | eda1d12 | 2013-03-20 14:44:31 +0000 | [diff] [blame] | 82 | resp, cls.rule = cls.security_client.create_security_group_rule( |
| 83 | parent_group_id, ip_protocol, from_port, to_port) |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 84 | |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 85 | @classmethod |
| 86 | def tearDownClass(cls): |
Matthew Treinish | f7fca6a | 2013-12-09 16:27:23 +0000 | [diff] [blame] | 87 | if cls.multi_user: |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 88 | cls.images_client.delete_image(cls.image['id']) |
rajalakshmi-ganesan | b74a11a | 2012-05-16 10:37:58 +0530 | [diff] [blame] | 89 | cls.keypairs_client.delete_keypair(cls.keypairname) |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 90 | cls.security_client.delete_security_group(cls.security_group['id']) |
nayna-patel | eda1d12 | 2013-03-20 14:44:31 +0000 | [diff] [blame] | 91 | super(AuthorizationTestJSON, cls).tearDownClass() |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 92 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 93 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 94 | def test_get_server_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 95 | # A GET request for a server on another user's account should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 96 | self.assertRaises(exceptions.NotFound, self.alt_client.get_server, |
| 97 | self.server['id']) |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 98 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 99 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 100 | def test_delete_server_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 101 | # A DELETE request for another user's server should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 102 | self.assertRaises(exceptions.NotFound, self.alt_client.delete_server, |
| 103 | self.server['id']) |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 104 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 105 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 106 | def test_update_server_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 107 | # An update server request for another user's server should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 108 | self.assertRaises(exceptions.NotFound, self.alt_client.update_server, |
| 109 | self.server['id'], name='test') |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 110 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 111 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 112 | def test_list_server_addresses_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 113 | # A list addresses request for another user's server should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 114 | self.assertRaises(exceptions.NotFound, self.alt_client.list_addresses, |
| 115 | self.server['id']) |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 116 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 117 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 118 | def test_list_server_addresses_by_network_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 119 | # A list address/network request for another user's server should fail |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 120 | server_id = self.server['id'] |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 121 | self.assertRaises(exceptions.NotFound, |
| 122 | self.alt_client.list_addresses_by_network, server_id, |
| 123 | 'public') |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 124 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 125 | @attr(type='gate') |
sapan-kona | 3793976 | 2012-06-28 20:22:43 +0530 | [diff] [blame] | 126 | def test_list_servers_with_alternate_tenant(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 127 | # A list on servers from one tenant should not |
| 128 | # show on alternate tenant |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 129 | # Listing servers from alternate tenant |
sapan-kona | 3793976 | 2012-06-28 20:22:43 +0530 | [diff] [blame] | 130 | alt_server_ids = [] |
| 131 | resp, body = self.alt_client.list_servers() |
| 132 | alt_server_ids = [s['id'] for s in body['servers']] |
| 133 | self.assertNotIn(self.server['id'], alt_server_ids) |
| 134 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 135 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 136 | def test_change_password_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 137 | # A change password request for another user's server should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 138 | self.assertRaises(exceptions.NotFound, self.alt_client.change_password, |
| 139 | self.server['id'], 'newpass') |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 140 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 141 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 142 | def test_reboot_server_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 143 | # A reboot request for another user's server should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 144 | self.assertRaises(exceptions.NotFound, self.alt_client.reboot, |
| 145 | self.server['id'], 'HARD') |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 146 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 147 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 148 | def test_rebuild_server_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 149 | # A rebuild request for another user's server should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 150 | self.assertRaises(exceptions.NotFound, self.alt_client.rebuild, |
| 151 | self.server['id'], self.image_ref_alt) |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 152 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 153 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 154 | def test_resize_server_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 155 | # A resize request for another user's server should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 156 | self.assertRaises(exceptions.NotFound, self.alt_client.resize, |
| 157 | self.server['id'], self.flavor_ref_alt) |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 158 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 159 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 160 | def test_create_image_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 161 | # A create image request for another user's server should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 162 | self.assertRaises(exceptions.NotFound, |
| 163 | self.alt_images_client.create_image, |
| 164 | self.server['id'], 'testImage') |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 165 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 166 | @attr(type='gate') |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 167 | def test_create_server_with_unauthorized_image(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 168 | # Server creation with another user's image should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 169 | self.assertRaises(exceptions.BadRequest, self.alt_client.create_server, |
| 170 | 'test', self.image['id'], self.flavor_ref) |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 171 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 172 | @attr(type='gate') |
Daryl Walleck | ced8eb8 | 2012-03-19 13:52:37 -0500 | [diff] [blame] | 173 | def test_create_server_fails_when_tenant_incorrect(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 174 | # A create server request should fail if the tenant id does not match |
| 175 | # the current user |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 176 | saved_base_url = self.alt_client.base_url |
Jay Pipes | ff10d55 | 2012-04-06 14:18:50 -0400 | [diff] [blame] | 177 | try: |
Jay Pipes | ff10d55 | 2012-04-06 14:18:50 -0400 | [diff] [blame] | 178 | # Change the base URL to impersonate another user |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 179 | self.alt_client.base_url = self.client.base_url |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 180 | self.assertRaises(exceptions.BadRequest, |
| 181 | self.alt_client.create_server, 'test', |
| 182 | self.image['id'], self.flavor_ref) |
Jay Pipes | ff10d55 | 2012-04-06 14:18:50 -0400 | [diff] [blame] | 183 | finally: |
| 184 | # Reset the base_url... |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 185 | self.alt_client.base_url = saved_base_url |
rajalakshmi-ganesan | b74a11a | 2012-05-16 10:37:58 +0530 | [diff] [blame] | 186 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 187 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 188 | def test_create_keypair_in_analt_user_tenant(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 189 | # A create keypair request should fail if the tenant id does not match |
| 190 | # the current user |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 191 | # POST keypair with other user tenant |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 192 | k_name = data_utils.rand_name('keypair-') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 193 | self.alt_keypairs_client._set_auth() |
| 194 | self.saved_base_url = self.alt_keypairs_client.base_url |
rajalakshmi-ganesan | b74a11a | 2012-05-16 10:37:58 +0530 | [diff] [blame] | 195 | try: |
| 196 | # Change the base URL to impersonate another user |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 197 | self.alt_keypairs_client.base_url = self.keypairs_client.base_url |
rajalakshmi-ganesan | b74a11a | 2012-05-16 10:37:58 +0530 | [diff] [blame] | 198 | resp = {} |
| 199 | resp['status'] = None |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 200 | self.assertRaises(exceptions.BadRequest, |
| 201 | self.alt_keypairs_client.create_keypair, k_name) |
rajalakshmi-ganesan | b74a11a | 2012-05-16 10:37:58 +0530 | [diff] [blame] | 202 | finally: |
| 203 | # Reset the base_url... |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 204 | self.alt_keypairs_client.base_url = self.saved_base_url |
Zhongyue Luo | e471d6e | 2012-09-17 17:02:43 +0800 | [diff] [blame] | 205 | if (resp['status'] is not None): |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 206 | resp, _ = self.alt_keypairs_client.delete_keypair(k_name) |
Giulio Fidente | 92f7719 | 2013-08-26 17:13:28 +0200 | [diff] [blame] | 207 | LOG.error("Create keypair request should not happen " |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 208 | "if the tenant id does not match the current user") |
rajalakshmi-ganesan | b74a11a | 2012-05-16 10:37:58 +0530 | [diff] [blame] | 209 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 210 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 211 | def test_get_keypair_of_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 212 | # A GET request for another user's keypair should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 213 | self.assertRaises(exceptions.NotFound, |
| 214 | self.alt_keypairs_client.get_keypair, |
| 215 | self.keypairname) |
rajalakshmi-ganesan | b74a11a | 2012-05-16 10:37:58 +0530 | [diff] [blame] | 216 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 217 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 218 | def test_delete_keypair_of_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 219 | # A DELETE request for another user's keypair should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 220 | self.assertRaises(exceptions.NotFound, |
| 221 | self.alt_keypairs_client.delete_keypair, |
| 222 | self.keypairname) |
rajalakshmi-ganesan | 32f8db6 | 2012-05-18 19:13:40 +0530 | [diff] [blame] | 223 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 224 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 225 | def test_get_image_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 226 | # A GET request for an image on another user's account should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 227 | self.assertRaises(exceptions.NotFound, |
| 228 | self.alt_images_client.get_image, self.image['id']) |
rajalakshmi-ganesan | 32f8db6 | 2012-05-18 19:13:40 +0530 | [diff] [blame] | 229 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 230 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 231 | def test_delete_image_for_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 232 | # A DELETE request for another user's image should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 233 | self.assertRaises(exceptions.NotFound, |
| 234 | self.alt_images_client.delete_image, |
| 235 | self.image['id']) |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 236 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 237 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 238 | def test_create_security_group_in_analt_user_tenant(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 239 | # A create security group request should fail if the tenant id does not |
| 240 | # match the current user |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 241 | # POST security group with other user tenant |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 242 | s_name = data_utils.rand_name('security-') |
| 243 | s_description = data_utils.rand_name('security') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 244 | self.saved_base_url = self.alt_security_client.base_url |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 245 | try: |
| 246 | # Change the base URL to impersonate another user |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 247 | self.alt_security_client.base_url = self.security_client.base_url |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 248 | resp = {} |
| 249 | resp['status'] = None |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 250 | self.assertRaises(exceptions.BadRequest, |
| 251 | self.alt_security_client.create_security_group, |
| 252 | s_name, s_description) |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 253 | finally: |
| 254 | # Reset the base_url... |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 255 | self.alt_security_client.base_url = self.saved_base_url |
Zhongyue Luo | e471d6e | 2012-09-17 17:02:43 +0800 | [diff] [blame] | 256 | if resp['status'] is not None: |
Monty Taylor | b2ca5ca | 2013-04-28 18:00:21 -0700 | [diff] [blame] | 257 | self.alt_security_client.delete_security_group(resp['id']) |
Giulio Fidente | 92f7719 | 2013-08-26 17:13:28 +0200 | [diff] [blame] | 258 | LOG.error("Create Security Group request should not happen if" |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 259 | "the tenant id does not match the current user") |
| 260 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 261 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 262 | def test_get_security_group_of_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 263 | # A GET request for another user's security group should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 264 | self.assertRaises(exceptions.NotFound, |
| 265 | self.alt_security_client.get_security_group, |
| 266 | self.security_group['id']) |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 267 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 268 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 269 | def test_delete_security_group_of_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 270 | # A DELETE request for another user's security group should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 271 | self.assertRaises(exceptions.NotFound, |
| 272 | self.alt_security_client.delete_security_group, |
| 273 | self.security_group['id']) |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 274 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 275 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 276 | def test_create_security_group_rule_in_analt_user_tenant(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 277 | # A create security group rule request should fail if the tenant id |
| 278 | # does not match the current user |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 279 | # POST security group rule with other user tenant |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 280 | parent_group_id = self.security_group['id'] |
| 281 | ip_protocol = 'icmp' |
| 282 | from_port = -1 |
| 283 | to_port = -1 |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 284 | self.saved_base_url = self.alt_security_client.base_url |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 285 | try: |
| 286 | # Change the base URL to impersonate another user |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 287 | self.alt_security_client.base_url = self.security_client.base_url |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 288 | resp = {} |
| 289 | resp['status'] = None |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 290 | self.assertRaises(exceptions.BadRequest, |
| 291 | self.alt_security_client. |
| 292 | create_security_group_rule, |
| 293 | parent_group_id, ip_protocol, from_port, |
| 294 | to_port) |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 295 | finally: |
| 296 | # Reset the base_url... |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 297 | self.alt_security_client.base_url = self.saved_base_url |
Zhongyue Luo | e471d6e | 2012-09-17 17:02:43 +0800 | [diff] [blame] | 298 | if resp['status'] is not None: |
Monty Taylor | b2ca5ca | 2013-04-28 18:00:21 -0700 | [diff] [blame] | 299 | self.alt_security_client.delete_security_group_rule(resp['id']) |
Giulio Fidente | 92f7719 | 2013-08-26 17:13:28 +0200 | [diff] [blame] | 300 | LOG.error("Create security group rule request should not " |
rajalakshmi-ganesan | 184daad | 2012-05-18 14:47:38 +0530 | [diff] [blame] | 301 | "happen if the tenant id does not match the" |
| 302 | " current user") |
| 303 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 304 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 305 | def test_delete_security_group_rule_of_alt_account_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 306 | # A DELETE request for another user's security group rule |
| 307 | # should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 308 | self.assertRaises(exceptions.NotFound, |
| 309 | self.alt_security_client.delete_security_group_rule, |
| 310 | self.rule['id']) |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 311 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 312 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 313 | def test_set_metadata_of_alt_account_server_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 314 | # A set metadata for another user's server should fail |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 315 | req_metadata = {'meta1': 'data1', 'meta2': 'data2'} |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 316 | self.assertRaises(exceptions.NotFound, |
| 317 | self.alt_client.set_server_metadata, |
| 318 | self.server['id'], |
| 319 | req_metadata) |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 320 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 321 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 322 | def test_set_metadata_of_alt_account_image_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 323 | # A set metadata for another user's image should fail |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 324 | req_metadata = {'meta1': 'value1', 'meta2': 'value2'} |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 325 | self.assertRaises(exceptions.NotFound, |
| 326 | self.alt_images_client.set_image_metadata, |
| 327 | self.image['id'], req_metadata) |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 328 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 329 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 330 | def test_get_metadata_of_alt_account_server_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 331 | # A get metadata for another user's server should fail |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 332 | req_metadata = {'meta1': 'data1'} |
Zhongyue Luo | e0884a3 | 2012-09-25 17:24:17 +0800 | [diff] [blame] | 333 | self.client.set_server_metadata(self.server['id'], req_metadata) |
hi2suresh | 31bb7cb | 2013-03-14 04:53:49 +0000 | [diff] [blame] | 334 | self.addCleanup(self.client.delete_server_metadata_item, |
| 335 | self.server['id'], 'meta1') |
| 336 | self.assertRaises(exceptions.NotFound, |
| 337 | self.alt_client.get_server_metadata_item, |
| 338 | self.server['id'], 'meta1') |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 339 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 340 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 341 | def test_get_metadata_of_alt_account_image_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 342 | # A get metadata for another user's image should fail |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 343 | req_metadata = {'meta1': 'value1'} |
hi2suresh | d0e2412 | 2013-03-15 03:06:53 +0000 | [diff] [blame] | 344 | self.addCleanup(self.images_client.delete_image_metadata_item, |
| 345 | self.image['id'], 'meta1') |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 346 | self.images_client.set_image_metadata(self.image['id'], |
Zhongyue Luo | 79d8d36 | 2012-09-25 13:49:27 +0800 | [diff] [blame] | 347 | req_metadata) |
hi2suresh | d0e2412 | 2013-03-15 03:06:53 +0000 | [diff] [blame] | 348 | self.assertRaises(exceptions.NotFound, |
| 349 | self.alt_images_client.get_image_metadata_item, |
| 350 | self.image['id'], 'meta1') |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 351 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 352 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 353 | def test_delete_metadata_of_alt_account_server_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 354 | # A delete metadata for another user's server should fail |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 355 | req_metadata = {'meta1': 'data1'} |
hi2suresh | d0e2412 | 2013-03-15 03:06:53 +0000 | [diff] [blame] | 356 | self.addCleanup(self.client.delete_server_metadata_item, |
| 357 | self.server['id'], 'meta1') |
Zhongyue Luo | e0884a3 | 2012-09-25 17:24:17 +0800 | [diff] [blame] | 358 | self.client.set_server_metadata(self.server['id'], req_metadata) |
hi2suresh | d0e2412 | 2013-03-15 03:06:53 +0000 | [diff] [blame] | 359 | self.assertRaises(exceptions.NotFound, |
| 360 | self.alt_client.delete_server_metadata_item, |
| 361 | self.server['id'], 'meta1') |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 362 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 363 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 364 | def test_delete_metadata_of_alt_account_image_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 365 | # A delete metadata for another user's image should fail |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 366 | req_metadata = {'meta1': 'data1'} |
hi2suresh | d0e2412 | 2013-03-15 03:06:53 +0000 | [diff] [blame] | 367 | self.addCleanup(self.images_client.delete_image_metadata_item, |
| 368 | self.image['id'], 'meta1') |
rajalakshmi-ganesan | 929a32a | 2012-05-29 18:00:25 +0530 | [diff] [blame] | 369 | self.images_client.set_image_metadata(self.image['id'], |
| 370 | req_metadata) |
hi2suresh | d0e2412 | 2013-03-15 03:06:53 +0000 | [diff] [blame] | 371 | self.assertRaises(exceptions.NotFound, |
| 372 | self.alt_images_client.delete_image_metadata_item, |
| 373 | self.image['id'], 'meta1') |
rajalakshmi-ganesan | 72ea31a | 2012-05-25 11:59:10 +0530 | [diff] [blame] | 374 | |
Giampaolo Lauria | e9c7702 | 2013-05-22 01:23:58 -0400 | [diff] [blame] | 375 | @attr(type='gate') |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 376 | def test_get_console_output_of_alt_account_server_fails(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 377 | # A Get Console Output for another user's server should fail |
Chris Yeoh | 8b4eaa5 | 2013-02-06 18:03:10 +1030 | [diff] [blame] | 378 | self.assertRaises(exceptions.NotFound, |
| 379 | self.alt_client.get_console_output, |
| 380 | self.server['id'], 10) |
nayna-patel | eda1d12 | 2013-03-20 14:44:31 +0000 | [diff] [blame] | 381 | |
| 382 | |
| 383 | class AuthorizationTestXML(AuthorizationTestJSON): |
| 384 | _interface = 'xml' |