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 | |
David Kranz | cf0040c | 2012-06-26 09:46:56 -0400 | [diff] [blame] | 16 | import time |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 17 | |
Doug Hellmann | 583ce2c | 2015-03-11 14:55:46 +0000 | [diff] [blame] | 18 | from oslo_log import log as logging |
Matthew Treinish | 01472ff | 2015-02-20 17:26:52 -0500 | [diff] [blame] | 19 | |
Ghanshyam | 05049dd | 2016-02-12 17:44:48 +0900 | [diff] [blame] | 20 | from tempest.api.compute import api_microversion_fixture |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 21 | from tempest.common import compute |
Ken'ichi Ohmichi | 8b9c780 | 2015-07-08 05:57:37 +0000 | [diff] [blame] | 22 | from tempest.common import waiters |
Matthew Treinish | b0a78fc | 2014-01-29 16:49:12 +0000 | [diff] [blame] | 23 | from tempest import config |
Sean Dague | 20e9861 | 2016-01-06 14:33:28 -0500 | [diff] [blame] | 24 | from tempest import exceptions |
Sergey Nikitin | 8654e5b | 2017-06-04 22:09:56 +0400 | [diff] [blame] | 25 | from tempest.lib.common import api_version_request |
Ghanshyam | 1f47cf9 | 2016-02-25 04:57:18 +0900 | [diff] [blame] | 26 | from tempest.lib.common import api_version_utils |
Ken'ichi Ohmichi | 757833a | 2017-03-10 10:30:30 -0800 | [diff] [blame] | 27 | from tempest.lib.common.utils import data_utils |
Jordan Pittier | 9e227c5 | 2016-02-09 14:35:18 +0100 | [diff] [blame] | 28 | from tempest.lib.common.utils import test_utils |
Andrea Frittoli (andreaf) | db9672e | 2016-02-23 14:07:24 -0500 | [diff] [blame] | 29 | from tempest.lib import exceptions as lib_exc |
Attila Fazekas | dc21642 | 2013-01-29 15:12:14 +0100 | [diff] [blame] | 30 | import tempest.test |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 31 | |
Matthew Treinish | b0a78fc | 2014-01-29 16:49:12 +0000 | [diff] [blame] | 32 | CONF = config.CONF |
Tiago Mello | eda03b5 | 2012-08-22 23:47:29 -0300 | [diff] [blame] | 33 | |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 34 | LOG = logging.getLogger(__name__) |
Daryl Walleck | c725196 | 2012-03-12 17:26:54 -0500 | [diff] [blame] | 35 | |
| 36 | |
Ken'ichi Ohmichi | 4d237e7 | 2015-11-05 06:32:33 +0000 | [diff] [blame] | 37 | class BaseV2ComputeTest(api_version_utils.BaseMicroversionTest, |
| 38 | tempest.test.BaseTestCase): |
Sean Dague | f237ccb | 2013-01-04 15:19:14 -0500 | [diff] [blame] | 39 | """Base test case class for all Compute API tests.""" |
Daryl Walleck | c725196 | 2012-03-12 17:26:54 -0500 | [diff] [blame] | 40 | |
Attila Fazekas | 430dae3 | 2013-10-17 15:19:32 +0200 | [diff] [blame] | 41 | force_tenant_isolation = False |
Chris Yeoh | 8a79b9d | 2013-01-18 19:32:47 +1030 | [diff] [blame] | 42 | |
Andrea Frittoli | b21de6c | 2015-02-06 20:12:38 +0000 | [diff] [blame] | 43 | # TODO(andreaf) We should care also for the alt_manager here |
| 44 | # but only once client lazy load in the manager is done |
| 45 | credentials = ['primary'] |
| 46 | |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 47 | @classmethod |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 48 | def skip_checks(cls): |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 49 | super(BaseV2ComputeTest, cls).skip_checks() |
Matthew Treinish | f8ff358 | 2015-08-25 12:41:56 -0400 | [diff] [blame] | 50 | if not CONF.service_available.nova: |
| 51 | raise cls.skipException("Nova is not available") |
ghanshyam | 2959153 | 2016-03-11 17:12:43 +0900 | [diff] [blame] | 52 | cfg_min_version = CONF.compute.min_microversion |
| 53 | cfg_max_version = CONF.compute.max_microversion |
Ken'ichi Ohmichi | 4d237e7 | 2015-11-05 06:32:33 +0000 | [diff] [blame] | 54 | api_version_utils.check_skip_with_microversion(cls.min_microversion, |
| 55 | cls.max_microversion, |
| 56 | cfg_min_version, |
| 57 | cfg_max_version) |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 58 | |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 59 | @classmethod |
| 60 | def setup_credentials(cls): |
| 61 | cls.set_network_resources() |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 62 | super(BaseV2ComputeTest, cls).setup_credentials() |
Daryl Walleck | c725196 | 2012-03-12 17:26:54 -0500 | [diff] [blame] | 63 | |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 64 | @classmethod |
| 65 | def setup_clients(cls): |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 66 | super(BaseV2ComputeTest, cls).setup_clients() |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 67 | cls.servers_client = cls.os_primary.servers_client |
| 68 | cls.server_groups_client = cls.os_primary.server_groups_client |
| 69 | cls.flavors_client = cls.os_primary.flavors_client |
| 70 | cls.compute_images_client = cls.os_primary.compute_images_client |
| 71 | cls.extensions_client = cls.os_primary.extensions_client |
| 72 | cls.floating_ip_pools_client = cls.os_primary.floating_ip_pools_client |
| 73 | cls.floating_ips_client = cls.os_primary.compute_floating_ips_client |
| 74 | cls.keypairs_client = cls.os_primary.keypairs_client |
John Warren | 5cdbf42 | 2016-01-05 12:42:43 -0500 | [diff] [blame] | 75 | cls.security_group_rules_client = ( |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 76 | cls.os_primary.compute_security_group_rules_client) |
| 77 | cls.security_groups_client =\ |
| 78 | cls.os_primary.compute_security_groups_client |
| 79 | cls.quotas_client = cls.os_primary.quotas_client |
| 80 | cls.compute_networks_client = cls.os_primary.compute_networks_client |
| 81 | cls.limits_client = cls.os_primary.limits_client |
| 82 | cls.volumes_extensions_client =\ |
| 83 | cls.os_primary.volumes_extensions_client |
| 84 | cls.snapshots_extensions_client =\ |
| 85 | cls.os_primary.snapshots_extensions_client |
| 86 | cls.interfaces_client = cls.os_primary.interfaces_client |
| 87 | cls.fixed_ips_client = cls.os_primary.fixed_ips_client |
| 88 | cls.availability_zone_client = cls.os_primary.availability_zone_client |
| 89 | cls.agents_client = cls.os_primary.agents_client |
| 90 | cls.aggregates_client = cls.os_primary.aggregates_client |
| 91 | cls.services_client = cls.os_primary.services_client |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 92 | cls.instance_usages_audit_log_client = ( |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 93 | cls.os_primary.instance_usages_audit_log_client) |
| 94 | cls.hypervisor_client = cls.os_primary.hypervisor_client |
| 95 | cls.certificates_client = cls.os_primary.certificates_client |
| 96 | cls.migrations_client = cls.os_primary.migrations_client |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 97 | cls.security_group_default_rules_client = ( |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 98 | cls.os_primary.security_group_default_rules_client) |
| 99 | cls.versions_client = cls.os_primary.compute_versions_client |
Andrea Frittoli | a6b3015 | 2017-08-04 10:46:10 +0100 | [diff] [blame] | 100 | if CONF.service_available.cinder: |
| 101 | cls.volumes_client = cls.os_primary.volumes_client_latest |
Ivan Kolodyazhny | bcfc32e | 2015-08-06 13:31:36 +0300 | [diff] [blame] | 102 | |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 103 | @classmethod |
| 104 | def resource_setup(cls): |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 105 | super(BaseV2ComputeTest, cls).resource_setup() |
Ghanshyam | 05049dd | 2016-02-12 17:44:48 +0900 | [diff] [blame] | 106 | cls.request_microversion = ( |
| 107 | api_version_utils.select_request_microversion( |
| 108 | cls.min_microversion, |
ghanshyam | 2959153 | 2016-03-11 17:12:43 +0900 | [diff] [blame] | 109 | CONF.compute.min_microversion)) |
Matthew Treinish | b0a78fc | 2014-01-29 16:49:12 +0000 | [diff] [blame] | 110 | cls.build_interval = CONF.compute.build_interval |
| 111 | cls.build_timeout = CONF.compute.build_timeout |
Matthew Treinish | b0a78fc | 2014-01-29 16:49:12 +0000 | [diff] [blame] | 112 | cls.image_ref = CONF.compute.image_ref |
| 113 | cls.image_ref_alt = CONF.compute.image_ref_alt |
| 114 | cls.flavor_ref = CONF.compute.flavor_ref |
| 115 | cls.flavor_ref_alt = CONF.compute.flavor_ref_alt |
lanoux | 283273b | 2015-12-04 03:01:54 -0800 | [diff] [blame] | 116 | cls.ssh_user = CONF.validation.image_ssh_user |
| 117 | cls.image_ssh_user = CONF.validation.image_ssh_user |
| 118 | cls.image_ssh_password = CONF.validation.image_ssh_password |
Ken'ichi Ohmichi | 543a5d4 | 2014-05-02 08:44:15 +0900 | [diff] [blame] | 119 | |
Matthew Treinish | f7fca6a | 2013-12-09 16:27:23 +0000 | [diff] [blame] | 120 | @classmethod |
Attila Fazekas | 305e65b | 2013-10-29 13:23:07 +0100 | [diff] [blame] | 121 | def server_check_teardown(cls): |
| 122 | """Checks is the shared server clean enough for subsequent test. |
Ken'ichi Ohmichi | 88363cb | 2015-11-19 08:00:54 +0000 | [diff] [blame] | 123 | |
Attila Fazekas | 305e65b | 2013-10-29 13:23:07 +0100 | [diff] [blame] | 124 | Method will delete the server when it's dirty. |
| 125 | The setUp method is responsible for creating a new server. |
| 126 | Exceptions raised in tearDown class are fails the test case, |
Marian Horban | 6afb023 | 2015-11-10 22:47:12 -0500 | [diff] [blame] | 127 | This method supposed to use only by tearDown methods, when |
Attila Fazekas | 305e65b | 2013-10-29 13:23:07 +0100 | [diff] [blame] | 128 | the shared server_id is stored in the server_id of the class. |
| 129 | """ |
| 130 | if getattr(cls, 'server_id', None) is not None: |
| 131 | try: |
Ken'ichi Ohmichi | 0eb153c | 2015-07-13 02:18:25 +0000 | [diff] [blame] | 132 | waiters.wait_for_server_status(cls.servers_client, |
| 133 | cls.server_id, 'ACTIVE') |
Attila Fazekas | 305e65b | 2013-10-29 13:23:07 +0100 | [diff] [blame] | 134 | except Exception as exc: |
| 135 | LOG.exception(exc) |
| 136 | cls.servers_client.delete_server(cls.server_id) |
Ken'ichi Ohmichi | e91a0c6 | 2015-08-13 02:09:16 +0000 | [diff] [blame] | 137 | waiters.wait_for_server_termination(cls.servers_client, |
| 138 | cls.server_id) |
Attila Fazekas | 305e65b | 2013-10-29 13:23:07 +0100 | [diff] [blame] | 139 | cls.server_id = None |
| 140 | raise |
| 141 | |
| 142 | @classmethod |
Masayuki Igawa | 77d3f0f | 2017-03-10 18:57:00 +0900 | [diff] [blame] | 143 | def clear_resources(cls, resource_name, resources, resource_del_func): |
| 144 | LOG.debug('Clearing %s: %s', resource_name, |
| 145 | ','.join(map(str, resources))) |
| 146 | for res_id in resources: |
Sean Dague | d62bf1c | 2013-06-05 14:36:25 -0400 | [diff] [blame] | 147 | try: |
Jordan Pittier | 9e227c5 | 2016-02-09 14:35:18 +0100 | [diff] [blame] | 148 | test_utils.call_and_ignore_notfound_exc( |
Masayuki Igawa | 77d3f0f | 2017-03-10 18:57:00 +0900 | [diff] [blame] | 149 | resource_del_func, res_id) |
Zhi Kun Liu | 02e7a7b | 2014-01-08 16:08:32 +0800 | [diff] [blame] | 150 | except Exception as exc: |
Masayuki Igawa | 77d3f0f | 2017-03-10 18:57:00 +0900 | [diff] [blame] | 151 | LOG.exception('Exception raised deleting %s: %s', |
| 152 | resource_name, res_id) |
Zhi Kun Liu | 02e7a7b | 2014-01-08 16:08:32 +0800 | [diff] [blame] | 153 | LOG.exception(exc) |
Zhi Kun Liu | 02e7a7b | 2014-01-08 16:08:32 +0800 | [diff] [blame] | 154 | |
| 155 | @classmethod |
Joe Gordon | 8843f0f | 2015-03-17 15:07:34 -0700 | [diff] [blame] | 156 | def create_test_server(cls, validatable=False, volume_backed=False, |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 157 | validation_resources=None, **kwargs): |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 158 | """Wrapper utility that returns a test server. |
Rohit Karajgi | dc300b2 | 2012-05-04 08:11:00 -0700 | [diff] [blame] | 159 | |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 160 | This wrapper utility calls the common create test server and |
| 161 | returns a test server. The purpose of this wrapper is to minimize |
| 162 | the impact on the code of the tests already using this |
| 163 | function. |
Joe Gordon | 8843f0f | 2015-03-17 15:07:34 -0700 | [diff] [blame] | 164 | |
| 165 | :param validatable: Whether the server will be pingable or sshable. |
| 166 | :param volume_backed: Whether the instance is volume backed or not. |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 167 | :param validation_resources: Dictionary of validation resources as |
| 168 | returned by `get_class_validation_resources`. |
| 169 | :param kwargs: Extra arguments are passed down to the |
| 170 | `compute.create_test_server` call. |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 171 | """ |
zhufl | 7ae2268 | 2016-09-18 15:22:33 +0800 | [diff] [blame] | 172 | if 'name' not in kwargs: |
| 173 | kwargs['name'] = data_utils.rand_name(cls.__name__ + "-server") |
Sergey Nikitin | 8654e5b | 2017-06-04 22:09:56 +0400 | [diff] [blame] | 174 | |
| 175 | request_version = api_version_request.APIVersionRequest( |
| 176 | cls.request_microversion) |
| 177 | v2_37_version = api_version_request.APIVersionRequest('2.37') |
| 178 | |
| 179 | # NOTE(snikitin): since microversion v2.37 'networks' field is required |
| 180 | if request_version >= v2_37_version and 'networks' not in kwargs: |
| 181 | kwargs['networks'] = 'none' |
| 182 | |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 183 | tenant_network = cls.get_tenant_network() |
| 184 | body, servers = compute.create_test_server( |
zhufl | 0419088 | 2017-05-23 10:21:48 +0800 | [diff] [blame] | 185 | cls.os_primary, |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 186 | validatable, |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 187 | validation_resources=validation_resources, |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 188 | tenant_network=tenant_network, |
Joe Gordon | 8843f0f | 2015-03-17 15:07:34 -0700 | [diff] [blame] | 189 | volume_backed=volume_backed, |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 190 | **kwargs) |
Ken'ichi Ohmichi | 51c8c26 | 2013-12-21 03:30:37 +0900 | [diff] [blame] | 191 | |
Andrea Frittoli | 0d0a3f3 | 2017-08-29 18:21:37 +0100 | [diff] [blame] | 192 | # For each server schedule wait and delete, so we first delete all |
| 193 | # and then wait for all |
| 194 | for server in servers: |
| 195 | cls.addClassResourceCleanup(waiters.wait_for_server_termination, |
| 196 | cls.servers_client, server['id']) |
| 197 | for server in servers: |
| 198 | cls.addClassResourceCleanup( |
| 199 | test_utils.call_and_ignore_notfound_exc, |
| 200 | cls.servers_client.delete_server, server['id']) |
Sean Dague | 9b669e3 | 2012-12-13 18:40:08 -0500 | [diff] [blame] | 201 | |
David Kranz | 0fb1429 | 2015-02-11 15:55:20 -0500 | [diff] [blame] | 202 | return body |
Sean Dague | 9b669e3 | 2012-12-13 18:40:08 -0500 | [diff] [blame] | 203 | |
Zhi Kun Liu | 02e7a7b | 2014-01-08 16:08:32 +0800 | [diff] [blame] | 204 | @classmethod |
| 205 | def create_security_group(cls, name=None, description=None): |
| 206 | if name is None: |
| 207 | name = data_utils.rand_name(cls.__name__ + "-securitygroup") |
| 208 | if description is None: |
Ken'ichi Ohmichi | 4937f56 | 2015-03-23 00:15:01 +0000 | [diff] [blame] | 209 | description = data_utils.rand_name('description') |
ghanshyam | b610b77 | 2015-08-24 17:29:38 +0900 | [diff] [blame] | 210 | body = cls.security_groups_client.create_security_group( |
| 211 | name=name, description=description)['security_group'] |
Andrea Frittoli | 238818c | 2017-08-29 18:28:11 +0100 | [diff] [blame] | 212 | cls.addClassResourceCleanup( |
| 213 | test_utils.call_and_ignore_notfound_exc, |
| 214 | cls.security_groups_client.delete_security_group, |
| 215 | body['id']) |
Zhi Kun Liu | 02e7a7b | 2014-01-08 16:08:32 +0800 | [diff] [blame] | 216 | |
David Kranz | 9964b4e | 2015-02-06 15:45:29 -0500 | [diff] [blame] | 217 | return body |
Zhi Kun Liu | 02e7a7b | 2014-01-08 16:08:32 +0800 | [diff] [blame] | 218 | |
Abhijeet.Jain | 87dd445 | 2014-04-23 15:51:23 +0530 | [diff] [blame] | 219 | @classmethod |
Ghanshyam | 2a180b8 | 2014-06-16 13:54:22 +0900 | [diff] [blame] | 220 | def create_test_server_group(cls, name="", policy=None): |
Abhijeet.Jain | 87dd445 | 2014-04-23 15:51:23 +0530 | [diff] [blame] | 221 | if not name: |
| 222 | name = data_utils.rand_name(cls.__name__ + "-Server-Group") |
Ghanshyam | 2a180b8 | 2014-06-16 13:54:22 +0900 | [diff] [blame] | 223 | if policy is None: |
Abhijeet.Jain | 87dd445 | 2014-04-23 15:51:23 +0530 | [diff] [blame] | 224 | policy = ['affinity'] |
Ken'ichi Ohmichi | 1f36daa | 2015-09-30 01:41:34 +0000 | [diff] [blame] | 225 | body = cls.server_groups_client.create_server_group( |
| 226 | name=name, policies=policy)['server_group'] |
Andrea Frittoli | 238818c | 2017-08-29 18:28:11 +0100 | [diff] [blame] | 227 | cls.addClassResourceCleanup( |
| 228 | test_utils.call_and_ignore_notfound_exc, |
| 229 | cls.server_groups_client.delete_server_group, |
| 230 | body['id']) |
David Kranz | ae99b9a | 2015-02-16 13:37:01 -0500 | [diff] [blame] | 231 | return body |
Abhijeet.Jain | 87dd445 | 2014-04-23 15:51:23 +0530 | [diff] [blame] | 232 | |
David Kranz | cf0040c | 2012-06-26 09:46:56 -0400 | [diff] [blame] | 233 | def wait_for(self, condition): |
Sean Dague | f237ccb | 2013-01-04 15:19:14 -0500 | [diff] [blame] | 234 | """Repeatedly calls condition() until a timeout.""" |
David Kranz | cf0040c | 2012-06-26 09:46:56 -0400 | [diff] [blame] | 235 | start_time = int(time.time()) |
| 236 | while True: |
| 237 | try: |
| 238 | condition() |
Matthew Treinish | 05d9fb9 | 2012-12-07 16:14:05 -0500 | [diff] [blame] | 239 | except Exception: |
David Kranz | cf0040c | 2012-06-26 09:46:56 -0400 | [diff] [blame] | 240 | pass |
| 241 | else: |
| 242 | return |
| 243 | if int(time.time()) - start_time >= self.build_timeout: |
| 244 | condition() |
| 245 | return |
| 246 | time.sleep(self.build_interval) |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 247 | |
Matt Riedemann | 807d056 | 2014-01-27 12:03:10 -0800 | [diff] [blame] | 248 | @staticmethod |
| 249 | def _delete_volume(volumes_client, volume_id): |
| 250 | """Deletes the given volume and waits for it to be gone.""" |
| 251 | try: |
Joseph Lanoux | 6809bab | 2014-12-18 14:57:18 +0000 | [diff] [blame] | 252 | volumes_client.delete_volume(volume_id) |
Matt Riedemann | 807d056 | 2014-01-27 12:03:10 -0800 | [diff] [blame] | 253 | # TODO(mriedem): We should move the wait_for_resource_deletion |
| 254 | # into the delete_volume method as a convenience to the caller. |
| 255 | volumes_client.wait_for_resource_deletion(volume_id) |
Masayuki Igawa | bfa0760 | 2015-01-20 18:47:17 +0900 | [diff] [blame] | 256 | except lib_exc.NotFound: |
zhangguoqing | 6c09664 | 2016-01-04 06:17:21 +0000 | [diff] [blame] | 257 | LOG.warning("Unable to delete volume '%s' since it was not found. " |
Jordan Pittier | 525ec71 | 2016-12-07 17:51:26 +0100 | [diff] [blame] | 258 | "Maybe it was already deleted?", volume_id) |
Matt Riedemann | 807d056 | 2014-01-27 12:03:10 -0800 | [diff] [blame] | 259 | |
Attila Fazekas | 423834d | 2014-03-14 17:33:13 +0100 | [diff] [blame] | 260 | @classmethod |
| 261 | def prepare_instance_network(cls): |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 262 | if (CONF.validation.auth_method != 'disabled' and |
| 263 | CONF.validation.connect_method == 'floating'): |
Attila Fazekas | 423834d | 2014-03-14 17:33:13 +0100 | [diff] [blame] | 264 | cls.set_network_resources(network=True, subnet=True, router=True, |
| 265 | dhcp=True) |
| 266 | |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 267 | @classmethod |
| 268 | def create_image_from_server(cls, server_id, **kwargs): |
| 269 | """Wrapper utility that returns an image created from the server.""" |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 270 | name = kwargs.pop('name', |
| 271 | data_utils.rand_name(cls.__name__ + "-image")) |
| 272 | wait_until = kwargs.pop('wait_until', None) |
| 273 | wait_for_server = kwargs.pop('wait_for_server', True) |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 274 | |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 275 | image = cls.compute_images_client.create_image(server_id, name=name, |
| 276 | **kwargs) |
David Kranz | a5299eb | 2015-01-15 17:24:05 -0500 | [diff] [blame] | 277 | image_id = data_utils.parse_image_id(image.response['location']) |
Andrea Frittoli | b17f7a3 | 2017-08-29 17:45:58 +0100 | [diff] [blame] | 278 | cls.addClassResourceCleanup(test_utils.call_and_ignore_notfound_exc, |
| 279 | cls.compute_images_client.delete_image, |
| 280 | image_id) |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 281 | |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 282 | if wait_until is not None: |
Matt Riedemann | 1395435 | 2017-02-07 14:03:54 -0500 | [diff] [blame] | 283 | try: |
| 284 | waiters.wait_for_image_status(cls.compute_images_client, |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 285 | image_id, wait_until) |
Matt Riedemann | 1395435 | 2017-02-07 14:03:54 -0500 | [diff] [blame] | 286 | except lib_exc.NotFound: |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 287 | if wait_until.upper() == 'ACTIVE': |
Matt Riedemann | 1395435 | 2017-02-07 14:03:54 -0500 | [diff] [blame] | 288 | # If the image is not found after create_image returned |
| 289 | # that means the snapshot failed in nova-compute and nova |
| 290 | # deleted the image. There should be a compute fault |
| 291 | # recorded with the server in that case, so get the server |
| 292 | # and dump some details. |
| 293 | server = ( |
| 294 | cls.servers_client.show_server(server_id)['server']) |
| 295 | if 'fault' in server: |
| 296 | raise exceptions.SnapshotNotFoundException( |
| 297 | server['fault'], image_id=image_id) |
| 298 | else: |
| 299 | raise exceptions.SnapshotNotFoundException( |
| 300 | image_id=image_id) |
| 301 | else: |
| 302 | raise |
Ghanshyam | ae76c12 | 2015-12-22 13:41:35 +0900 | [diff] [blame] | 303 | image = cls.compute_images_client.show_image(image_id)['image'] |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 304 | |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 305 | if wait_until.upper() == 'ACTIVE': |
| 306 | if wait_for_server: |
Bob Ball | 5fe6239 | 2017-02-20 09:51:00 +0000 | [diff] [blame] | 307 | waiters.wait_for_server_status(cls.servers_client, |
| 308 | server_id, 'ACTIVE') |
David Kranz | a5299eb | 2015-01-15 17:24:05 -0500 | [diff] [blame] | 309 | return image |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 310 | |
| 311 | @classmethod |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 312 | def rebuild_server(cls, server_id, validatable=False, **kwargs): |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 313 | """Destroy an existing class level server and creates a new one |
| 314 | |
| 315 | Some test classes use a test server that can be used by multiple |
| 316 | tests. This is done to optimise runtime and test load. |
| 317 | If something goes wrong with the test server, it can be rebuilt |
| 318 | using this helper. |
| 319 | |
| 320 | This helper can also be used for the initial provisioning if no |
| 321 | server_id is specified. |
| 322 | |
| 323 | :param server_id: UUID of the server to be rebuilt. If None is |
| 324 | specified, a new server is provisioned. |
| 325 | :param validatable: whether to the server needs to be |
| 326 | validatable. When True, validation resources are acquired via |
| 327 | the `get_class_validation_resources` helper. |
| 328 | :param kwargs: extra paramaters are passed through to the |
| 329 | `create_test_server` call. |
| 330 | :return: the UUID of the created server. |
| 331 | """ |
Matthew Treinish | 2cd19a4 | 2013-12-02 21:54:42 +0000 | [diff] [blame] | 332 | if server_id: |
zhufl | 9b68290 | 2016-12-15 09:16:34 +0800 | [diff] [blame] | 333 | cls.delete_server(server_id) |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 334 | |
Ghanshyam | 3390d9f | 2015-12-25 12:48:02 +0900 | [diff] [blame] | 335 | cls.password = data_utils.rand_password() |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 336 | server = cls.create_test_server( |
| 337 | validatable, |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 338 | validation_resources=cls.get_class_validation_resources( |
| 339 | cls.os_primary), |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 340 | wait_until='ACTIVE', |
Ghanshyam | 3390d9f | 2015-12-25 12:48:02 +0900 | [diff] [blame] | 341 | adminPass=cls.password, |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 342 | **kwargs) |
Matthew Treinish | 2cd19a4 | 2013-12-02 21:54:42 +0000 | [diff] [blame] | 343 | return server['id'] |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 344 | |
Matt Riedemann | 5dc594c | 2014-01-27 11:40:28 -0800 | [diff] [blame] | 345 | @classmethod |
Jesse Keating | 613b498 | 2015-05-04 15:05:19 -0700 | [diff] [blame] | 346 | def delete_server(cls, server_id): |
| 347 | """Deletes an existing server and waits for it to be gone.""" |
| 348 | try: |
| 349 | cls.servers_client.delete_server(server_id) |
Ken'ichi Ohmichi | e91a0c6 | 2015-08-13 02:09:16 +0000 | [diff] [blame] | 350 | waiters.wait_for_server_termination(cls.servers_client, |
| 351 | server_id) |
Jesse Keating | 613b498 | 2015-05-04 15:05:19 -0700 | [diff] [blame] | 352 | except Exception: |
Jordan Pittier | 525ec71 | 2016-12-07 17:51:26 +0100 | [diff] [blame] | 353 | LOG.exception('Failed to delete server %s', server_id) |
Jesse Keating | 613b498 | 2015-05-04 15:05:19 -0700 | [diff] [blame] | 354 | |
| 355 | @classmethod |
zhufl | 3d018b0 | 2016-11-25 16:43:04 +0800 | [diff] [blame] | 356 | def resize_server(cls, server_id, new_flavor_id, **kwargs): |
| 357 | """resize and confirm_resize an server, waits for it to be ACTIVE.""" |
| 358 | cls.servers_client.resize_server(server_id, new_flavor_id, **kwargs) |
| 359 | waiters.wait_for_server_status(cls.servers_client, server_id, |
| 360 | 'VERIFY_RESIZE') |
| 361 | cls.servers_client.confirm_resize_server(server_id) |
| 362 | waiters.wait_for_server_status(cls.servers_client, server_id, 'ACTIVE') |
| 363 | |
| 364 | @classmethod |
Matt Riedemann | 5dc594c | 2014-01-27 11:40:28 -0800 | [diff] [blame] | 365 | def delete_volume(cls, volume_id): |
| 366 | """Deletes the given volume and waits for it to be gone.""" |
Felipe Monteiro | a40e9b7 | 2017-05-05 17:47:10 +0100 | [diff] [blame] | 367 | cls._delete_volume(cls.volumes_client, volume_id) |
Ken'ichi Ohmichi | 543a5d4 | 2014-05-02 08:44:15 +0900 | [diff] [blame] | 368 | |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 369 | @classmethod |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 370 | def get_server_ip(cls, server, validation_resources=None): |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 371 | """Get the server fixed or floating IP. |
| 372 | |
Sean Dague | 20e9861 | 2016-01-06 14:33:28 -0500 | [diff] [blame] | 373 | Based on the configuration we're in, return a correct ip |
| 374 | address for validating that a guest is up. |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 375 | |
| 376 | :param server: The server dict as returned by the API |
| 377 | :param validation_resources: The dict of validation resources |
| 378 | provisioned for the server. |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 379 | """ |
| 380 | if CONF.validation.connect_method == 'floating': |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 381 | if validation_resources: |
| 382 | return validation_resources['floating_ip']['ip'] |
| 383 | else: |
| 384 | msg = ('When validation.connect_method equals floating, ' |
| 385 | 'validation_resources cannot be None') |
| 386 | raise exceptions.InvalidParam(invalid_param=msg) |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 387 | elif CONF.validation.connect_method == 'fixed': |
Sean Dague | 20e9861 | 2016-01-06 14:33:28 -0500 | [diff] [blame] | 388 | addresses = server['addresses'][CONF.validation.network_for_ssh] |
| 389 | for address in addresses: |
| 390 | if address['version'] == CONF.validation.ip_version_for_ssh: |
| 391 | return address['addr'] |
zhufl | 955f82b | 2016-07-22 11:14:34 +0800 | [diff] [blame] | 392 | raise exceptions.ServerUnreachable(server_id=server['id']) |
Sean Dague | 20e9861 | 2016-01-06 14:33:28 -0500 | [diff] [blame] | 393 | else: |
guo yunxian | ffc4fc0 | 2016-11-15 09:56:08 +0800 | [diff] [blame] | 394 | raise lib_exc.InvalidConfiguration() |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 395 | |
Ghanshyam | 05049dd | 2016-02-12 17:44:48 +0900 | [diff] [blame] | 396 | def setUp(self): |
| 397 | super(BaseV2ComputeTest, self).setUp() |
| 398 | self.useFixture(api_microversion_fixture.APIMicroversionFixture( |
| 399 | self.request_microversion)) |
| 400 | |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 401 | @classmethod |
zhufl | 8d23f92 | 2016-12-12 17:29:42 +0800 | [diff] [blame] | 402 | def create_volume(cls, image_ref=None, **kwargs): |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 403 | """Create a volume and wait for it to become 'available'. |
| 404 | |
Artom Lifshitz | fc8f8e6 | 2016-04-13 11:08:32 +0000 | [diff] [blame] | 405 | :param image_ref: Specify an image id to create a bootable volume. |
zhufl | 8d23f92 | 2016-12-12 17:29:42 +0800 | [diff] [blame] | 406 | :**kwargs: other parameters to create volume. |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 407 | :returns: The available volume. |
| 408 | """ |
zhufl | 8d23f92 | 2016-12-12 17:29:42 +0800 | [diff] [blame] | 409 | if 'size' not in kwargs: |
| 410 | kwargs['size'] = CONF.volume.volume_size |
| 411 | if 'display_name' not in kwargs: |
| 412 | vol_name = data_utils.rand_name(cls.__name__ + '-volume') |
| 413 | kwargs['display_name'] = vol_name |
Artom Lifshitz | fc8f8e6 | 2016-04-13 11:08:32 +0000 | [diff] [blame] | 414 | if image_ref is not None: |
zhufl | 8d23f92 | 2016-12-12 17:29:42 +0800 | [diff] [blame] | 415 | kwargs['imageRef'] = image_ref |
| 416 | volume = cls.volumes_client.create_volume(**kwargs)['volume'] |
Andrea Frittoli | 1fc499e | 2017-08-29 18:33:03 +0100 | [diff] [blame^] | 417 | cls.addClassResourceCleanup( |
| 418 | cls.volumes_client.wait_for_resource_deletion, volume['id']) |
| 419 | cls.addClassResourceCleanup(test_utils.call_and_ignore_notfound_exc, |
| 420 | cls.volumes_client.delete_volume, |
| 421 | volume['id']) |
lkuchlan | 52d7b0d | 2016-11-07 20:53:19 +0200 | [diff] [blame] | 422 | waiters.wait_for_volume_resource_status(cls.volumes_client, |
| 423 | volume['id'], 'available') |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 424 | return volume |
| 425 | |
John Griffith | 030dd17 | 2017-06-27 23:22:14 +0000 | [diff] [blame] | 426 | def attach_volume(self, server, volume, device=None, check_reserved=False): |
Matt Riedemann | cb16a66 | 2016-10-01 18:30:05 -0400 | [diff] [blame] | 427 | """Attaches volume to server and waits for 'in-use' volume status. |
| 428 | |
| 429 | The volume will be detached when the test tears down. |
| 430 | |
| 431 | :param server: The server to which the volume will be attached. |
| 432 | :param volume: The volume to attach. |
| 433 | :param device: Optional mountpoint for the attached volume. Note that |
| 434 | this is not guaranteed for all hypervisors and is not recommended. |
John Griffith | 030dd17 | 2017-06-27 23:22:14 +0000 | [diff] [blame] | 435 | :param check_reserved: Consider a status of reserved as valid for |
| 436 | completion. This is to handle new Cinder attach where we more |
| 437 | accurately use 'reserved' for things like attaching to a shelved |
| 438 | server. |
Matt Riedemann | cb16a66 | 2016-10-01 18:30:05 -0400 | [diff] [blame] | 439 | """ |
| 440 | attach_kwargs = dict(volumeId=volume['id']) |
| 441 | if device: |
| 442 | attach_kwargs['device'] = device |
John Griffith | 030dd17 | 2017-06-27 23:22:14 +0000 | [diff] [blame] | 443 | |
zhufl | 36f0a97 | 2017-02-28 15:43:33 +0800 | [diff] [blame] | 444 | attachment = self.servers_client.attach_volume( |
| 445 | server['id'], **attach_kwargs)['volumeAttachment'] |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 446 | # On teardown detach the volume and wait for it to be available. This |
| 447 | # is so we don't error out when trying to delete the volume during |
| 448 | # teardown. |
lkuchlan | 52d7b0d | 2016-11-07 20:53:19 +0200 | [diff] [blame] | 449 | self.addCleanup(waiters.wait_for_volume_resource_status, |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 450 | self.volumes_client, volume['id'], 'available') |
| 451 | # Ignore 404s on detach in case the server is deleted or the volume |
| 452 | # is already detached. |
| 453 | self.addCleanup(test_utils.call_and_ignore_notfound_exc, |
| 454 | self.servers_client.detach_volume, |
| 455 | server['id'], volume['id']) |
John Griffith | 030dd17 | 2017-06-27 23:22:14 +0000 | [diff] [blame] | 456 | statuses = ['in-use'] |
| 457 | if check_reserved: |
| 458 | statuses.append('reserved') |
lkuchlan | 52d7b0d | 2016-11-07 20:53:19 +0200 | [diff] [blame] | 459 | waiters.wait_for_volume_resource_status(self.volumes_client, |
John Griffith | 030dd17 | 2017-06-27 23:22:14 +0000 | [diff] [blame] | 460 | volume['id'], statuses) |
zhufl | 36f0a97 | 2017-02-28 15:43:33 +0800 | [diff] [blame] | 461 | return attachment |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 462 | |
Ken'ichi Ohmichi | 543a5d4 | 2014-05-02 08:44:15 +0900 | [diff] [blame] | 463 | |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 464 | class BaseV2ComputeAdminTest(BaseV2ComputeTest): |
Ken'ichi Ohmichi | bcefa3d | 2014-05-09 08:14:05 +0900 | [diff] [blame] | 465 | """Base test case class for Compute Admin API tests.""" |
ivan-zhu | f2b0050 | 2013-10-18 10:06:52 +0800 | [diff] [blame] | 466 | |
Andrea Frittoli | b21de6c | 2015-02-06 20:12:38 +0000 | [diff] [blame] | 467 | credentials = ['primary', 'admin'] |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 468 | |
| 469 | @classmethod |
| 470 | def setup_clients(cls): |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 471 | super(BaseV2ComputeAdminTest, cls).setup_clients() |
Ken'ichi Ohmichi | 9f5adf8 | 2014-12-12 04:01:32 +0000 | [diff] [blame] | 472 | cls.availability_zone_admin_client = ( |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 473 | cls.os_admin.availability_zone_client) |
| 474 | cls.admin_flavors_client = cls.os_admin.flavors_client |
| 475 | cls.admin_servers_client = cls.os_admin.servers_client |
zhufl | 36eeab0 | 2017-01-18 11:49:04 +0800 | [diff] [blame] | 476 | |
| 477 | def create_flavor(self, ram, vcpus, disk, name=None, |
| 478 | is_public='True', **kwargs): |
| 479 | if name is None: |
| 480 | name = data_utils.rand_name(self.__class__.__name__ + "-flavor") |
| 481 | id = kwargs.pop('id', data_utils.rand_int_id(start=1000)) |
| 482 | client = self.admin_flavors_client |
| 483 | flavor = client.create_flavor( |
| 484 | ram=ram, vcpus=vcpus, disk=disk, name=name, |
| 485 | id=id, is_public=is_public, **kwargs)['flavor'] |
| 486 | self.addCleanup(client.wait_for_resource_deletion, flavor['id']) |
| 487 | self.addCleanup(client.delete_flavor, flavor['id']) |
| 488 | return flavor |
Duc Truong | 0994120 | 2017-06-07 10:15:20 -0700 | [diff] [blame] | 489 | |
| 490 | def get_host_for_server(self, server_id): |
| 491 | server_details = self.admin_servers_client.show_server(server_id) |
| 492 | return server_details['server']['OS-EXT-SRV-ATTR:host'] |
| 493 | |
| 494 | def get_host_other_than(self, server_id): |
| 495 | source_host = self.get_host_for_server(server_id) |
| 496 | |
zhufl | 1dc2e23 | 2017-09-04 10:19:17 +0800 | [diff] [blame] | 497 | hypers = self.os_admin.hypervisor_client.list_hypervisors( |
| 498 | )['hypervisors'] |
| 499 | hosts = [hyper['hypervisor_hostname'] for hyper in hypers |
| 500 | if hyper['state'] == 'up' and hyper['status'] == 'enabled'] |
Duc Truong | 0994120 | 2017-06-07 10:15:20 -0700 | [diff] [blame] | 501 | |
| 502 | for target_host in hosts: |
| 503 | if source_host != target_host: |
| 504 | return target_host |