Matt Riedemann | bc8dbd3 | 2013-08-02 14:02:12 -0700 | [diff] [blame] | 1 | # Copyright 2013 IBM Corp. |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [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 | |
Anusha Ramineni | f3eb947 | 2017-01-13 08:54:01 +0530 | [diff] [blame] | 16 | from oslo_log import log as logging |
ivan-zhu | 1feeb38 | 2013-01-24 10:14:39 +0800 | [diff] [blame] | 17 | import testtools |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 18 | |
Sean Dague | 1937d09 | 2013-05-17 16:36:38 -0400 | [diff] [blame] | 19 | from tempest.api.compute import base |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 20 | from tempest.common import compute |
Masayuki Igawa | 209fd50 | 2014-02-17 14:46:43 +0900 | [diff] [blame] | 21 | from tempest.common.utils.linux import remote_client |
Ken'ichi Ohmichi | 0eb153c | 2015-07-13 02:18:25 +0000 | [diff] [blame] | 22 | from tempest.common import waiters |
Sean Dague | 86bd842 | 2013-12-20 09:56:44 -0500 | [diff] [blame] | 23 | from tempest import config |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 24 | from tempest.lib import exceptions as lib_exc |
Masayuki Igawa | 209fd50 | 2014-02-17 14:46:43 +0900 | [diff] [blame] | 25 | from tempest import test |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 26 | |
Sean Dague | 86bd842 | 2013-12-20 09:56:44 -0500 | [diff] [blame] | 27 | CONF = config.CONF |
| 28 | |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 29 | LOG = logging.getLogger(__name__) |
| 30 | |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 31 | |
ivan-zhu | f2b0050 | 2013-10-18 10:06:52 +0800 | [diff] [blame] | 32 | class AttachVolumeTestJSON(base.BaseV2ComputeTest): |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 33 | max_microversion = '2.19' |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 34 | |
Attila Fazekas | 19044d5 | 2013-02-16 07:35:06 +0100 | [diff] [blame] | 35 | @classmethod |
Emily Hugenbruch | 8284a34 | 2014-12-11 22:04:55 +0000 | [diff] [blame] | 36 | def skip_checks(cls): |
| 37 | super(AttachVolumeTestJSON, cls).skip_checks() |
Matthew Treinish | b0a78fc | 2014-01-29 16:49:12 +0000 | [diff] [blame] | 38 | if not CONF.service_available.cinder: |
Matthew Treinish | 4c41292 | 2013-07-16 15:27:42 -0400 | [diff] [blame] | 39 | skip_msg = ("%s skipped as Cinder is not available" % cls.__name__) |
| 40 | raise cls.skipException(skip_msg) |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 41 | |
Emily Hugenbruch | 8284a34 | 2014-12-11 22:04:55 +0000 | [diff] [blame] | 42 | @classmethod |
| 43 | def setup_credentials(cls): |
| 44 | cls.prepare_instance_network() |
| 45 | super(AttachVolumeTestJSON, cls).setup_credentials() |
| 46 | |
| 47 | @classmethod |
| 48 | def resource_setup(cls): |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 49 | cls.set_validation_resources() |
Emily Hugenbruch | 8284a34 | 2014-12-11 22:04:55 +0000 | [diff] [blame] | 50 | super(AttachVolumeTestJSON, cls).resource_setup() |
| 51 | cls.device = CONF.compute.volume_device_name |
| 52 | |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 53 | def _create_server(self): |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 54 | # Start a server and wait for it to become ready |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 55 | server = self.create_test_server( |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 56 | validatable=True, |
| 57 | wait_until='ACTIVE', |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 58 | adminPass=self.image_ssh_password) |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 59 | # Record addresses so that we can ssh later |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 60 | server['addresses'] = self.servers_client.list_addresses( |
| 61 | server['id'])['addresses'] |
| 62 | return server |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 63 | |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 64 | def _detach_volume(self, server_id, volume_id): |
| 65 | try: |
| 66 | self.servers_client.detach_volume(server_id, volume_id) |
| 67 | waiters.wait_for_volume_status(self.volumes_client, |
| 68 | volume_id, 'available') |
| 69 | except lib_exc.NotFound: |
| 70 | LOG.warning("Unable to detach volume %s from server %s " |
Jordan Pittier | 525ec71 | 2016-12-07 17:51:26 +0100 | [diff] [blame] | 71 | "possibly it was already detached", volume_id, |
| 72 | server_id) |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 73 | |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 74 | def _attach_volume(self, server_id, volume_id, device=None): |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 75 | # Attach the volume to the server |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 76 | kwargs = {'volumeId': volume_id} |
Ken'ichi Ohmichi | 2b74f10 | 2016-11-22 17:18:36 -0800 | [diff] [blame] | 77 | if device: |
| 78 | kwargs.update({'device': '/dev/%s' % device}) |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 79 | attachment = self.servers_client.attach_volume( |
| 80 | server_id, **kwargs)['volumeAttachment'] |
Yaroslav Lobankov | ed3a35b | 2016-03-24 22:41:30 -0500 | [diff] [blame] | 81 | waiters.wait_for_volume_status(self.volumes_client, |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 82 | volume_id, 'in-use') |
| 83 | self.addCleanup(self._detach_volume, server_id, |
| 84 | volume_id) |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 85 | |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 86 | return attachment |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 87 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 88 | @test.idempotent_id('52e9045a-e90d-4c0d-9087-79d657faffff') |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 89 | def test_attach_detach_volume(self): |
Sean Dague | 4dd2c0b | 2013-01-03 17:50:28 -0500 | [diff] [blame] | 90 | # Stop and Start a server with an attached volume, ensuring that |
| 91 | # the volume remains attached. |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 92 | server = self._create_server() |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 93 | volume = self.create_volume() |
| 94 | attachment = self._attach_volume(server['id'], volume['id'], |
| 95 | device=self.device) |
ivan-zhu | 2f54b28 | 2013-03-11 16:39:25 +0800 | [diff] [blame] | 96 | |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 97 | self.servers_client.stop_server(server['id']) |
| 98 | waiters.wait_for_server_status(self.servers_client, server['id'], |
Ken'ichi Ohmichi | 0eb153c | 2015-07-13 02:18:25 +0000 | [diff] [blame] | 99 | 'SHUTOFF') |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 100 | |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 101 | self.servers_client.start_server(server['id']) |
| 102 | waiters.wait_for_server_status(self.servers_client, server['id'], |
Ken'ichi Ohmichi | 0eb153c | 2015-07-13 02:18:25 +0000 | [diff] [blame] | 103 | 'ACTIVE') |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 104 | |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 105 | if CONF.validation.run_validation: |
| 106 | linux_client = remote_client.RemoteClient( |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 107 | self.get_server_ip(server), |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 108 | self.image_ssh_user, |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 109 | self.image_ssh_password, |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 110 | self.validation_resources['keypair']['private_key'], |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 111 | server=server, |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 112 | servers_client=self.servers_client) |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 113 | |
Evgeny Antyshev | 4894a91 | 2016-11-21 12:17:18 +0000 | [diff] [blame] | 114 | disks = linux_client.get_disks() |
| 115 | device_name_to_match = '\n' + self.device + ' ' |
| 116 | self.assertIn(device_name_to_match, disks) |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 117 | |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 118 | self._detach_volume(server['id'], attachment['volumeId']) |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 119 | self.servers_client.stop_server(server['id']) |
| 120 | waiters.wait_for_server_status(self.servers_client, server['id'], |
Ken'ichi Ohmichi | 0eb153c | 2015-07-13 02:18:25 +0000 | [diff] [blame] | 121 | 'SHUTOFF') |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 122 | |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 123 | self.servers_client.start_server(server['id']) |
| 124 | waiters.wait_for_server_status(self.servers_client, server['id'], |
Ken'ichi Ohmichi | 0eb153c | 2015-07-13 02:18:25 +0000 | [diff] [blame] | 125 | 'ACTIVE') |
Dan Smith | c18d8c6 | 2012-07-02 08:09:26 -0700 | [diff] [blame] | 126 | |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 127 | if CONF.validation.run_validation: |
| 128 | linux_client = remote_client.RemoteClient( |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 129 | self.get_server_ip(server), |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 130 | self.image_ssh_user, |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 131 | self.image_ssh_password, |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 132 | self.validation_resources['keypair']['private_key'], |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 133 | server=server, |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 134 | servers_client=self.servers_client) |
Joseph Lanoux | ffe09dd | 2015-03-18 16:45:33 +0000 | [diff] [blame] | 135 | |
Evgeny Antyshev | 4894a91 | 2016-11-21 12:17:18 +0000 | [diff] [blame] | 136 | disks = linux_client.get_disks() |
| 137 | self.assertNotIn(device_name_to_match, disks) |
Dan Smith | 1ced842 | 2012-08-16 10:35:19 -0700 | [diff] [blame] | 138 | |
Chris Hoge | 7579c1a | 2015-02-26 14:12:15 -0800 | [diff] [blame] | 139 | @test.idempotent_id('7fa563fe-f0f7-43eb-9e22-a1ece036b513') |
Ghanshyam | 5c2a558 | 2014-04-14 17:16:57 +0900 | [diff] [blame] | 140 | def test_list_get_volume_attachments(self): |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 141 | # List volume attachment of the server |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 142 | server = self._create_server() |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 143 | volume = self.create_volume() |
| 144 | attachment = self._attach_volume(server['id'], volume['id'], |
| 145 | device=self.device) |
David Kranz | 3ebc721 | 2015-02-10 12:19:19 -0500 | [diff] [blame] | 146 | body = self.servers_client.list_volume_attachments( |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 147 | server['id'])['volumeAttachments'] |
Ghanshyam | 5c2a558 | 2014-04-14 17:16:57 +0900 | [diff] [blame] | 148 | self.assertEqual(1, len(body)) |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 149 | self.assertIn(attachment, body) |
Ghanshyam | 5c2a558 | 2014-04-14 17:16:57 +0900 | [diff] [blame] | 150 | |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 151 | # Get volume attachment of the server |
Ken'ichi Ohmichi | 277d188 | 2015-11-20 00:44:06 +0000 | [diff] [blame] | 152 | body = self.servers_client.show_volume_attachment( |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 153 | server['id'], |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 154 | attachment['id'])['volumeAttachment'] |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 155 | self.assertEqual(server['id'], body['serverId']) |
Anna Babich | 506efe2 | 2016-09-08 18:14:22 +0300 | [diff] [blame] | 156 | self.assertEqual(volume['id'], body['volumeId']) |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 157 | self.assertEqual(attachment['id'], body['id']) |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 158 | |
Ken'ichi Ohmichi | 0913928 | 2016-12-01 14:36:22 -0800 | [diff] [blame] | 159 | @test.idempotent_id('757d488b-a951-4bc7-b3cd-f417028da08a') |
| 160 | def test_list_get_two_volume_attachments(self): |
| 161 | # NOTE: This test is using the volume device auto-assignment |
| 162 | # without specifying the device ("/dev/sdb", etc). The feature |
| 163 | # is supported since Nova Liberty release or later. So this should |
| 164 | # be skipped on older clouds. |
| 165 | server = self._create_server() |
| 166 | volume_1st = self.create_volume() |
| 167 | volume_2nd = self.create_volume() |
| 168 | attachment_1st = self._attach_volume(server['id'], volume_1st['id']) |
| 169 | attachment_2nd = self._attach_volume(server['id'], volume_2nd['id']) |
| 170 | |
| 171 | body = self.servers_client.list_volume_attachments( |
| 172 | server['id'])['volumeAttachments'] |
| 173 | self.assertEqual(2, len(body)) |
| 174 | |
| 175 | body = self.servers_client.show_volume_attachment( |
| 176 | server['id'], |
| 177 | attachment_1st['id'])['volumeAttachment'] |
| 178 | self.assertEqual(server['id'], body['serverId']) |
| 179 | self.assertEqual(attachment_1st['volumeId'], body['volumeId']) |
| 180 | self.assertEqual(attachment_1st['id'], body['id']) |
| 181 | |
| 182 | body = self.servers_client.show_volume_attachment( |
| 183 | server['id'], |
| 184 | attachment_2nd['id'])['volumeAttachment'] |
| 185 | self.assertEqual(server['id'], body['serverId']) |
| 186 | self.assertEqual(attachment_2nd['volumeId'], body['volumeId']) |
| 187 | self.assertEqual(attachment_2nd['id'], body['id']) |
| 188 | |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 189 | |
| 190 | class AttachVolumeShelveTestJSON(AttachVolumeTestJSON): |
| 191 | """Testing volume with shelved instance. |
| 192 | |
| 193 | This test checks the attaching and detaching volumes from |
Tianbiao Qi | 0d1d24e | 2016-09-28 14:17:12 +0800 | [diff] [blame] | 194 | a shelved or shelved offload instance. |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 195 | """ |
| 196 | |
| 197 | min_microversion = '2.20' |
| 198 | max_microversion = 'latest' |
| 199 | |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 200 | def _count_volumes(self, server): |
| 201 | # Count number of volumes on an instance |
| 202 | volumes = 0 |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 203 | if CONF.validation.run_validation: |
| 204 | linux_client = remote_client.RemoteClient( |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 205 | self.get_server_ip(server), |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 206 | self.image_ssh_user, |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 207 | self.image_ssh_password, |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 208 | self.validation_resources['keypair']['private_key'], |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 209 | server=server, |
Andrea Frittoli (andreaf) | 3f5aa98 | 2016-07-08 12:10:36 +0100 | [diff] [blame] | 210 | servers_client=self.servers_client) |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 211 | |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 212 | command = 'grep -c -E [vs]d.$ /proc/partitions' |
| 213 | volumes = int(linux_client.exec_command(command).strip()) |
| 214 | return volumes |
| 215 | |
| 216 | def _shelve_server(self, server): |
| 217 | # NOTE(andreaf) If we are going to shelve a server, we should |
| 218 | # check first whether the server is ssh-able. Otherwise we |
| 219 | # won't be able to distinguish failures introduced by shelve |
| 220 | # from pre-existing ones. Also it's good to wait for cloud-init |
| 221 | # to be done and sshd server to be running before shelving to |
| 222 | # avoid breaking the VM |
| 223 | if CONF.validation.run_validation: |
| 224 | linux_client = remote_client.RemoteClient( |
| 225 | self.get_server_ip(server), |
| 226 | self.image_ssh_user, |
| 227 | self.image_ssh_password, |
| 228 | self.validation_resources['keypair']['private_key'], |
| 229 | server=server, |
| 230 | servers_client=self.servers_client) |
| 231 | linux_client.validate_authentication() |
| 232 | |
| 233 | # If validation went ok, or it was skipped, shelve the server |
| 234 | compute.shelve_server(self.servers_client, server['id']) |
| 235 | |
| 236 | def _unshelve_server_and_check_volumes(self, server, number_of_volumes): |
| 237 | # Unshelve the instance and check that there are expected volumes |
| 238 | self.servers_client.unshelve_server(server['id']) |
| 239 | waiters.wait_for_server_status(self.servers_client, |
| 240 | server['id'], |
| 241 | 'ACTIVE') |
| 242 | if CONF.validation.run_validation: |
| 243 | counted_volumes = self._count_volumes(server) |
| 244 | self.assertEqual(number_of_volumes, counted_volumes) |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 245 | |
| 246 | @test.idempotent_id('13a940b6-3474-4c3c-b03f-29b89112bfee') |
| 247 | @testtools.skipUnless(CONF.compute_feature_enabled.shelve, |
| 248 | 'Shelve is not available.') |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 249 | def test_attach_volume_shelved_or_offload_server(self): |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 250 | # Create server, count number of volumes on it, shelve |
| 251 | # server and attach pre-created volume to shelved server |
| 252 | server = self._create_server() |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 253 | volume = self.create_volume() |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 254 | num_vol = self._count_volumes(server) |
| 255 | self._shelve_server(server) |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 256 | attachment = self._attach_volume(server['id'], volume['id'], |
| 257 | device=self.device) |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 258 | |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 259 | # Unshelve the instance and check that attached volume exists |
| 260 | self._unshelve_server_and_check_volumes(server, num_vol + 1) |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 261 | |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 262 | # Get volume attachment of the server |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 263 | volume_attachment = self.servers_client.show_volume_attachment( |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 264 | server['id'], |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 265 | attachment['id'])['volumeAttachment'] |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 266 | self.assertEqual(server['id'], volume_attachment['serverId']) |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 267 | self.assertEqual(attachment['id'], volume_attachment['id']) |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 268 | # Check the mountpoint is not None after unshelve server even in |
| 269 | # case of shelved_offloaded. |
| 270 | self.assertIsNotNone(volume_attachment['device']) |
| 271 | |
| 272 | @test.idempotent_id('b54e86dd-a070-49c4-9c07-59ae6dae15aa') |
| 273 | @testtools.skipUnless(CONF.compute_feature_enabled.shelve, |
| 274 | 'Shelve is not available.') |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 275 | def test_detach_volume_shelved_or_offload_server(self): |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 276 | # Count number of volumes on instance, shelve |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 277 | # server and attach pre-created volume to shelved server |
| 278 | server = self._create_server() |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 279 | volume = self.create_volume() |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 280 | num_vol = self._count_volumes(server) |
| 281 | self._shelve_server(server) |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 282 | self._attach_volume(server['id'], volume['id'], device=self.device) |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 283 | # Detach the volume |
Benny Kopilov | 5c5f7d8 | 2016-09-13 14:19:53 +0300 | [diff] [blame] | 284 | self._detach_volume(server['id'], volume['id']) |
lanoux | 2746ba0 | 2016-03-16 17:41:01 +0900 | [diff] [blame] | 285 | |
Fabian Zimmermann | bbef276 | 2016-06-23 14:05:33 +0200 | [diff] [blame] | 286 | # Unshelve the instance and check that we have the expected number of |
| 287 | # volume(s) |
| 288 | self._unshelve_server_and_check_volumes(server, num_vol) |