ZhiQiang Fan | 39f9722 | 2013-09-20 04:49:44 +0800 | [diff] [blame] | 1 | # Copyright 2012 OpenStack Foundation |
Jay Pipes | 13b479b | 2012-06-11 14:52:27 -0400 | [diff] [blame] | 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 | |
Ken'ichi Ohmichi | 3577260 | 2013-11-14 15:03:27 +0900 | [diff] [blame] | 16 | import uuid |
| 17 | |
Sean Dague | 1937d09 | 2013-05-17 16:36:38 -0400 | [diff] [blame] | 18 | from tempest.api.compute import base |
Fei Long Wang | d39431f | 2015-05-14 11:30:48 +1200 | [diff] [blame] | 19 | from tempest.common.utils import data_utils |
Andrea Frittoli (andreaf) | db9672e | 2016-02-23 14:07:24 -0500 | [diff] [blame] | 20 | from tempest.lib import exceptions as lib_exc |
Matthew Treinish | e3d2614 | 2013-11-26 19:14:58 +0000 | [diff] [blame] | 21 | from tempest import test |
Rohit Karajgi | 0353029 | 2012-04-24 17:00:50 -0700 | [diff] [blame] | 22 | |
| 23 | |
ivan-zhu | f2b0050 | 2013-10-18 10:06:52 +0800 | [diff] [blame] | 24 | class FlavorsAdminTestJSON(base.BaseV2ComputeAdminTest): |
Ken'ichi Ohmichi | 88363cb | 2015-11-19 08:00:54 +0000 | [diff] [blame] | 25 | """Tests Flavors API Create and Delete that require admin privileges""" |
Rohit Karajgi | 0353029 | 2012-04-24 17:00:50 -0700 | [diff] [blame] | 26 | |
rajalakshmi-ganesan | f344cc3 | 2012-12-31 20:02:27 +0530 | [diff] [blame] | 27 | @classmethod |
Rohan Kanade | 60b7309 | 2015-02-04 17:58:19 +0530 | [diff] [blame] | 28 | def skip_checks(cls): |
| 29 | super(FlavorsAdminTestJSON, cls).skip_checks() |
Ken'ichi Ohmichi | a7e6871 | 2014-05-06 10:47:26 +0900 | [diff] [blame] | 30 | if not test.is_extension_enabled('OS-FLV-EXT-DATA', 'compute'): |
| 31 | msg = "OS-FLV-EXT-DATA extension not enabled." |
ivan-zhu | 1feeb38 | 2013-01-24 10:14:39 +0800 | [diff] [blame] | 32 | raise cls.skipException(msg) |
Rohit Karajgi | 0353029 | 2012-04-24 17:00:50 -0700 | [diff] [blame] | 33 | |
Rohan Kanade | 60b7309 | 2015-02-04 17:58:19 +0530 | [diff] [blame] | 34 | @classmethod |
| 35 | def setup_clients(cls): |
| 36 | super(FlavorsAdminTestJSON, cls).setup_clients() |
Attila Fazekas | 4ba3658 | 2013-02-12 08:26:17 +0100 | [diff] [blame] | 37 | cls.client = cls.os_adm.flavors_client |
afazekas | cda4082 | 2013-03-25 16:50:41 +0100 | [diff] [blame] | 38 | cls.user_client = cls.os.flavors_client |
Rohan Kanade | 60b7309 | 2015-02-04 17:58:19 +0530 | [diff] [blame] | 39 | |
| 40 | @classmethod |
| 41 | def resource_setup(cls): |
| 42 | super(FlavorsAdminTestJSON, cls).resource_setup() |
| 43 | |
Chris Yeoh | 8abacf3 | 2013-01-21 17:08:32 +1030 | [diff] [blame] | 44 | cls.flavor_name_prefix = 'test_flavor_' |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 45 | cls.ram = 512 |
| 46 | cls.vcpus = 1 |
| 47 | cls.disk = 10 |
| 48 | cls.ephemeral = 10 |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 49 | cls.swap = 1024 |
rajalakshmi-ganesan | f344cc3 | 2012-12-31 20:02:27 +0530 | [diff] [blame] | 50 | cls.rxtx = 2 |
Rohit Karajgi | 0353029 | 2012-04-24 17:00:50 -0700 | [diff] [blame] | 51 | |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 52 | def flavor_clean_up(self, flavor_id): |
David Kranz | 2fa77b2 | 2015-02-09 11:39:50 -0500 | [diff] [blame] | 53 | self.client.delete_flavor(flavor_id) |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 54 | self.client.wait_for_resource_deletion(flavor_id) |
| 55 | |
Ken'ichi Ohmichi | 3577260 | 2013-11-14 15:03:27 +0900 | [diff] [blame] | 56 | def _create_flavor(self, flavor_id): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 57 | # Create a flavor and ensure it is listed |
| 58 | # This operation requires the user to have 'admin' role |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 59 | flavor_name = data_utils.rand_name(self.flavor_name_prefix) |
Chris Yeoh | 8abacf3 | 2013-01-21 17:08:32 +1030 | [diff] [blame] | 60 | |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 61 | # Create the flavor |
Ken'ichi Ohmichi | 96338c4 | 2015-07-17 06:25:14 +0000 | [diff] [blame] | 62 | flavor = self.client.create_flavor(name=flavor_name, |
| 63 | ram=self.ram, vcpus=self.vcpus, |
| 64 | disk=self.disk, |
| 65 | id=flavor_id, |
David Kranz | 2fa77b2 | 2015-02-09 11:39:50 -0500 | [diff] [blame] | 66 | ephemeral=self.ephemeral, |
| 67 | swap=self.swap, |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 68 | rxtx_factor=self.rxtx)['flavor'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 69 | self.addCleanup(self.flavor_clean_up, flavor['id']) |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 70 | self.assertEqual(flavor['name'], flavor_name) |
| 71 | self.assertEqual(flavor['vcpus'], self.vcpus) |
| 72 | self.assertEqual(flavor['disk'], self.disk) |
| 73 | self.assertEqual(flavor['ram'], self.ram) |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 74 | self.assertEqual(flavor['swap'], self.swap) |
| 75 | self.assertEqual(flavor['rxtx_factor'], self.rxtx) |
| 76 | self.assertEqual(flavor['OS-FLV-EXT-DATA:ephemeral'], |
| 77 | self.ephemeral) |
ivan-zhu | ae7c7c5 | 2013-10-21 22:13:22 +0800 | [diff] [blame] | 78 | self.assertEqual(flavor['os-flavor-access:is_public'], True) |
Rohit Karajgi | 0353029 | 2012-04-24 17:00:50 -0700 | [diff] [blame] | 79 | |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 80 | # Verify flavor is retrieved |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 81 | flavor = self.client.show_flavor(flavor['id'])['flavor'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 82 | self.assertEqual(flavor['name'], flavor_name) |
Rohit Karajgi | 0353029 | 2012-04-24 17:00:50 -0700 | [diff] [blame] | 83 | |
Ken'ichi Ohmichi | 3577260 | 2013-11-14 15:03:27 +0900 | [diff] [blame] | 84 | return flavor['id'] |
| 85 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 86 | @test.idempotent_id('8b4330e1-12c4-4554-9390-e6639971f086') |
Ken'ichi Ohmichi | 3577260 | 2013-11-14 15:03:27 +0900 | [diff] [blame] | 87 | def test_create_flavor_with_int_id(self): |
| 88 | flavor_id = data_utils.rand_int_id(start=1000) |
| 89 | new_flavor_id = self._create_flavor(flavor_id) |
| 90 | self.assertEqual(new_flavor_id, str(flavor_id)) |
| 91 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 92 | @test.idempotent_id('94c9bb4e-2c2a-4f3c-bb1f-5f0daf918e6d') |
Ken'ichi Ohmichi | 3577260 | 2013-11-14 15:03:27 +0900 | [diff] [blame] | 93 | def test_create_flavor_with_uuid_id(self): |
Ken'ichi Ohmichi | d079c89 | 2016-04-19 11:23:36 -0700 | [diff] [blame] | 94 | flavor_id = data_utils.rand_uuid() |
Ken'ichi Ohmichi | 3577260 | 2013-11-14 15:03:27 +0900 | [diff] [blame] | 95 | new_flavor_id = self._create_flavor(flavor_id) |
| 96 | self.assertEqual(new_flavor_id, flavor_id) |
| 97 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 98 | @test.idempotent_id('f83fe669-6758-448a-a85e-32d351f36fe0') |
Ken'ichi Ohmichi | 3577260 | 2013-11-14 15:03:27 +0900 | [diff] [blame] | 99 | def test_create_flavor_with_none_id(self): |
| 100 | # If nova receives a request with None as flavor_id, |
| 101 | # nova generates flavor_id of uuid. |
| 102 | flavor_id = None |
| 103 | new_flavor_id = self._create_flavor(flavor_id) |
| 104 | self.assertEqual(new_flavor_id, str(uuid.UUID(new_flavor_id))) |
| 105 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 106 | @test.idempotent_id('8261d7b0-be58-43ec-a2e5-300573c3f6c5') |
Rohit Karajgi | 0353029 | 2012-04-24 17:00:50 -0700 | [diff] [blame] | 107 | def test_create_flavor_verify_entry_in_list_details(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 108 | # Create a flavor and ensure it's details are listed |
| 109 | # This operation requires the user to have 'admin' role |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 110 | flavor_name = data_utils.rand_name(self.flavor_name_prefix) |
| 111 | new_flavor_id = data_utils.rand_int_id(start=1000) |
Chris Yeoh | 8abacf3 | 2013-01-21 17:08:32 +1030 | [diff] [blame] | 112 | |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 113 | # Create the flavor |
Ken'ichi Ohmichi | 96338c4 | 2015-07-17 06:25:14 +0000 | [diff] [blame] | 114 | flavor = self.client.create_flavor(name=flavor_name, |
| 115 | ram=self.ram, vcpus=self.vcpus, |
| 116 | disk=self.disk, |
| 117 | id=new_flavor_id, |
David Kranz | 2fa77b2 | 2015-02-09 11:39:50 -0500 | [diff] [blame] | 118 | ephemeral=self.ephemeral, |
| 119 | swap=self.swap, |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 120 | rxtx_factor=self.rxtx)['flavor'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 121 | self.addCleanup(self.flavor_clean_up, flavor['id']) |
| 122 | flag = False |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 123 | # Verify flavor is retrieved |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 124 | flavors = self.client.list_flavors(detail=True)['flavors'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 125 | for flavor in flavors: |
| 126 | if flavor['name'] == flavor_name: |
| 127 | flag = True |
| 128 | self.assertTrue(flag) |
Rohit Karajgi | 0353029 | 2012-04-24 17:00:50 -0700 | [diff] [blame] | 129 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 130 | @test.idempotent_id('63dc64e6-2e79-4fdf-868f-85500d308d66') |
rajalakshmi-ganesan | f344cc3 | 2012-12-31 20:02:27 +0530 | [diff] [blame] | 131 | def test_create_list_flavor_without_extra_data(self): |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 132 | # Create a flavor and ensure it is listed |
| 133 | # This operation requires the user to have 'admin' role |
ivan-zhu | ae7c7c5 | 2013-10-21 22:13:22 +0800 | [diff] [blame] | 134 | |
| 135 | def verify_flavor_response_extension(flavor): |
| 136 | # check some extensions for the flavor create/show/detail response |
| 137 | self.assertEqual(flavor['swap'], '') |
| 138 | self.assertEqual(int(flavor['rxtx_factor']), 1) |
| 139 | self.assertEqual(int(flavor['OS-FLV-EXT-DATA:ephemeral']), 0) |
| 140 | self.assertEqual(flavor['os-flavor-access:is_public'], True) |
| 141 | |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 142 | flavor_name = data_utils.rand_name(self.flavor_name_prefix) |
| 143 | new_flavor_id = data_utils.rand_int_id(start=1000) |
Chris Yeoh | 8abacf3 | 2013-01-21 17:08:32 +1030 | [diff] [blame] | 144 | |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 145 | # Create the flavor |
Ken'ichi Ohmichi | 96338c4 | 2015-07-17 06:25:14 +0000 | [diff] [blame] | 146 | flavor = self.client.create_flavor(name=flavor_name, |
| 147 | ram=self.ram, vcpus=self.vcpus, |
| 148 | disk=self.disk, |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 149 | id=new_flavor_id)['flavor'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 150 | self.addCleanup(self.flavor_clean_up, flavor['id']) |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 151 | self.assertEqual(flavor['name'], flavor_name) |
| 152 | self.assertEqual(flavor['ram'], self.ram) |
| 153 | self.assertEqual(flavor['vcpus'], self.vcpus) |
| 154 | self.assertEqual(flavor['disk'], self.disk) |
| 155 | self.assertEqual(int(flavor['id']), new_flavor_id) |
ivan-zhu | ae7c7c5 | 2013-10-21 22:13:22 +0800 | [diff] [blame] | 156 | verify_flavor_response_extension(flavor) |
rajalakshmi-ganesan | f344cc3 | 2012-12-31 20:02:27 +0530 | [diff] [blame] | 157 | |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 158 | # Verify flavor is retrieved |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 159 | flavor = self.client.show_flavor(new_flavor_id)['flavor'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 160 | self.assertEqual(flavor['name'], flavor_name) |
ivan-zhu | ae7c7c5 | 2013-10-21 22:13:22 +0800 | [diff] [blame] | 161 | verify_flavor_response_extension(flavor) |
| 162 | |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 163 | # Check if flavor is present in list |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 164 | flavors = self.user_client.list_flavors(detail=True)['flavors'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 165 | for flavor in flavors: |
| 166 | if flavor['name'] == flavor_name: |
ivan-zhu | ae7c7c5 | 2013-10-21 22:13:22 +0800 | [diff] [blame] | 167 | verify_flavor_response_extension(flavor) |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 168 | flag = True |
| 169 | self.assertTrue(flag) |
rajalakshmi-ganesan | f344cc3 | 2012-12-31 20:02:27 +0530 | [diff] [blame] | 170 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 171 | @test.idempotent_id('be6cc18c-7c5d-48c0-ac16-17eaf03c54eb') |
Sumanth Nagadavalli | 3d955d9 | 2013-08-13 15:16:30 +0530 | [diff] [blame] | 172 | def test_list_non_public_flavor(self): |
Santiago Baldassin | 7f98916 | 2014-03-19 12:02:47 -0300 | [diff] [blame] | 173 | # Create a flavor with os-flavor-access:is_public false. |
| 174 | # The flavor should not be present in list_details as the |
| 175 | # tenant is not automatically added access list. |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 176 | # This operation requires the user to have 'admin' role |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 177 | flavor_name = data_utils.rand_name(self.flavor_name_prefix) |
| 178 | new_flavor_id = data_utils.rand_int_id(start=1000) |
Chris Yeoh | 8abacf3 | 2013-01-21 17:08:32 +1030 | [diff] [blame] | 179 | |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 180 | # Create the flavor |
Ken'ichi Ohmichi | 96338c4 | 2015-07-17 06:25:14 +0000 | [diff] [blame] | 181 | flavor = self.client.create_flavor(name=flavor_name, |
| 182 | ram=self.ram, vcpus=self.vcpus, |
| 183 | disk=self.disk, |
| 184 | id=new_flavor_id, |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 185 | is_public="False")['flavor'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 186 | self.addCleanup(self.flavor_clean_up, flavor['id']) |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 187 | # Verify flavor is retrieved |
Sumanth Nagadavalli | 3d955d9 | 2013-08-13 15:16:30 +0530 | [diff] [blame] | 188 | flag = False |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 189 | flavors = self.client.list_flavors(detail=True)['flavors'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 190 | for flavor in flavors: |
| 191 | if flavor['name'] == flavor_name: |
| 192 | flag = True |
Santiago Baldassin | 7f98916 | 2014-03-19 12:02:47 -0300 | [diff] [blame] | 193 | self.assertFalse(flag) |
Sumanth Nagadavalli | 3d955d9 | 2013-08-13 15:16:30 +0530 | [diff] [blame] | 194 | |
| 195 | # Verify flavor is not retrieved with other user |
| 196 | flag = False |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 197 | flavors = self.user_client.list_flavors(detail=True)['flavors'] |
Sumanth Nagadavalli | 3d955d9 | 2013-08-13 15:16:30 +0530 | [diff] [blame] | 198 | for flavor in flavors: |
| 199 | if flavor['name'] == flavor_name: |
| 200 | flag = True |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 201 | self.assertFalse(flag) |
rajalakshmi-ganesan | f344cc3 | 2012-12-31 20:02:27 +0530 | [diff] [blame] | 202 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 203 | @test.idempotent_id('bcc418ef-799b-47cc-baa1-ce01368b8987') |
Ken'ichi Ohmichi | 32b1e6c | 2013-09-06 12:07:07 +0900 | [diff] [blame] | 204 | def test_create_server_with_non_public_flavor(self): |
| 205 | # Create a flavor with os-flavor-access:is_public false |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 206 | flavor_name = data_utils.rand_name(self.flavor_name_prefix) |
| 207 | new_flavor_id = data_utils.rand_int_id(start=1000) |
Ken'ichi Ohmichi | 32b1e6c | 2013-09-06 12:07:07 +0900 | [diff] [blame] | 208 | |
| 209 | # Create the flavor |
Ken'ichi Ohmichi | 96338c4 | 2015-07-17 06:25:14 +0000 | [diff] [blame] | 210 | flavor = self.client.create_flavor(name=flavor_name, |
| 211 | ram=self.ram, vcpus=self.vcpus, |
| 212 | disk=self.disk, |
| 213 | id=new_flavor_id, |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 214 | is_public="False")['flavor'] |
Ken'ichi Ohmichi | 32b1e6c | 2013-09-06 12:07:07 +0900 | [diff] [blame] | 215 | self.addCleanup(self.flavor_clean_up, flavor['id']) |
Ken'ichi Ohmichi | 32b1e6c | 2013-09-06 12:07:07 +0900 | [diff] [blame] | 216 | |
| 217 | # Verify flavor is not used by other user |
Masayuki Igawa | 4b29e47 | 2015-02-16 10:41:54 +0900 | [diff] [blame] | 218 | self.assertRaises(lib_exc.BadRequest, |
Ken'ichi Ohmichi | 32b1e6c | 2013-09-06 12:07:07 +0900 | [diff] [blame] | 219 | self.os.servers_client.create_server, |
Ken'ichi Ohmichi | f2d436e | 2015-09-03 01:13:16 +0000 | [diff] [blame] | 220 | name='test', imageRef=self.image_ref, |
| 221 | flavorRef=flavor['id']) |
Ken'ichi Ohmichi | 32b1e6c | 2013-09-06 12:07:07 +0900 | [diff] [blame] | 222 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 223 | @test.idempotent_id('b345b196-bfbd-4231-8ac1-6d7fe15ff3a3') |
rajalakshmi-ganesan | f344cc3 | 2012-12-31 20:02:27 +0530 | [diff] [blame] | 224 | def test_list_public_flavor_with_other_user(self): |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 225 | # Create a Flavor with public access. |
| 226 | # Try to List/Get flavor with another user |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 227 | flavor_name = data_utils.rand_name(self.flavor_name_prefix) |
| 228 | new_flavor_id = data_utils.rand_int_id(start=1000) |
Chris Yeoh | 8abacf3 | 2013-01-21 17:08:32 +1030 | [diff] [blame] | 229 | |
Matthew Treinish | c795b9e | 2014-06-09 17:01:10 -0400 | [diff] [blame] | 230 | # Create the flavor |
Ken'ichi Ohmichi | 96338c4 | 2015-07-17 06:25:14 +0000 | [diff] [blame] | 231 | flavor = self.client.create_flavor(name=flavor_name, |
| 232 | ram=self.ram, vcpus=self.vcpus, |
| 233 | disk=self.disk, |
| 234 | id=new_flavor_id, |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 235 | is_public="True")['flavor'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 236 | self.addCleanup(self.flavor_clean_up, flavor['id']) |
| 237 | flag = False |
| 238 | self.new_client = self.flavors_client |
Attila Fazekas | f7f34f9 | 2013-08-01 17:01:44 +0200 | [diff] [blame] | 239 | # Verify flavor is retrieved with new user |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 240 | flavors = self.new_client.list_flavors(detail=True)['flavors'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 241 | for flavor in flavors: |
| 242 | if flavor['name'] == flavor_name: |
| 243 | flag = True |
| 244 | self.assertTrue(flag) |
rajalakshmi-ganesan | f344cc3 | 2012-12-31 20:02:27 +0530 | [diff] [blame] | 245 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 246 | @test.idempotent_id('fb9cbde6-3a0e-41f2-a983-bdb0a823c44e') |
Tiago Mello | 0d835d2 | 2013-02-06 13:57:50 -0500 | [diff] [blame] | 247 | def test_is_public_string_variations(self): |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 248 | flavor_id_not_public = data_utils.rand_int_id(start=1000) |
| 249 | flavor_name_not_public = data_utils.rand_name(self.flavor_name_prefix) |
| 250 | flavor_id_public = data_utils.rand_int_id(start=1000) |
| 251 | flavor_name_public = data_utils.rand_name(self.flavor_name_prefix) |
Tiago Mello | 0d835d2 | 2013-02-06 13:57:50 -0500 | [diff] [blame] | 252 | |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 253 | # Create a non public flavor |
Ken'ichi Ohmichi | 96338c4 | 2015-07-17 06:25:14 +0000 | [diff] [blame] | 254 | flavor = self.client.create_flavor(name=flavor_name_not_public, |
| 255 | ram=self.ram, vcpus=self.vcpus, |
| 256 | disk=self.disk, |
| 257 | id=flavor_id_not_public, |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 258 | is_public="False")['flavor'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 259 | self.addCleanup(self.flavor_clean_up, flavor['id']) |
Tiago Mello | 0d835d2 | 2013-02-06 13:57:50 -0500 | [diff] [blame] | 260 | |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 261 | # Create a public flavor |
Ken'ichi Ohmichi | 96338c4 | 2015-07-17 06:25:14 +0000 | [diff] [blame] | 262 | flavor = self.client.create_flavor(name=flavor_name_public, |
| 263 | ram=self.ram, vcpus=self.vcpus, |
| 264 | disk=self.disk, |
| 265 | id=flavor_id_public, |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 266 | is_public="True")['flavor'] |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 267 | self.addCleanup(self.flavor_clean_up, flavor['id']) |
Tiago Mello | 0d835d2 | 2013-02-06 13:57:50 -0500 | [diff] [blame] | 268 | |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 269 | def _flavor_lookup(flavors, flavor_name): |
| 270 | for flavor in flavors: |
| 271 | if flavor['name'] == flavor_name: |
| 272 | return flavor |
| 273 | return None |
Tiago Mello | 0d835d2 | 2013-02-06 13:57:50 -0500 | [diff] [blame] | 274 | |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 275 | def _test_string_variations(variations, flavor_name): |
| 276 | for string in variations: |
| 277 | params = {'is_public': string} |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 278 | flavors = (self.client.list_flavors(detail=True, **params) |
| 279 | ['flavors']) |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 280 | flavor = _flavor_lookup(flavors, flavor_name) |
Ionuț Arțăriși | 7f7d452 | 2013-08-21 11:47:47 +0200 | [diff] [blame] | 281 | self.assertIsNotNone(flavor) |
Tiago Mello | 0d835d2 | 2013-02-06 13:57:50 -0500 | [diff] [blame] | 282 | |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 283 | _test_string_variations(['f', 'false', 'no', '0'], |
| 284 | flavor_name_not_public) |
Tiago Mello | 0d835d2 | 2013-02-06 13:57:50 -0500 | [diff] [blame] | 285 | |
Attila Fazekas | 2883fd5 | 2013-03-10 12:05:21 +0100 | [diff] [blame] | 286 | _test_string_variations(['t', 'true', 'yes', '1'], |
| 287 | flavor_name_public) |
Tiago Mello | 0d835d2 | 2013-02-06 13:57:50 -0500 | [diff] [blame] | 288 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 289 | @test.idempotent_id('3b541a2e-2ac2-4b42-8b8d-ba6e22fcd4da') |
Zhi Kun Liu | 20f1322 | 2013-09-03 00:36:38 +0800 | [diff] [blame] | 290 | def test_create_flavor_using_string_ram(self): |
Masayuki Igawa | 259c113 | 2013-10-31 17:48:44 +0900 | [diff] [blame] | 291 | flavor_name = data_utils.rand_name(self.flavor_name_prefix) |
| 292 | new_flavor_id = data_utils.rand_int_id(start=1000) |
Zhi Kun Liu | 20f1322 | 2013-09-03 00:36:38 +0800 | [diff] [blame] | 293 | |
Ken'ichi Ohmichi | 2d2d4d9 | 2014-10-29 05:10:24 +0000 | [diff] [blame] | 294 | ram = "1024" |
Ken'ichi Ohmichi | 96338c4 | 2015-07-17 06:25:14 +0000 | [diff] [blame] | 295 | flavor = self.client.create_flavor(name=flavor_name, |
| 296 | ram=ram, vcpus=self.vcpus, |
| 297 | disk=self.disk, |
ghanshyam | 19973be | 2015-08-18 15:46:42 +0900 | [diff] [blame] | 298 | id=new_flavor_id)['flavor'] |
Zhi Kun Liu | 20f1322 | 2013-09-03 00:36:38 +0800 | [diff] [blame] | 299 | self.addCleanup(self.flavor_clean_up, flavor['id']) |
Zhi Kun Liu | 20f1322 | 2013-09-03 00:36:38 +0800 | [diff] [blame] | 300 | self.assertEqual(flavor['name'], flavor_name) |
| 301 | self.assertEqual(flavor['vcpus'], self.vcpus) |
| 302 | self.assertEqual(flavor['disk'], self.disk) |
| 303 | self.assertEqual(flavor['ram'], int(ram)) |
| 304 | self.assertEqual(int(flavor['id']), new_flavor_id) |