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 | |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 20 | from tempest.common import compute |
Ken'ichi Ohmichi | 8b9c780 | 2015-07-08 05:57:37 +0000 | [diff] [blame] | 21 | from tempest.common import waiters |
Matthew Treinish | b0a78fc | 2014-01-29 16:49:12 +0000 | [diff] [blame] | 22 | from tempest import config |
Sean Dague | 20e9861 | 2016-01-06 14:33:28 -0500 | [diff] [blame] | 23 | from tempest import exceptions |
Ghanshyam Mann | ad8737c | 2019-04-13 01:12:58 +0000 | [diff] [blame] | 24 | from tempest.lib.common import api_microversion_fixture |
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 |
Eric Fried | bfaa50f | 2020-01-09 12:04:54 -0600 | [diff] [blame] | 42 | # Set this to True in subclasses to create a default network. See |
| 43 | # https://bugs.launchpad.net/tempest/+bug/1844568 |
| 44 | create_default_network = False |
Chris Yeoh | 8a79b9d | 2013-01-18 19:32:47 +1030 | [diff] [blame] | 45 | |
Andrea Frittoli | b21de6c | 2015-02-06 20:12:38 +0000 | [diff] [blame] | 46 | # TODO(andreaf) We should care also for the alt_manager here |
| 47 | # but only once client lazy load in the manager is done |
| 48 | credentials = ['primary'] |
| 49 | |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 50 | @classmethod |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 51 | def skip_checks(cls): |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 52 | super(BaseV2ComputeTest, cls).skip_checks() |
Matthew Treinish | f8ff358 | 2015-08-25 12:41:56 -0400 | [diff] [blame] | 53 | if not CONF.service_available.nova: |
| 54 | raise cls.skipException("Nova is not available") |
Lee Yarwood | 4b95d4b | 2020-01-15 10:49:54 +0000 | [diff] [blame] | 55 | api_version_utils.check_skip_with_microversion( |
| 56 | cls.min_microversion, cls.max_microversion, |
| 57 | CONF.compute.min_microversion, CONF.compute.max_microversion) |
| 58 | api_version_utils.check_skip_with_microversion( |
| 59 | cls.volume_min_microversion, cls.volume_max_microversion, |
| 60 | CONF.volume.min_microversion, CONF.volume.max_microversion) |
| 61 | api_version_utils.check_skip_with_microversion( |
| 62 | cls.placement_min_microversion, cls.placement_max_microversion, |
| 63 | CONF.placement.min_microversion, CONF.placement.max_microversion) |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 64 | |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 65 | @classmethod |
| 66 | def setup_credentials(cls): |
Eric Fried | bfaa50f | 2020-01-09 12:04:54 -0600 | [diff] [blame] | 67 | # Setting network=True, subnet=True creates a default network |
| 68 | cls.set_network_resources( |
| 69 | network=cls.create_default_network, |
| 70 | subnet=cls.create_default_network) |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 71 | super(BaseV2ComputeTest, cls).setup_credentials() |
Daryl Walleck | c725196 | 2012-03-12 17:26:54 -0500 | [diff] [blame] | 72 | |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 73 | @classmethod |
| 74 | def setup_clients(cls): |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 75 | super(BaseV2ComputeTest, cls).setup_clients() |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 76 | cls.servers_client = cls.os_primary.servers_client |
| 77 | cls.server_groups_client = cls.os_primary.server_groups_client |
| 78 | cls.flavors_client = cls.os_primary.flavors_client |
| 79 | cls.compute_images_client = cls.os_primary.compute_images_client |
| 80 | cls.extensions_client = cls.os_primary.extensions_client |
| 81 | cls.floating_ip_pools_client = cls.os_primary.floating_ip_pools_client |
| 82 | cls.floating_ips_client = cls.os_primary.compute_floating_ips_client |
| 83 | cls.keypairs_client = cls.os_primary.keypairs_client |
John Warren | 5cdbf42 | 2016-01-05 12:42:43 -0500 | [diff] [blame] | 84 | cls.security_group_rules_client = ( |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 85 | cls.os_primary.compute_security_group_rules_client) |
| 86 | cls.security_groups_client =\ |
| 87 | cls.os_primary.compute_security_groups_client |
| 88 | cls.quotas_client = cls.os_primary.quotas_client |
| 89 | cls.compute_networks_client = cls.os_primary.compute_networks_client |
| 90 | cls.limits_client = cls.os_primary.limits_client |
| 91 | cls.volumes_extensions_client =\ |
| 92 | cls.os_primary.volumes_extensions_client |
| 93 | cls.snapshots_extensions_client =\ |
| 94 | cls.os_primary.snapshots_extensions_client |
| 95 | cls.interfaces_client = cls.os_primary.interfaces_client |
| 96 | cls.fixed_ips_client = cls.os_primary.fixed_ips_client |
| 97 | cls.availability_zone_client = cls.os_primary.availability_zone_client |
| 98 | cls.agents_client = cls.os_primary.agents_client |
| 99 | cls.aggregates_client = cls.os_primary.aggregates_client |
| 100 | cls.services_client = cls.os_primary.services_client |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 101 | cls.instance_usages_audit_log_client = ( |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 102 | cls.os_primary.instance_usages_audit_log_client) |
| 103 | cls.hypervisor_client = cls.os_primary.hypervisor_client |
| 104 | cls.certificates_client = cls.os_primary.certificates_client |
| 105 | cls.migrations_client = cls.os_primary.migrations_client |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 106 | cls.security_group_default_rules_client = ( |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 107 | cls.os_primary.security_group_default_rules_client) |
| 108 | cls.versions_client = cls.os_primary.compute_versions_client |
Andrea Frittoli | a6b3015 | 2017-08-04 10:46:10 +0100 | [diff] [blame] | 109 | if CONF.service_available.cinder: |
| 110 | cls.volumes_client = cls.os_primary.volumes_client_latest |
Lee Yarwood | 4b10852 | 2020-01-15 10:50:24 +0000 | [diff] [blame] | 111 | cls.attachments_client = cls.os_primary.attachments_client_latest |
Matt Riedemann | f110a4b | 2018-01-08 15:03:36 -0500 | [diff] [blame] | 112 | if CONF.service_available.glance: |
| 113 | if CONF.image_feature_enabled.api_v1: |
| 114 | cls.images_client = cls.os_primary.image_client |
| 115 | elif CONF.image_feature_enabled.api_v2: |
| 116 | cls.images_client = cls.os_primary.image_client_v2 |
| 117 | else: |
| 118 | raise lib_exc.InvalidConfiguration( |
| 119 | 'Either api_v1 or api_v2 must be True in ' |
| 120 | '[image-feature-enabled].') |
Matt Riedemann | 14e5e48 | 2018-05-31 15:13:18 -0400 | [diff] [blame] | 121 | cls._check_depends_on_nova_network() |
| 122 | |
| 123 | @classmethod |
| 124 | def _check_depends_on_nova_network(cls): |
| 125 | # Since nova-network APIs were removed from Nova in the Rocky release, |
| 126 | # determine, based on the max version from the version document, if |
| 127 | # the compute API is >Queens and if so, skip tests that rely on |
| 128 | # nova-network. |
| 129 | if not getattr(cls, 'depends_on_nova_network', False): |
| 130 | return |
| 131 | versions = cls.versions_client.list_versions()['versions'] |
| 132 | # Find the v2.1 version which will tell us our max version for the |
| 133 | # compute API we're testing against. |
| 134 | for version in versions: |
| 135 | if version['id'] == 'v2.1': |
| 136 | max_version = api_version_request.APIVersionRequest( |
| 137 | version['version']) |
| 138 | break |
| 139 | else: |
| 140 | LOG.warning( |
| 141 | 'Unable to determine max v2.1 compute API version: %s', |
| 142 | versions) |
| 143 | return |
| 144 | |
| 145 | # The max compute API version in Queens is 2.60 so we cap |
| 146 | # at that version. |
| 147 | queens = api_version_request.APIVersionRequest('2.60') |
| 148 | if max_version > queens: |
| 149 | raise cls.skipException('nova-network is gone') |
Ivan Kolodyazhny | bcfc32e | 2015-08-06 13:31:36 +0300 | [diff] [blame] | 150 | |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 151 | @classmethod |
| 152 | def resource_setup(cls): |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 153 | super(BaseV2ComputeTest, cls).resource_setup() |
Ghanshyam | 05049dd | 2016-02-12 17:44:48 +0900 | [diff] [blame] | 154 | cls.request_microversion = ( |
| 155 | api_version_utils.select_request_microversion( |
| 156 | cls.min_microversion, |
ghanshyam | 2959153 | 2016-03-11 17:12:43 +0900 | [diff] [blame] | 157 | CONF.compute.min_microversion)) |
Lee Yarwood | 4b95d4b | 2020-01-15 10:49:54 +0000 | [diff] [blame] | 158 | cls.volume_request_microversion = ( |
| 159 | api_version_utils.select_request_microversion( |
| 160 | cls.volume_min_microversion, |
| 161 | CONF.volume.min_microversion)) |
| 162 | cls.placement_request_microversion = ( |
| 163 | api_version_utils.select_request_microversion( |
| 164 | cls.placement_min_microversion, |
| 165 | CONF.placement.min_microversion)) |
Matthew Treinish | b0a78fc | 2014-01-29 16:49:12 +0000 | [diff] [blame] | 166 | cls.build_interval = CONF.compute.build_interval |
| 167 | cls.build_timeout = CONF.compute.build_timeout |
Matthew Treinish | b0a78fc | 2014-01-29 16:49:12 +0000 | [diff] [blame] | 168 | cls.image_ref = CONF.compute.image_ref |
| 169 | cls.image_ref_alt = CONF.compute.image_ref_alt |
| 170 | cls.flavor_ref = CONF.compute.flavor_ref |
| 171 | cls.flavor_ref_alt = CONF.compute.flavor_ref_alt |
lanoux | 283273b | 2015-12-04 03:01:54 -0800 | [diff] [blame] | 172 | cls.ssh_user = CONF.validation.image_ssh_user |
| 173 | cls.image_ssh_user = CONF.validation.image_ssh_user |
| 174 | cls.image_ssh_password = CONF.validation.image_ssh_password |
Ken'ichi Ohmichi | 543a5d4 | 2014-05-02 08:44:15 +0900 | [diff] [blame] | 175 | |
Matthew Treinish | f7fca6a | 2013-12-09 16:27:23 +0000 | [diff] [blame] | 176 | @classmethod |
ghanshyam | 66b9aed | 2018-03-30 08:11:10 +0000 | [diff] [blame] | 177 | def is_requested_microversion_compatible(cls, max_version): |
| 178 | """Check the compatibility of selected request microversion |
| 179 | |
| 180 | This method will check if selected request microversion |
| 181 | (cls.request_microversion) for test is compatible with respect |
| 182 | to 'max_version'. Compatible means if selected request microversion |
| 183 | is in the range(<=) of 'max_version'. |
| 184 | |
| 185 | :param max_version: maximum microversion to compare for compatibility. |
| 186 | Example: '2.30' |
| 187 | :returns: True if selected request microversion is compatible with |
| 188 | 'max_version'. False in other case. |
| 189 | """ |
| 190 | try: |
| 191 | req_version_obj = api_version_request.APIVersionRequest( |
| 192 | cls.request_microversion) |
| 193 | # NOTE(gmann): This is case where this method is used before calling |
| 194 | # resource_setup(), where cls.request_microversion is set. There may |
| 195 | # not be any such case but still we can handle this case. |
| 196 | except AttributeError: |
| 197 | request_microversion = ( |
| 198 | api_version_utils.select_request_microversion( |
| 199 | cls.min_microversion, |
| 200 | CONF.compute.min_microversion)) |
| 201 | req_version_obj = api_version_request.APIVersionRequest( |
| 202 | request_microversion) |
| 203 | max_version_obj = api_version_request.APIVersionRequest(max_version) |
| 204 | return req_version_obj <= max_version_obj |
| 205 | |
| 206 | @classmethod |
Attila Fazekas | 305e65b | 2013-10-29 13:23:07 +0100 | [diff] [blame] | 207 | def server_check_teardown(cls): |
| 208 | """Checks is the shared server clean enough for subsequent test. |
Ken'ichi Ohmichi | 88363cb | 2015-11-19 08:00:54 +0000 | [diff] [blame] | 209 | |
Attila Fazekas | 305e65b | 2013-10-29 13:23:07 +0100 | [diff] [blame] | 210 | Method will delete the server when it's dirty. |
| 211 | The setUp method is responsible for creating a new server. |
| 212 | Exceptions raised in tearDown class are fails the test case, |
Marian Horban | 6afb023 | 2015-11-10 22:47:12 -0500 | [diff] [blame] | 213 | This method supposed to use only by tearDown methods, when |
Attila Fazekas | 305e65b | 2013-10-29 13:23:07 +0100 | [diff] [blame] | 214 | the shared server_id is stored in the server_id of the class. |
| 215 | """ |
| 216 | if getattr(cls, 'server_id', None) is not None: |
| 217 | try: |
Ken'ichi Ohmichi | 0eb153c | 2015-07-13 02:18:25 +0000 | [diff] [blame] | 218 | waiters.wait_for_server_status(cls.servers_client, |
| 219 | cls.server_id, 'ACTIVE') |
Attila Fazekas | 305e65b | 2013-10-29 13:23:07 +0100 | [diff] [blame] | 220 | except Exception as exc: |
| 221 | LOG.exception(exc) |
| 222 | cls.servers_client.delete_server(cls.server_id) |
Ken'ichi Ohmichi | e91a0c6 | 2015-08-13 02:09:16 +0000 | [diff] [blame] | 223 | waiters.wait_for_server_termination(cls.servers_client, |
| 224 | cls.server_id) |
Attila Fazekas | 305e65b | 2013-10-29 13:23:07 +0100 | [diff] [blame] | 225 | cls.server_id = None |
| 226 | raise |
| 227 | |
| 228 | @classmethod |
Joe Gordon | 8843f0f | 2015-03-17 15:07:34 -0700 | [diff] [blame] | 229 | def create_test_server(cls, validatable=False, volume_backed=False, |
Artom Lifshitz | 68f1e5e | 2020-04-03 13:12:36 -0400 | [diff] [blame] | 230 | validation_resources=None, clients=None, **kwargs): |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 231 | """Wrapper utility that returns a test server. |
Rohit Karajgi | dc300b2 | 2012-05-04 08:11:00 -0700 | [diff] [blame] | 232 | |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 233 | This wrapper utility calls the common create test server and |
| 234 | returns a test server. The purpose of this wrapper is to minimize |
| 235 | the impact on the code of the tests already using this |
| 236 | function. |
Joe Gordon | 8843f0f | 2015-03-17 15:07:34 -0700 | [diff] [blame] | 237 | |
| 238 | :param validatable: Whether the server will be pingable or sshable. |
| 239 | :param volume_backed: Whether the instance is volume backed or not. |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 240 | :param validation_resources: Dictionary of validation resources as |
| 241 | returned by `get_class_validation_resources`. |
Artom Lifshitz | 68f1e5e | 2020-04-03 13:12:36 -0400 | [diff] [blame] | 242 | :param clients: Client manager, defaults to os_primary. |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 243 | :param kwargs: Extra arguments are passed down to the |
| 244 | `compute.create_test_server` call. |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 245 | """ |
zhufl | 7ae2268 | 2016-09-18 15:22:33 +0800 | [diff] [blame] | 246 | if 'name' not in kwargs: |
| 247 | kwargs['name'] = data_utils.rand_name(cls.__name__ + "-server") |
Sergey Nikitin | 8654e5b | 2017-06-04 22:09:56 +0400 | [diff] [blame] | 248 | |
| 249 | request_version = api_version_request.APIVersionRequest( |
| 250 | cls.request_microversion) |
| 251 | v2_37_version = api_version_request.APIVersionRequest('2.37') |
| 252 | |
zhufl | ff9779c | 2018-01-04 14:41:40 +0800 | [diff] [blame] | 253 | tenant_network = cls.get_tenant_network() |
Sergey Nikitin | 8654e5b | 2017-06-04 22:09:56 +0400 | [diff] [blame] | 254 | # NOTE(snikitin): since microversion v2.37 'networks' field is required |
zhufl | ff9779c | 2018-01-04 14:41:40 +0800 | [diff] [blame] | 255 | if (request_version >= v2_37_version and 'networks' not in kwargs and |
| 256 | not tenant_network): |
Sergey Nikitin | 8654e5b | 2017-06-04 22:09:56 +0400 | [diff] [blame] | 257 | kwargs['networks'] = 'none' |
| 258 | |
Artom Lifshitz | 68f1e5e | 2020-04-03 13:12:36 -0400 | [diff] [blame] | 259 | if clients is None: |
| 260 | clients = cls.os_primary |
| 261 | |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 262 | body, servers = compute.create_test_server( |
Artom Lifshitz | 68f1e5e | 2020-04-03 13:12:36 -0400 | [diff] [blame] | 263 | clients, |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 264 | validatable, |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 265 | validation_resources=validation_resources, |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 266 | tenant_network=tenant_network, |
Joe Gordon | 8843f0f | 2015-03-17 15:07:34 -0700 | [diff] [blame] | 267 | volume_backed=volume_backed, |
Joseph Lanoux | b3e1f87 | 2015-01-30 11:13:07 +0000 | [diff] [blame] | 268 | **kwargs) |
Ken'ichi Ohmichi | 51c8c26 | 2013-12-21 03:30:37 +0900 | [diff] [blame] | 269 | |
Andrea Frittoli | 0d0a3f3 | 2017-08-29 18:21:37 +0100 | [diff] [blame] | 270 | # For each server schedule wait and delete, so we first delete all |
| 271 | # and then wait for all |
| 272 | for server in servers: |
| 273 | cls.addClassResourceCleanup(waiters.wait_for_server_termination, |
Artom Lifshitz | 68f1e5e | 2020-04-03 13:12:36 -0400 | [diff] [blame] | 274 | clients.servers_client, server['id']) |
Andrea Frittoli | 0d0a3f3 | 2017-08-29 18:21:37 +0100 | [diff] [blame] | 275 | for server in servers: |
| 276 | cls.addClassResourceCleanup( |
| 277 | test_utils.call_and_ignore_notfound_exc, |
Artom Lifshitz | 68f1e5e | 2020-04-03 13:12:36 -0400 | [diff] [blame] | 278 | clients.servers_client.delete_server, server['id']) |
Sean Dague | 9b669e3 | 2012-12-13 18:40:08 -0500 | [diff] [blame] | 279 | |
David Kranz | 0fb1429 | 2015-02-11 15:55:20 -0500 | [diff] [blame] | 280 | return body |
Sean Dague | 9b669e3 | 2012-12-13 18:40:08 -0500 | [diff] [blame] | 281 | |
Zhi Kun Liu | 02e7a7b | 2014-01-08 16:08:32 +0800 | [diff] [blame] | 282 | @classmethod |
| 283 | def create_security_group(cls, name=None, description=None): |
| 284 | if name is None: |
| 285 | name = data_utils.rand_name(cls.__name__ + "-securitygroup") |
| 286 | if description is None: |
Ken'ichi Ohmichi | 4937f56 | 2015-03-23 00:15:01 +0000 | [diff] [blame] | 287 | description = data_utils.rand_name('description') |
ghanshyam | b610b77 | 2015-08-24 17:29:38 +0900 | [diff] [blame] | 288 | body = cls.security_groups_client.create_security_group( |
| 289 | name=name, description=description)['security_group'] |
Andrea Frittoli | 238818c | 2017-08-29 18:28:11 +0100 | [diff] [blame] | 290 | cls.addClassResourceCleanup( |
| 291 | test_utils.call_and_ignore_notfound_exc, |
| 292 | cls.security_groups_client.delete_security_group, |
| 293 | body['id']) |
Zhi Kun Liu | 02e7a7b | 2014-01-08 16:08:32 +0800 | [diff] [blame] | 294 | |
David Kranz | 9964b4e | 2015-02-06 15:45:29 -0500 | [diff] [blame] | 295 | return body |
Zhi Kun Liu | 02e7a7b | 2014-01-08 16:08:32 +0800 | [diff] [blame] | 296 | |
Abhijeet.Jain | 87dd445 | 2014-04-23 15:51:23 +0530 | [diff] [blame] | 297 | @classmethod |
Ghanshyam | 2a180b8 | 2014-06-16 13:54:22 +0900 | [diff] [blame] | 298 | def create_test_server_group(cls, name="", policy=None): |
Abhijeet.Jain | 87dd445 | 2014-04-23 15:51:23 +0530 | [diff] [blame] | 299 | if not name: |
| 300 | name = data_utils.rand_name(cls.__name__ + "-Server-Group") |
Ghanshyam | 2a180b8 | 2014-06-16 13:54:22 +0900 | [diff] [blame] | 301 | if policy is None: |
Abhijeet.Jain | 87dd445 | 2014-04-23 15:51:23 +0530 | [diff] [blame] | 302 | policy = ['affinity'] |
Ken'ichi Ohmichi | 1f36daa | 2015-09-30 01:41:34 +0000 | [diff] [blame] | 303 | body = cls.server_groups_client.create_server_group( |
| 304 | name=name, policies=policy)['server_group'] |
Andrea Frittoli | 238818c | 2017-08-29 18:28:11 +0100 | [diff] [blame] | 305 | cls.addClassResourceCleanup( |
| 306 | test_utils.call_and_ignore_notfound_exc, |
| 307 | cls.server_groups_client.delete_server_group, |
| 308 | body['id']) |
David Kranz | ae99b9a | 2015-02-16 13:37:01 -0500 | [diff] [blame] | 309 | return body |
Abhijeet.Jain | 87dd445 | 2014-04-23 15:51:23 +0530 | [diff] [blame] | 310 | |
David Kranz | cf0040c | 2012-06-26 09:46:56 -0400 | [diff] [blame] | 311 | def wait_for(self, condition): |
Sean Dague | f237ccb | 2013-01-04 15:19:14 -0500 | [diff] [blame] | 312 | """Repeatedly calls condition() until a timeout.""" |
David Kranz | cf0040c | 2012-06-26 09:46:56 -0400 | [diff] [blame] | 313 | start_time = int(time.time()) |
| 314 | while True: |
| 315 | try: |
| 316 | condition() |
Matthew Treinish | 05d9fb9 | 2012-12-07 16:14:05 -0500 | [diff] [blame] | 317 | except Exception: |
David Kranz | cf0040c | 2012-06-26 09:46:56 -0400 | [diff] [blame] | 318 | pass |
| 319 | else: |
| 320 | return |
| 321 | if int(time.time()) - start_time >= self.build_timeout: |
| 322 | condition() |
| 323 | return |
| 324 | time.sleep(self.build_interval) |
Jay Pipes | f38eaac | 2012-06-21 13:37:35 -0400 | [diff] [blame] | 325 | |
Attila Fazekas | 423834d | 2014-03-14 17:33:13 +0100 | [diff] [blame] | 326 | @classmethod |
| 327 | def prepare_instance_network(cls): |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 328 | if (CONF.validation.auth_method != 'disabled' and |
| 329 | CONF.validation.connect_method == 'floating'): |
Attila Fazekas | 423834d | 2014-03-14 17:33:13 +0100 | [diff] [blame] | 330 | cls.set_network_resources(network=True, subnet=True, router=True, |
| 331 | dhcp=True) |
| 332 | |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 333 | @classmethod |
| 334 | def create_image_from_server(cls, server_id, **kwargs): |
Matt Riedemann | f110a4b | 2018-01-08 15:03:36 -0500 | [diff] [blame] | 335 | """Wrapper utility that returns an image created from the server. |
| 336 | |
| 337 | If compute microversion >= 2.36, the returned image response will |
| 338 | be from the image service API rather than the compute image proxy API. |
| 339 | """ |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 340 | name = kwargs.pop('name', |
| 341 | data_utils.rand_name(cls.__name__ + "-image")) |
| 342 | wait_until = kwargs.pop('wait_until', None) |
| 343 | wait_for_server = kwargs.pop('wait_for_server', True) |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 344 | |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 345 | image = cls.compute_images_client.create_image(server_id, name=name, |
| 346 | **kwargs) |
Felipe Monteiro | e65ec45 | 2017-09-26 06:47:03 +0100 | [diff] [blame] | 347 | if api_version_utils.compare_version_header_to_response( |
| 348 | "OpenStack-API-Version", "compute 2.45", image.response, "lt"): |
| 349 | image_id = image['image_id'] |
| 350 | else: |
| 351 | image_id = data_utils.parse_image_id(image.response['location']) |
Matt Riedemann | f110a4b | 2018-01-08 15:03:36 -0500 | [diff] [blame] | 352 | |
| 353 | # The compute image proxy APIs were deprecated in 2.35 so |
| 354 | # use the images client directly if the API microversion being |
| 355 | # used is >=2.36. |
zhufl | 66275c2 | 2018-03-28 15:32:14 +0800 | [diff] [blame] | 356 | if not cls.is_requested_microversion_compatible('2.35'): |
Matt Riedemann | f110a4b | 2018-01-08 15:03:36 -0500 | [diff] [blame] | 357 | client = cls.images_client |
| 358 | else: |
| 359 | client = cls.compute_images_client |
Andrea Frittoli | b17f7a3 | 2017-08-29 17:45:58 +0100 | [diff] [blame] | 360 | cls.addClassResourceCleanup(test_utils.call_and_ignore_notfound_exc, |
Matt Riedemann | f110a4b | 2018-01-08 15:03:36 -0500 | [diff] [blame] | 361 | client.delete_image, image_id) |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 362 | |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 363 | if wait_until is not None: |
Matt Riedemann | 1395435 | 2017-02-07 14:03:54 -0500 | [diff] [blame] | 364 | try: |
zhufl | 66275c2 | 2018-03-28 15:32:14 +0800 | [diff] [blame] | 365 | wait_until = wait_until.upper() |
| 366 | if not cls.is_requested_microversion_compatible('2.35'): |
| 367 | wait_until = wait_until.lower() |
Matt Riedemann | f110a4b | 2018-01-08 15:03:36 -0500 | [diff] [blame] | 368 | waiters.wait_for_image_status(client, image_id, wait_until) |
Matt Riedemann | 1395435 | 2017-02-07 14:03:54 -0500 | [diff] [blame] | 369 | except lib_exc.NotFound: |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 370 | if wait_until.upper() == 'ACTIVE': |
Matt Riedemann | 1395435 | 2017-02-07 14:03:54 -0500 | [diff] [blame] | 371 | # If the image is not found after create_image returned |
| 372 | # that means the snapshot failed in nova-compute and nova |
| 373 | # deleted the image. There should be a compute fault |
| 374 | # recorded with the server in that case, so get the server |
| 375 | # and dump some details. |
| 376 | server = ( |
| 377 | cls.servers_client.show_server(server_id)['server']) |
| 378 | if 'fault' in server: |
| 379 | raise exceptions.SnapshotNotFoundException( |
| 380 | server['fault'], image_id=image_id) |
| 381 | else: |
| 382 | raise exceptions.SnapshotNotFoundException( |
| 383 | image_id=image_id) |
| 384 | else: |
| 385 | raise |
Matt Riedemann | f110a4b | 2018-01-08 15:03:36 -0500 | [diff] [blame] | 386 | image = client.show_image(image_id) |
| 387 | # Compute image client returns response wrapped in 'image' element |
| 388 | # which is not the case with Glance image client. |
| 389 | if 'image' in image: |
| 390 | image = image['image'] |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 391 | |
zhufl | 35a694b | 2017-02-14 17:10:53 +0800 | [diff] [blame] | 392 | if wait_until.upper() == 'ACTIVE': |
| 393 | if wait_for_server: |
Bob Ball | 5fe6239 | 2017-02-20 09:51:00 +0000 | [diff] [blame] | 394 | waiters.wait_for_server_status(cls.servers_client, |
| 395 | server_id, 'ACTIVE') |
David Kranz | a5299eb | 2015-01-15 17:24:05 -0500 | [diff] [blame] | 396 | return image |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 397 | |
| 398 | @classmethod |
zhufl | ba0e553 | 2017-09-13 10:51:07 +0800 | [diff] [blame] | 399 | def recreate_server(cls, server_id, validatable=False, **kwargs): |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 400 | """Destroy an existing class level server and creates a new one |
| 401 | |
| 402 | Some test classes use a test server that can be used by multiple |
| 403 | tests. This is done to optimise runtime and test load. |
| 404 | If something goes wrong with the test server, it can be rebuilt |
| 405 | using this helper. |
| 406 | |
| 407 | This helper can also be used for the initial provisioning if no |
| 408 | server_id is specified. |
| 409 | |
| 410 | :param server_id: UUID of the server to be rebuilt. If None is |
| 411 | specified, a new server is provisioned. |
| 412 | :param validatable: whether to the server needs to be |
| 413 | validatable. When True, validation resources are acquired via |
| 414 | the `get_class_validation_resources` helper. |
| 415 | :param kwargs: extra paramaters are passed through to the |
| 416 | `create_test_server` call. |
| 417 | :return: the UUID of the created server. |
| 418 | """ |
Matthew Treinish | 2cd19a4 | 2013-12-02 21:54:42 +0000 | [diff] [blame] | 419 | if server_id: |
zhufl | 9b68290 | 2016-12-15 09:16:34 +0800 | [diff] [blame] | 420 | cls.delete_server(server_id) |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 421 | |
Ghanshyam | 3390d9f | 2015-12-25 12:48:02 +0900 | [diff] [blame] | 422 | cls.password = data_utils.rand_password() |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 423 | server = cls.create_test_server( |
| 424 | validatable, |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 425 | validation_resources=cls.get_class_validation_resources( |
| 426 | cls.os_primary), |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 427 | wait_until='ACTIVE', |
Ghanshyam | 3390d9f | 2015-12-25 12:48:02 +0900 | [diff] [blame] | 428 | adminPass=cls.password, |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 429 | **kwargs) |
Matthew Treinish | 2cd19a4 | 2013-12-02 21:54:42 +0000 | [diff] [blame] | 430 | return server['id'] |
ivan-zhu | 8f992be | 2013-07-31 14:56:58 +0800 | [diff] [blame] | 431 | |
Matt Riedemann | 5dc594c | 2014-01-27 11:40:28 -0800 | [diff] [blame] | 432 | @classmethod |
Jesse Keating | 613b498 | 2015-05-04 15:05:19 -0700 | [diff] [blame] | 433 | def delete_server(cls, server_id): |
| 434 | """Deletes an existing server and waits for it to be gone.""" |
| 435 | try: |
| 436 | cls.servers_client.delete_server(server_id) |
Ken'ichi Ohmichi | e91a0c6 | 2015-08-13 02:09:16 +0000 | [diff] [blame] | 437 | waiters.wait_for_server_termination(cls.servers_client, |
| 438 | server_id) |
Jesse Keating | 613b498 | 2015-05-04 15:05:19 -0700 | [diff] [blame] | 439 | except Exception: |
Jordan Pittier | 525ec71 | 2016-12-07 17:51:26 +0100 | [diff] [blame] | 440 | LOG.exception('Failed to delete server %s', server_id) |
Jesse Keating | 613b498 | 2015-05-04 15:05:19 -0700 | [diff] [blame] | 441 | |
zhufl | bcb7117 | 2018-03-29 13:49:31 +0800 | [diff] [blame] | 442 | def resize_server(self, server_id, new_flavor_id, **kwargs): |
zhufl | 3d018b0 | 2016-11-25 16:43:04 +0800 | [diff] [blame] | 443 | """resize and confirm_resize an server, waits for it to be ACTIVE.""" |
zhufl | bcb7117 | 2018-03-29 13:49:31 +0800 | [diff] [blame] | 444 | self.servers_client.resize_server(server_id, new_flavor_id, **kwargs) |
| 445 | waiters.wait_for_server_status(self.servers_client, server_id, |
zhufl | 3d018b0 | 2016-11-25 16:43:04 +0800 | [diff] [blame] | 446 | 'VERIFY_RESIZE') |
zhufl | bcb7117 | 2018-03-29 13:49:31 +0800 | [diff] [blame] | 447 | self.servers_client.confirm_resize_server(server_id) |
| 448 | waiters.wait_for_server_status( |
| 449 | self.servers_client, server_id, 'ACTIVE') |
| 450 | server = self.servers_client.show_server(server_id)['server'] |
| 451 | self.assert_flavor_equal(new_flavor_id, server['flavor']) |
zhufl | 3d018b0 | 2016-11-25 16:43:04 +0800 | [diff] [blame] | 452 | |
| 453 | @classmethod |
Matt Riedemann | 5dc594c | 2014-01-27 11:40:28 -0800 | [diff] [blame] | 454 | def delete_volume(cls, volume_id): |
| 455 | """Deletes the given volume and waits for it to be gone.""" |
zhufl | decdcf6 | 2017-09-13 10:27:28 +0800 | [diff] [blame] | 456 | try: |
| 457 | cls.volumes_client.delete_volume(volume_id) |
| 458 | # TODO(mriedem): We should move the wait_for_resource_deletion |
| 459 | # into the delete_volume method as a convenience to the caller. |
| 460 | cls.volumes_client.wait_for_resource_deletion(volume_id) |
| 461 | except lib_exc.NotFound: |
| 462 | LOG.warning("Unable to delete volume '%s' since it was not found. " |
| 463 | "Maybe it was already deleted?", volume_id) |
Ken'ichi Ohmichi | 543a5d4 | 2014-05-02 08:44:15 +0900 | [diff] [blame] | 464 | |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 465 | @classmethod |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 466 | def get_server_ip(cls, server, validation_resources=None): |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 467 | """Get the server fixed or floating IP. |
| 468 | |
Sean Dague | 20e9861 | 2016-01-06 14:33:28 -0500 | [diff] [blame] | 469 | Based on the configuration we're in, return a correct ip |
| 470 | address for validating that a guest is up. |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 471 | |
| 472 | :param server: The server dict as returned by the API |
| 473 | :param validation_resources: The dict of validation resources |
| 474 | provisioned for the server. |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 475 | """ |
| 476 | if CONF.validation.connect_method == 'floating': |
Andrea Frittoli | 9f416dd | 2017-08-10 15:38:00 +0100 | [diff] [blame] | 477 | if validation_resources: |
| 478 | return validation_resources['floating_ip']['ip'] |
| 479 | else: |
| 480 | msg = ('When validation.connect_method equals floating, ' |
| 481 | 'validation_resources cannot be None') |
Masayuki Igawa | 2efbea1 | 2019-01-29 18:46:38 +0900 | [diff] [blame] | 482 | raise lib_exc.InvalidParam(invalid_param=msg) |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 483 | elif CONF.validation.connect_method == 'fixed': |
Sean Dague | 20e9861 | 2016-01-06 14:33:28 -0500 | [diff] [blame] | 484 | addresses = server['addresses'][CONF.validation.network_for_ssh] |
| 485 | for address in addresses: |
| 486 | if address['version'] == CONF.validation.ip_version_for_ssh: |
| 487 | return address['addr'] |
zhufl | 955f82b | 2016-07-22 11:14:34 +0800 | [diff] [blame] | 488 | raise exceptions.ServerUnreachable(server_id=server['id']) |
Sean Dague | 20e9861 | 2016-01-06 14:33:28 -0500 | [diff] [blame] | 489 | else: |
guo yunxian | ffc4fc0 | 2016-11-15 09:56:08 +0800 | [diff] [blame] | 490 | raise lib_exc.InvalidConfiguration() |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 491 | |
Ghanshyam | 05049dd | 2016-02-12 17:44:48 +0900 | [diff] [blame] | 492 | def setUp(self): |
| 493 | super(BaseV2ComputeTest, self).setUp() |
| 494 | self.useFixture(api_microversion_fixture.APIMicroversionFixture( |
Lee Yarwood | 4b95d4b | 2020-01-15 10:49:54 +0000 | [diff] [blame] | 495 | compute_microversion=self.request_microversion, |
| 496 | volume_microversion=self.volume_request_microversion, |
| 497 | placement_microversion=self.placement_request_microversion)) |
Ghanshyam | 05049dd | 2016-02-12 17:44:48 +0900 | [diff] [blame] | 498 | |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 499 | @classmethod |
zhufl | 8d23f92 | 2016-12-12 17:29:42 +0800 | [diff] [blame] | 500 | def create_volume(cls, image_ref=None, **kwargs): |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 501 | """Create a volume and wait for it to become 'available'. |
| 502 | |
Artom Lifshitz | fc8f8e6 | 2016-04-13 11:08:32 +0000 | [diff] [blame] | 503 | :param image_ref: Specify an image id to create a bootable volume. |
Sergey Vilgelm | eac094a | 2018-11-21 18:27:51 -0600 | [diff] [blame] | 504 | :param kwargs: other parameters to create volume. |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 505 | :returns: The available volume. |
| 506 | """ |
zhufl | 8d23f92 | 2016-12-12 17:29:42 +0800 | [diff] [blame] | 507 | if 'size' not in kwargs: |
| 508 | kwargs['size'] = CONF.volume.volume_size |
| 509 | if 'display_name' not in kwargs: |
| 510 | vol_name = data_utils.rand_name(cls.__name__ + '-volume') |
| 511 | kwargs['display_name'] = vol_name |
Artom Lifshitz | fc8f8e6 | 2016-04-13 11:08:32 +0000 | [diff] [blame] | 512 | if image_ref is not None: |
zhufl | 8d23f92 | 2016-12-12 17:29:42 +0800 | [diff] [blame] | 513 | kwargs['imageRef'] = image_ref |
Martin Kopec | 00e6d6c | 2019-06-05 14:30:06 +0000 | [diff] [blame] | 514 | if CONF.compute.compute_volume_common_az: |
| 515 | kwargs.setdefault('availability_zone', |
| 516 | CONF.compute.compute_volume_common_az) |
zhufl | 8d23f92 | 2016-12-12 17:29:42 +0800 | [diff] [blame] | 517 | volume = cls.volumes_client.create_volume(**kwargs)['volume'] |
Andrea Frittoli | 1fc499e | 2017-08-29 18:33:03 +0100 | [diff] [blame] | 518 | cls.addClassResourceCleanup( |
| 519 | cls.volumes_client.wait_for_resource_deletion, volume['id']) |
| 520 | cls.addClassResourceCleanup(test_utils.call_and_ignore_notfound_exc, |
| 521 | cls.volumes_client.delete_volume, |
| 522 | volume['id']) |
lkuchlan | 52d7b0d | 2016-11-07 20:53:19 +0200 | [diff] [blame] | 523 | waiters.wait_for_volume_resource_status(cls.volumes_client, |
| 524 | volume['id'], 'available') |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 525 | return volume |
| 526 | |
Matt Riedemann | 0d4551b | 2017-10-10 13:00:48 -0400 | [diff] [blame] | 527 | def _detach_volume(self, server, volume): |
| 528 | """Helper method to detach a volume. |
| 529 | |
| 530 | Ignores 404 responses if the volume or server do not exist, or the |
| 531 | volume is already detached from the server. |
| 532 | """ |
| 533 | try: |
| 534 | volume = self.volumes_client.show_volume(volume['id'])['volume'] |
| 535 | # Check the status. You can only detach an in-use volume, otherwise |
| 536 | # the compute API will return a 400 response. |
| 537 | if volume['status'] == 'in-use': |
| 538 | self.servers_client.detach_volume(server['id'], volume['id']) |
Eric Fried | b3eab67 | 2017-11-21 12:42:18 -0600 | [diff] [blame] | 539 | except lib_exc.NotFound: |
Matt Riedemann | 0d4551b | 2017-10-10 13:00:48 -0400 | [diff] [blame] | 540 | # Ignore 404s on detach in case the server is deleted or the volume |
| 541 | # is already detached. |
| 542 | pass |
| 543 | |
Artom Lifshitz | b6b2bba | 2016-10-31 14:56:40 -0400 | [diff] [blame] | 544 | def attach_volume(self, server, volume, device=None, tag=None): |
Matt Riedemann | cb16a66 | 2016-10-01 18:30:05 -0400 | [diff] [blame] | 545 | """Attaches volume to server and waits for 'in-use' volume status. |
| 546 | |
| 547 | The volume will be detached when the test tears down. |
| 548 | |
| 549 | :param server: The server to which the volume will be attached. |
| 550 | :param volume: The volume to attach. |
| 551 | :param device: Optional mountpoint for the attached volume. Note that |
| 552 | this is not guaranteed for all hypervisors and is not recommended. |
Artom Lifshitz | b6b2bba | 2016-10-31 14:56:40 -0400 | [diff] [blame] | 553 | :param tag: Optional device role tag to apply to the volume. |
Matt Riedemann | cb16a66 | 2016-10-01 18:30:05 -0400 | [diff] [blame] | 554 | """ |
| 555 | attach_kwargs = dict(volumeId=volume['id']) |
| 556 | if device: |
| 557 | attach_kwargs['device'] = device |
Artom Lifshitz | b6b2bba | 2016-10-31 14:56:40 -0400 | [diff] [blame] | 558 | if tag: |
| 559 | attach_kwargs['tag'] = tag |
| 560 | |
zhufl | 36f0a97 | 2017-02-28 15:43:33 +0800 | [diff] [blame] | 561 | attachment = self.servers_client.attach_volume( |
| 562 | server['id'], **attach_kwargs)['volumeAttachment'] |
Lee Yarwood | f644baa | 2020-01-08 19:03:05 +0000 | [diff] [blame] | 563 | # On teardown detach the volume and for multiattach volumes wait for |
| 564 | # the attachment to be removed. For non-multiattach volumes wait for |
| 565 | # the state of the volume to change to available. This is so we don't |
| 566 | # error out when trying to delete the volume during teardown. |
| 567 | if volume['multiattach']: |
| 568 | self.addCleanup(waiters.wait_for_volume_attachment_remove, |
| 569 | self.volumes_client, volume['id'], |
| 570 | attachment['id']) |
| 571 | else: |
| 572 | self.addCleanup(waiters.wait_for_volume_resource_status, |
| 573 | self.volumes_client, volume['id'], 'available') |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 574 | # Ignore 404s on detach in case the server is deleted or the volume |
| 575 | # is already detached. |
Matt Riedemann | 0d4551b | 2017-10-10 13:00:48 -0400 | [diff] [blame] | 576 | self.addCleanup(self._detach_volume, server, volume) |
lkuchlan | 52d7b0d | 2016-11-07 20:53:19 +0200 | [diff] [blame] | 577 | waiters.wait_for_volume_resource_status(self.volumes_client, |
Matt Riedemann | b36186b | 2017-12-04 17:54:08 +0000 | [diff] [blame] | 578 | volume['id'], 'in-use') |
zhufl | 36f0a97 | 2017-02-28 15:43:33 +0800 | [diff] [blame] | 579 | return attachment |
Matt Riedemann | 342b37c | 2016-09-21 15:38:12 -0400 | [diff] [blame] | 580 | |
zhufl | bcb7117 | 2018-03-29 13:49:31 +0800 | [diff] [blame] | 581 | def assert_flavor_equal(self, flavor_id, server_flavor): |
| 582 | """Check whether server_flavor equals to flavor. |
| 583 | |
| 584 | :param flavor_id: flavor id |
| 585 | :param server_flavor: flavor info returned by show_server. |
| 586 | """ |
| 587 | # Nova API > 2.46 no longer includes flavor.id, and schema check |
| 588 | # will cover whether 'id' should be in flavor |
| 589 | if server_flavor.get('id'): |
| 590 | msg = ('server flavor is not same as flavor!') |
| 591 | self.assertEqual(flavor_id, server_flavor['id'], msg) |
| 592 | else: |
| 593 | flavor = self.flavors_client.show_flavor(flavor_id)['flavor'] |
| 594 | self.assertEqual(flavor['name'], server_flavor['original_name'], |
| 595 | "original_name in server flavor is not same as " |
| 596 | "flavor name!") |
| 597 | for key in ['ram', 'vcpus', 'disk']: |
| 598 | msg = ('attribute %s in server flavor is not same as ' |
| 599 | 'flavor!' % key) |
| 600 | self.assertEqual(flavor[key], server_flavor[key], msg) |
| 601 | |
Ken'ichi Ohmichi | 543a5d4 | 2014-05-02 08:44:15 +0900 | [diff] [blame] | 602 | |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 603 | class BaseV2ComputeAdminTest(BaseV2ComputeTest): |
Ken'ichi Ohmichi | bcefa3d | 2014-05-09 08:14:05 +0900 | [diff] [blame] | 604 | """Base test case class for Compute Admin API tests.""" |
ivan-zhu | f2b0050 | 2013-10-18 10:06:52 +0800 | [diff] [blame] | 605 | |
Andrea Frittoli | b21de6c | 2015-02-06 20:12:38 +0000 | [diff] [blame] | 606 | credentials = ['primary', 'admin'] |
Emily Hugenbruch | e7991d9 | 2014-12-12 16:53:36 +0000 | [diff] [blame] | 607 | |
| 608 | @classmethod |
| 609 | def setup_clients(cls): |
Ken'ichi Ohmichi | 02a8ccd | 2015-11-05 06:05:29 +0000 | [diff] [blame] | 610 | super(BaseV2ComputeAdminTest, cls).setup_clients() |
Ken'ichi Ohmichi | 9f5adf8 | 2014-12-12 04:01:32 +0000 | [diff] [blame] | 611 | cls.availability_zone_admin_client = ( |
Jordan Pittier | 8160d31 | 2017-04-18 11:52:23 +0200 | [diff] [blame] | 612 | cls.os_admin.availability_zone_client) |
| 613 | cls.admin_flavors_client = cls.os_admin.flavors_client |
| 614 | cls.admin_servers_client = cls.os_admin.servers_client |
zhufl | 36eeab0 | 2017-01-18 11:49:04 +0800 | [diff] [blame] | 615 | |
| 616 | def create_flavor(self, ram, vcpus, disk, name=None, |
| 617 | is_public='True', **kwargs): |
| 618 | if name is None: |
| 619 | name = data_utils.rand_name(self.__class__.__name__ + "-flavor") |
| 620 | id = kwargs.pop('id', data_utils.rand_int_id(start=1000)) |
| 621 | client = self.admin_flavors_client |
| 622 | flavor = client.create_flavor( |
| 623 | ram=ram, vcpus=vcpus, disk=disk, name=name, |
| 624 | id=id, is_public=is_public, **kwargs)['flavor'] |
| 625 | self.addCleanup(client.wait_for_resource_deletion, flavor['id']) |
| 626 | self.addCleanup(client.delete_flavor, flavor['id']) |
| 627 | return flavor |
Duc Truong | 0994120 | 2017-06-07 10:15:20 -0700 | [diff] [blame] | 628 | |
zhufl | 7bc916d | 2018-08-22 14:47:39 +0800 | [diff] [blame] | 629 | @classmethod |
| 630 | def get_host_for_server(cls, server_id): |
| 631 | server_details = cls.admin_servers_client.show_server(server_id) |
Duc Truong | 0994120 | 2017-06-07 10:15:20 -0700 | [diff] [blame] | 632 | return server_details['server']['OS-EXT-SRV-ATTR:host'] |
| 633 | |
| 634 | def get_host_other_than(self, server_id): |
| 635 | source_host = self.get_host_for_server(server_id) |
| 636 | |
Radoslav Gerganov | 50325e2 | 2018-03-29 12:02:04 +0300 | [diff] [blame] | 637 | svcs = self.os_admin.services_client.list_services( |
| 638 | binary='nova-compute')['services'] |
Marian Krcmarik | 6e3f99e | 2020-02-26 23:27:51 +0100 | [diff] [blame] | 639 | hosts = [] |
| 640 | for svc in svcs: |
| 641 | if svc['state'] == 'up' and svc['status'] == 'enabled': |
| 642 | if CONF.compute.compute_volume_common_az: |
| 643 | if svc['zone'] == CONF.compute.compute_volume_common_az: |
| 644 | hosts.append(svc['host']) |
| 645 | else: |
| 646 | hosts.append(svc['host']) |
Duc Truong | 0994120 | 2017-06-07 10:15:20 -0700 | [diff] [blame] | 647 | |
| 648 | for target_host in hosts: |
| 649 | if source_host != target_host: |
| 650 | return target_host |