blob: b5fc39c4aa83a3da76a3b39b10f74f0806877700 [file] [log] [blame]
ZhiQiang Fan39f97222013-09-20 04:49:44 +08001# Copyright 2012 OpenStack Foundation
Jay Pipes13b479b2012-06-11 14:52:27 -04002# 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
Cady_Chen72014bc2016-11-16 09:43:49 +080016from oslo_log import log as logging
Matthew Treinishf077dd22015-04-23 09:37:41 -040017from six.moves.urllib import parse as urlparse
ivan-zhu1feeb382013-01-24 10:14:39 +080018import testtools
Jay Pipes13b479b2012-06-11 14:52:27 -040019
Sean Dague1937d092013-05-17 16:36:38 -040020from tempest.api.compute import base
ghanshyam017b5fe2016-04-15 18:49:26 +090021from tempest.common import compute
Andrea Frittolicd368412017-08-14 21:37:56 +010022from tempest.common import utils
Masayuki Igawa209fd502014-02-17 14:46:43 +090023from tempest.common.utils.linux import remote_client
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +000024from tempest.common import waiters
Sean Dague86bd8422013-12-20 09:56:44 -050025from tempest import config
Felipe Monteiro9ff5c282017-06-21 21:05:07 +010026from tempest.lib.common import api_version_utils
Ken'ichi Ohmichi757833a2017-03-10 10:30:30 -080027from tempest.lib.common.utils import data_utils
Andrea Frittoli (andreaf)db9672e2016-02-23 14:07:24 -050028from tempest.lib import decorators
29from tempest.lib import exceptions as lib_exc
Daryl Wallecked8bef32011-12-05 23:02:08 -060030
Sean Dague86bd8422013-12-20 09:56:44 -050031CONF = config.CONF
32
Ryan Hsu38231092014-06-08 23:12:10 -070033LOG = logging.getLogger(__name__)
34
Daryl Walleckc7251962012-03-12 17:26:54 -050035
ivan-zhuf2b00502013-10-18 10:06:52 +080036class ServerActionsTestJSON(base.BaseV2ComputeTest):
Attila Fazekas19044d52013-02-16 07:35:06 +010037 def setUp(self):
Matt Riedemann2da58592014-06-08 12:46:02 +000038 # NOTE(afazekas): Normally we use the same server with all test cases,
39 # but if it has an issue, we build a new one
Attila Fazekas19044d52013-02-16 07:35:06 +010040 super(ServerActionsTestJSON, self).setUp()
Matt Riedemann2da58592014-06-08 12:46:02 +000041 # Check if the server is in a clean state after test
42 try:
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +000043 waiters.wait_for_server_status(self.client,
44 self.server_id, 'ACTIVE')
Viktor Tikkanen1b0ce232015-05-28 10:21:27 +030045 except lib_exc.NotFound:
46 # The server was deleted by previous test, create a new one
Joseph Lanouxffe09dd2015-03-18 16:45:33 +000047 server = self.create_test_server(
48 validatable=True,
49 wait_until='ACTIVE')
Viktor Tikkanen1b0ce232015-05-28 10:21:27 +030050 self.__class__.server_id = server['id']
Matt Riedemann2da58592014-06-08 12:46:02 +000051 except Exception:
52 # Rebuild server if something happened to it during a test
ghanshyam0f825252015-08-25 16:02:50 +090053 self.__class__.server_id = self.rebuild_server(
ghanshyam52d22352015-11-10 13:35:27 +090054 self.server_id, validatable=True)
Matt Riedemann2da58592014-06-08 12:46:02 +000055
56 def tearDown(self):
Matt Riedemann2da58592014-06-08 12:46:02 +000057 self.server_check_teardown()
58 super(ServerActionsTestJSON, self).tearDown()
Attila Fazekas305e65b2013-10-29 13:23:07 +010059
Attila Fazekas19044d52013-02-16 07:35:06 +010060 @classmethod
Joseph Lanouxffe09dd2015-03-18 16:45:33 +000061 def setup_credentials(cls):
62 cls.prepare_instance_network()
63 super(ServerActionsTestJSON, cls).setup_credentials()
64
65 @classmethod
Rohan Kanade60b73092015-02-04 17:58:19 +053066 def setup_clients(cls):
67 super(ServerActionsTestJSON, cls).setup_clients()
68 cls.client = cls.servers_client
69
70 @classmethod
Andrea Frittoli50bb80d2014-09-15 12:34:27 +010071 def resource_setup(cls):
Joseph Lanouxffe09dd2015-03-18 16:45:33 +000072 cls.set_validation_resources()
73
Andrea Frittoli50bb80d2014-09-15 12:34:27 +010074 super(ServerActionsTestJSON, cls).resource_setup()
Joseph Lanouxffe09dd2015-03-18 16:45:33 +000075 cls.server_id = cls.rebuild_server(None, validatable=True)
Attila Fazekas19044d52013-02-16 07:35:06 +010076
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -080077 @decorators.idempotent_id('6158df09-4b82-4ab3-af6d-29cf36af858d')
Sean Dague86bd8422013-12-20 09:56:44 -050078 @testtools.skipUnless(CONF.compute_feature_enabled.change_password,
ivan-zhu1feeb382013-01-24 10:14:39 +080079 'Change password not available.')
Daryl Wallecked8bef32011-12-05 23:02:08 -060080 def test_change_server_password(self):
Emily Hugenbruch1315e182016-10-10 17:34:06 -040081 # Since this test messes with the password and makes the
82 # server unreachable, it should create its own server
83 newserver = self.create_test_server(
84 validatable=True,
85 wait_until='ACTIVE')
Sean Dague4dd2c0b2013-01-03 17:50:28 -050086 # The server's password should be set to the provided password
Daryl Walleck98e66dd2012-06-21 04:58:39 -050087 new_password = 'Newpass1234'
Emily Hugenbruch1315e182016-10-10 17:34:06 -040088 self.client.change_password(newserver['id'], adminPass=new_password)
89 waiters.wait_for_server_status(self.client, newserver['id'], 'ACTIVE')
Daryl Wallecked8bef32011-12-05 23:02:08 -060090
Joseph Lanouxffe09dd2015-03-18 16:45:33 +000091 if CONF.validation.run_validation:
Daryl Walleck98e66dd2012-06-21 04:58:39 -050092 # Verify that the user can authenticate with the new password
Emily Hugenbruch1315e182016-10-10 17:34:06 -040093 server = self.client.show_server(newserver['id'])['server']
Joseph Lanouxffe09dd2015-03-18 16:45:33 +000094 linux_client = remote_client.RemoteClient(
95 self.get_server_ip(server),
96 self.ssh_user,
Andrea Frittoli (andreaf)2a70a612016-04-29 16:09:13 -050097 new_password,
98 server=server,
99 servers_client=self.client)
Attila Fazekasad7ef7d2013-11-20 10:12:53 +0100100 linux_client.validate_authentication()
Daryl Walleck98e66dd2012-06-21 04:58:39 -0500101
Ken'ichi Ohmichi84d7fbc2014-09-30 01:42:33 +0000102 def _test_reboot_server(self, reboot_type):
Joseph Lanouxffe09dd2015-03-18 16:45:33 +0000103 if CONF.validation.run_validation:
Daryl Walleck98e66dd2012-06-21 04:58:39 -0500104 # Get the time the server was last rebooted,
ghanshyam0f825252015-08-25 16:02:50 +0900105 server = self.client.show_server(self.server_id)['server']
Joseph Lanouxffe09dd2015-03-18 16:45:33 +0000106 linux_client = remote_client.RemoteClient(
107 self.get_server_ip(server),
108 self.ssh_user,
109 self.password,
Andrea Frittoli (andreaf)2a70a612016-04-29 16:09:13 -0500110 self.validation_resources['keypair']['private_key'],
111 server=server,
112 servers_client=self.client)
Daryl Walleck98e66dd2012-06-21 04:58:39 -0500113 boot_time = linux_client.get_boot_time()
Daryl Walleck98e66dd2012-06-21 04:58:39 -0500114
Ken'ichi Ohmichi05ab8562016-07-12 00:06:18 -0700115 # NOTE: This sync is for avoiding the loss of pub key data
116 # in a server
117 linux_client.exec_command("sync")
118
Ken'ichi Ohmichi36b714c2015-12-09 08:12:47 +0000119 self.client.reboot_server(self.server_id, type=reboot_type)
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000120 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
Daryl Wallecked8bef32011-12-05 23:02:08 -0600121
Joseph Lanouxffe09dd2015-03-18 16:45:33 +0000122 if CONF.validation.run_validation:
Daryl Walleck98e66dd2012-06-21 04:58:39 -0500123 # Log in and verify the boot time has changed
Joseph Lanouxffe09dd2015-03-18 16:45:33 +0000124 linux_client = remote_client.RemoteClient(
125 self.get_server_ip(server),
126 self.ssh_user,
127 self.password,
Andrea Frittoli (andreaf)2a70a612016-04-29 16:09:13 -0500128 self.validation_resources['keypair']['private_key'],
129 server=server,
130 servers_client=self.client)
Daryl Walleck98e66dd2012-06-21 04:58:39 -0500131 new_boot_time = linux_client.get_boot_time()
Béla Vancsics64862f72016-11-08 09:12:31 +0100132 self.assertGreater(new_boot_time, boot_time,
133 '%s > %s' % (new_boot_time, boot_time))
Daryl Walleck98e66dd2012-06-21 04:58:39 -0500134
Jordan Pittier3b46d272017-04-12 16:17:28 +0200135 @decorators.attr(type='smoke')
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800136 @decorators.idempotent_id('2cb1baf6-ac8d-4429-bf0d-ba8a0ba53e32')
Ken'ichi Ohmichi84d7fbc2014-09-30 01:42:33 +0000137 def test_reboot_server_hard(self):
138 # The server should be power cycled
139 self._test_reboot_server('HARD')
140
Matthew Treinishc49fcbe2015-02-05 23:37:34 -0500141 @decorators.skip_because(bug="1014647")
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800142 @decorators.idempotent_id('4640e3ef-a5df-482e-95a1-ceeeb0faa84d')
Daryl Wallecked8bef32011-12-05 23:02:08 -0600143 def test_reboot_server_soft(self):
Sean Dague4dd2c0b2013-01-03 17:50:28 -0500144 # The server should be signaled to reboot gracefully
Ken'ichi Ohmichi84d7fbc2014-09-30 01:42:33 +0000145 self._test_reboot_server('SOFT')
Daryl Walleck98e66dd2012-06-21 04:58:39 -0500146
lianghaofd8d3152017-04-01 15:34:26 +0800147 @decorators.idempotent_id('1d1c9104-1b0a-11e7-a3d4-fa163e65f5ce')
148 def test_remove_server_all_security_groups(self):
149 server = self.create_test_server(wait_until='ACTIVE')
150
151 # Remove all Security group
152 self.client.remove_security_group(
153 server['id'], name=server['security_groups'][0]['name'])
154
155 # Verify all Security group
156 server = self.client.show_server(server['id'])['server']
157 self.assertNotIn('security_groups', server)
158
Yaroslav Lobankovfb1c3642015-05-13 18:38:47 +0300159 def _rebuild_server_and_check(self, image_ref):
ghanshyam0f825252015-08-25 16:02:50 +0900160 rebuilt_server = (self.client.rebuild_server(self.server_id, image_ref)
161 ['server'])
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000162 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
Yaroslav Lobankovfb1c3642015-05-13 18:38:47 +0300163 msg = ('Server was not rebuilt to the original image. '
164 'The original image: {0}. The current image: {1}'
165 .format(image_ref, rebuilt_server['image']['id']))
166 self.assertEqual(image_ref, rebuilt_server['image']['id'], msg)
167
zhufl01512d22017-08-03 16:30:23 +0800168 def _test_rebuild_server(self):
Castulo J. Martinez0d1b63f2016-07-08 13:55:57 -0700169 # Get the IPs the server has before rebuilding it
170 original_addresses = (self.client.show_server(self.server_id)['server']
171 ['addresses'])
Sean Dague4dd2c0b2013-01-03 17:50:28 -0500172 # The server should be rebuilt using the provided image and data
Daryl Wallecked8bef32011-12-05 23:02:08 -0600173 meta = {'rebuild': 'server'}
zhuflc6ce5392016-08-17 14:34:37 +0800174 new_name = data_utils.rand_name(self.__class__.__name__ + '-server')
Daryl Walleck98e66dd2012-06-21 04:58:39 -0500175 password = 'rebuildPassw0rd'
ghanshyam0f825252015-08-25 16:02:50 +0900176 rebuilt_server = self.client.rebuild_server(
177 self.server_id,
178 self.image_ref_alt,
179 name=new_name,
180 metadata=meta,
ghanshyam0f825252015-08-25 16:02:50 +0900181 adminPass=password)['server']
Daryl Wallecked8bef32011-12-05 23:02:08 -0600182
Andrea Frittolie8040812015-04-28 14:26:35 +0100183 # If the server was rebuilt on a different image, restore it to the
184 # original image once the test ends
185 if self.image_ref_alt != self.image_ref:
Yaroslav Lobankovfb1c3642015-05-13 18:38:47 +0300186 self.addCleanup(self._rebuild_server_and_check, self.image_ref)
Andrea Frittolie8040812015-04-28 14:26:35 +0100187
Attila Fazekasf7f34f92013-08-01 17:01:44 +0200188 # Verify the properties in the initial response are correct
Brian Waldon3bde07f2011-12-13 15:11:22 -0500189 self.assertEqual(self.server_id, rebuilt_server['id'])
Eoghan Glynn57c1a3d2012-03-01 16:50:29 -0500190 rebuilt_image_id = rebuilt_server['image']['id']
191 self.assertTrue(self.image_ref_alt.endswith(rebuilt_image_id))
Ken'ichi Ohmichi35772602013-11-14 15:03:27 +0900192 self.assertEqual(self.flavor_ref, rebuilt_server['flavor']['id'])
Daryl Wallecked8bef32011-12-05 23:02:08 -0600193
Attila Fazekasf7f34f92013-08-01 17:01:44 +0200194 # Verify the server properties after the rebuild completes
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000195 waiters.wait_for_server_status(self.client,
196 rebuilt_server['id'], 'ACTIVE')
ghanshyam0f825252015-08-25 16:02:50 +0900197 server = self.client.show_server(rebuilt_server['id'])['server']
Zhi Kun Liue5401762013-09-11 20:45:48 +0800198 rebuilt_image_id = server['image']['id']
Eoghan Glynn57c1a3d2012-03-01 16:50:29 -0500199 self.assertTrue(self.image_ref_alt.endswith(rebuilt_image_id))
Zhi Kun Liue5401762013-09-11 20:45:48 +0800200 self.assertEqual(new_name, server['name'])
Castulo J. Martinez0d1b63f2016-07-08 13:55:57 -0700201 self.assertEqual(original_addresses, server['addresses'])
Daryl Wallecked8bef32011-12-05 23:02:08 -0600202
Joseph Lanouxffe09dd2015-03-18 16:45:33 +0000203 if CONF.validation.run_validation:
wangxiyuan878c50e2016-01-04 19:24:17 +0800204 # Authentication is attempted in the following order of priority:
205 # 1.The key passed in, if one was passed in.
206 # 2.Any key we can find through an SSH agent (if allowed).
207 # 3.Any "id_rsa", "id_dsa" or "id_ecdsa" key discoverable in
208 # ~/.ssh/ (if allowed).
209 # 4.Plain username/password auth, if a password was given.
Joseph Lanouxffe09dd2015-03-18 16:45:33 +0000210 linux_client = remote_client.RemoteClient(
211 self.get_server_ip(rebuilt_server),
212 self.ssh_user,
wangxiyuan878c50e2016-01-04 19:24:17 +0800213 password,
Andrea Frittoli (andreaf)2a70a612016-04-29 16:09:13 -0500214 self.validation_resources['keypair']['private_key'],
215 server=rebuilt_server,
216 servers_client=self.client)
Attila Fazekasad7ef7d2013-11-20 10:12:53 +0100217 linux_client.validate_authentication()
Daryl Walleck98e66dd2012-06-21 04:58:39 -0500218
zhufl01512d22017-08-03 16:30:23 +0800219 @decorators.idempotent_id('aaa6cdf3-55a7-461a-add9-1c8596b9a07c')
220 def test_rebuild_server(self):
221 self._test_rebuild_server()
222
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800223 @decorators.idempotent_id('30449a88-5aff-4f9b-9866-6ee9b17f906d')
Zhi Kun Liue5401762013-09-11 20:45:48 +0800224 def test_rebuild_server_in_stop_state(self):
225 # The server in stop state should be rebuilt using the provided
226 # image and remain in SHUTOFF state
ghanshyam0f825252015-08-25 16:02:50 +0900227 server = self.client.show_server(self.server_id)['server']
Zhi Kun Liue5401762013-09-11 20:45:48 +0800228 old_image = server['image']['id']
Yaroslav Lobankovfb1c3642015-05-13 18:38:47 +0300229 new_image = (self.image_ref_alt
230 if old_image == self.image_ref else self.image_ref)
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000231 self.client.stop_server(self.server_id)
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000232 waiters.wait_for_server_status(self.client, self.server_id, 'SHUTOFF')
ghanshyam0f825252015-08-25 16:02:50 +0900233 rebuilt_server = (self.client.rebuild_server(self.server_id, new_image)
234 ['server'])
Yaroslav Lobankovfb1c3642015-05-13 18:38:47 +0300235 # If the server was rebuilt on a different image, restore it to the
236 # original image once the test ends
237 if self.image_ref_alt != self.image_ref:
238 self.addCleanup(self._rebuild_server_and_check, old_image)
Zhi Kun Liue5401762013-09-11 20:45:48 +0800239
240 # Verify the properties in the initial response are correct
241 self.assertEqual(self.server_id, rebuilt_server['id'])
242 rebuilt_image_id = rebuilt_server['image']['id']
243 self.assertEqual(new_image, rebuilt_image_id)
244 self.assertEqual(self.flavor_ref, rebuilt_server['flavor']['id'])
245
246 # Verify the server properties after the rebuild completes
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000247 waiters.wait_for_server_status(self.client,
248 rebuilt_server['id'], 'SHUTOFF')
ghanshyam0f825252015-08-25 16:02:50 +0900249 server = self.client.show_server(rebuilt_server['id'])['server']
Zhi Kun Liue5401762013-09-11 20:45:48 +0800250 rebuilt_image_id = server['image']['id']
251 self.assertEqual(new_image, rebuilt_image_id)
252
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000253 self.client.start_server(self.server_id)
Attila Fazekas305e65b2013-10-29 13:23:07 +0100254
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800255 @decorators.idempotent_id('b68bd8d6-855d-4212-b59b-2e704044dace')
Andrea Frittolicd368412017-08-14 21:37:56 +0100256 @utils.services('volume')
Roman Podoliaka908f9062015-04-21 16:52:20 +0300257 def test_rebuild_server_with_volume_attached(self):
258 # create a new volume and attach it to the server
Matt Riedemannf1448f02016-10-01 20:03:16 -0400259 volume = self.create_volume()
Roman Podoliaka908f9062015-04-21 16:52:20 +0300260
Matt Riedemannf1448f02016-10-01 20:03:16 -0400261 server = self.client.show_server(self.server_id)['server']
262 self.attach_volume(server, volume)
Roman Podoliaka908f9062015-04-21 16:52:20 +0300263
264 # run general rebuild test
zhufl01512d22017-08-03 16:30:23 +0800265 self._test_rebuild_server()
Roman Podoliaka908f9062015-04-21 16:52:20 +0300266
267 # make sure the volume is attached to the instance after rebuild
268 vol_after_rebuild = self.volumes_client.show_volume(volume['id'])
269 vol_after_rebuild = vol_after_rebuild['volume']
270 self.assertEqual('in-use', vol_after_rebuild['status'])
271 self.assertEqual(self.server_id,
272 vol_after_rebuild['attachments'][0]['server_id'])
273
Matt Riedemann9fd63b82014-04-14 09:33:05 -0700274 def _test_resize_server_confirm(self, stop=False):
Sean Dague4dd2c0b2013-01-03 17:50:28 -0500275 # The server's RAM and disk space should be modified to that of
276 # the provided flavor
Daryl Wallecked8bef32011-12-05 23:02:08 -0600277
Matt Riedemann9fd63b82014-04-14 09:33:05 -0700278 if stop:
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000279 self.client.stop_server(self.server_id)
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000280 waiters.wait_for_server_status(self.client, self.server_id,
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000281 'SHUTOFF')
Matt Riedemann9fd63b82014-04-14 09:33:05 -0700282
Ken'ichi Ohmichi5271b0f2015-08-10 07:53:27 +0000283 self.client.resize_server(self.server_id, self.flavor_ref_alt)
zhufl045d1f52016-08-05 15:06:35 +0800284 # NOTE(jlk): Explicitly delete the server to get a new one for later
285 # tests. Avoids resize down race issues.
286 self.addCleanup(self.delete_server, self.server_id)
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000287 waiters.wait_for_server_status(self.client, self.server_id,
288 'VERIFY_RESIZE')
Daryl Wallecked8bef32011-12-05 23:02:08 -0600289
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000290 self.client.confirm_resize_server(self.server_id)
Matt Riedemann9fd63b82014-04-14 09:33:05 -0700291 expected_status = 'SHUTOFF' if stop else 'ACTIVE'
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000292 waiters.wait_for_server_status(self.client, self.server_id,
293 expected_status)
Daryl Wallecked8bef32011-12-05 23:02:08 -0600294
ghanshyam0f825252015-08-25 16:02:50 +0900295 server = self.client.show_server(self.server_id)['server']
Jesse Keating613b4982015-05-04 15:05:19 -0700296 self.assertEqual(self.flavor_ref_alt, server['flavor']['id'])
Daryl Wallecked8bef32011-12-05 23:02:08 -0600297
Matt Riedemann9fd63b82014-04-14 09:33:05 -0700298 if stop:
299 # NOTE(mriedem): tearDown requires the server to be started.
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000300 self.client.start_server(self.server_id)
Matt Riedemann9fd63b82014-04-14 09:33:05 -0700301
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800302 @decorators.idempotent_id('1499262a-9328-4eda-9068-db1ac57498d2')
Matt Riedemann9fd63b82014-04-14 09:33:05 -0700303 @testtools.skipUnless(CONF.compute_feature_enabled.resize,
304 'Resize not available.')
Matt Riedemann9fd63b82014-04-14 09:33:05 -0700305 def test_resize_server_confirm(self):
306 self._test_resize_server_confirm(stop=False)
307
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800308 @decorators.idempotent_id('138b131d-66df-48c9-a171-64f45eb92962')
Matt Riedemann9fd63b82014-04-14 09:33:05 -0700309 @testtools.skipUnless(CONF.compute_feature_enabled.resize,
310 'Resize not available.')
Matt Riedemann9fd63b82014-04-14 09:33:05 -0700311 def test_resize_server_confirm_from_stopped(self):
312 self._test_resize_server_confirm(stop=True)
313
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800314 @decorators.idempotent_id('c03aab19-adb1-44f5-917d-c419577e9e68')
Matt Riedemann5d1f8b12014-03-24 09:12:53 -0700315 @testtools.skipUnless(CONF.compute_feature_enabled.resize,
316 'Resize not available.')
Daryl Wallecked8bef32011-12-05 23:02:08 -0600317 def test_resize_server_revert(self):
Sean Dague4dd2c0b2013-01-03 17:50:28 -0500318 # The server's RAM and disk space should return to its original
319 # values after a resize is reverted
Daryl Wallecked8bef32011-12-05 23:02:08 -0600320
Ken'ichi Ohmichi5271b0f2015-08-10 07:53:27 +0000321 self.client.resize_server(self.server_id, self.flavor_ref_alt)
zhufl045d1f52016-08-05 15:06:35 +0800322 # NOTE(zhufl): Explicitly delete the server to get a new one for later
323 # tests. Avoids resize down race issues.
324 self.addCleanup(self.delete_server, self.server_id)
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000325 waiters.wait_for_server_status(self.client, self.server_id,
326 'VERIFY_RESIZE')
Daryl Wallecked8bef32011-12-05 23:02:08 -0600327
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000328 self.client.revert_resize_server(self.server_id)
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000329 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
Daryl Wallecked8bef32011-12-05 23:02:08 -0600330
ghanshyam0f825252015-08-25 16:02:50 +0900331 server = self.client.show_server(self.server_id)['server']
Jesse Keating613b4982015-05-04 15:05:19 -0700332 self.assertEqual(self.flavor_ref, server['flavor']['id'])
sapan-konaf1e3f002011-12-22 23:18:44 +0530333
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800334 @decorators.idempotent_id('b963d4f1-94b3-4c40-9e97-7b583f46e470')
Adam Gandelmanfbc95ac2014-06-19 17:33:43 -0700335 @testtools.skipUnless(CONF.compute_feature_enabled.snapshot,
336 'Snapshotting not available, backup not possible.')
Andrea Frittolicd368412017-08-14 21:37:56 +0100337 @utils.services('image')
ivan-zhuccc89462013-10-31 16:53:12 +0800338 def test_create_backup(self):
339 # Positive test:create backup successfully and rotate backups correctly
340 # create the first and the second backup
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400341
342 # Check if glance v1 is available to determine which client to use. We
343 # prefer glance v1 for the compute API tests since the compute image
344 # API proxy was written for glance v1.
345 if CONF.image_feature_enabled.api_v1:
Jordan Pittier8160d312017-04-18 11:52:23 +0200346 glance_client = self.os_primary.image_client
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400347 elif CONF.image_feature_enabled.api_v2:
Jordan Pittier8160d312017-04-18 11:52:23 +0200348 glance_client = self.os_primary.image_client_v2
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400349 else:
Matthew Treinish4217a702016-10-07 17:27:11 -0400350 raise lib_exc.InvalidConfiguration(
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400351 'Either api_v1 or api_v2 must be True in '
352 '[image-feature-enabled].')
353
Matt Riedemanne699f822014-01-24 08:11:42 -0800354 backup1 = data_utils.rand_name('backup-1')
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000355 resp = self.client.create_backup(self.server_id,
Ken'ichi Ohmichie6349f32015-12-09 06:47:54 +0000356 backup_type='daily',
357 rotation=2,
358 name=backup1).response
ivan-zhuccc89462013-10-31 16:53:12 +0800359 oldest_backup_exist = True
360
361 # the oldest one should be deleted automatically in this test
362 def _clean_oldest_backup(oldest_backup):
363 if oldest_backup_exist:
Ryan Hsu38231092014-06-08 23:12:10 -0700364 try:
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400365 glance_client.delete_image(oldest_backup)
Masayuki Igawabfa07602015-01-20 18:47:17 +0900366 except lib_exc.NotFound:
Ryan Hsu38231092014-06-08 23:12:10 -0700367 pass
368 else:
369 LOG.warning("Deletion of oldest backup %s should not have "
370 "been successful as it should have been "
Jordan Pittier525ec712016-12-07 17:51:26 +0100371 "deleted during rotation.", oldest_backup)
ivan-zhuccc89462013-10-31 16:53:12 +0800372
Felipe Monteiro9ff5c282017-06-21 21:05:07 +0100373 if api_version_utils.compare_version_header_to_response(
374 "OpenStack-API-Version", "compute 2.45", resp, "lt"):
375 image1_id = resp['image_id']
376 else:
377 image1_id = data_utils.parse_image_id(resp['location'])
ivan-zhuccc89462013-10-31 16:53:12 +0800378 self.addCleanup(_clean_oldest_backup, image1_id)
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400379 waiters.wait_for_image_status(glance_client,
Yaroslav Lobankov2fea4052016-04-19 15:05:57 +0300380 image1_id, 'active')
ivan-zhuccc89462013-10-31 16:53:12 +0800381
Matt Riedemanne699f822014-01-24 08:11:42 -0800382 backup2 = data_utils.rand_name('backup-2')
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000383 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
384 resp = self.client.create_backup(self.server_id,
Ken'ichi Ohmichie6349f32015-12-09 06:47:54 +0000385 backup_type='daily',
386 rotation=2,
387 name=backup2).response
Felipe Monteiro9ff5c282017-06-21 21:05:07 +0100388 if api_version_utils.compare_version_header_to_response(
389 "OpenStack-API-Version", "compute 2.45", resp, "lt"):
390 image2_id = resp['image_id']
391 else:
392 image2_id = data_utils.parse_image_id(resp['location'])
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400393 self.addCleanup(glance_client.delete_image, image2_id)
394 waiters.wait_for_image_status(glance_client,
Yaroslav Lobankov2fea4052016-04-19 15:05:57 +0300395 image2_id, 'active')
ivan-zhuccc89462013-10-31 16:53:12 +0800396
397 # verify they have been created
398 properties = {
399 'image_type': 'backup',
400 'backup_type': "daily",
401 'instance_uuid': self.server_id,
402 }
Ken'ichi Ohmichi45d7d422016-06-22 10:03:28 -0700403 params = {
404 'status': 'active',
405 'sort_key': 'created_at',
406 'sort_dir': 'asc'
407 }
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400408 if CONF.image_feature_enabled.api_v1:
Ken'ichi Ohmichie28b2c22016-06-22 10:07:27 -0700409 for key, value in properties.items():
410 params['property-%s' % key] = value
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400411 image_list = glance_client.list_images(
412 detail=True,
413 **params)['images']
414 else:
415 # Additional properties are flattened in glance v2.
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400416 params.update(properties)
417 image_list = glance_client.list_images(params)['images']
418
ivan-zhuccc89462013-10-31 16:53:12 +0800419 self.assertEqual(2, len(image_list))
420 self.assertEqual((backup1, backup2),
421 (image_list[0]['name'], image_list[1]['name']))
422
423 # create the third one, due to the rotation is 2,
424 # the first one will be deleted
Matt Riedemanne699f822014-01-24 08:11:42 -0800425 backup3 = data_utils.rand_name('backup-3')
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000426 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
427 resp = self.client.create_backup(self.server_id,
Ken'ichi Ohmichie6349f32015-12-09 06:47:54 +0000428 backup_type='daily',
429 rotation=2,
430 name=backup3).response
Felipe Monteiro9ff5c282017-06-21 21:05:07 +0100431 if api_version_utils.compare_version_header_to_response(
432 "OpenStack-API-Version", "compute 2.45", resp, "lt"):
433 image3_id = resp['image_id']
434 else:
435 image3_id = data_utils.parse_image_id(resp['location'])
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400436 self.addCleanup(glance_client.delete_image, image3_id)
ivan-zhuccc89462013-10-31 16:53:12 +0800437 # the first back up should be deleted
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000438 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400439 glance_client.wait_for_resource_deletion(image1_id)
ivan-zhuccc89462013-10-31 16:53:12 +0800440 oldest_backup_exist = False
Matt Riedemann2aa19d42016-06-06 17:45:41 -0400441 if CONF.image_feature_enabled.api_v1:
442 image_list = glance_client.list_images(
443 detail=True, **params)['images']
444 else:
445 image_list = glance_client.list_images(params)['images']
Matt Riedemanne699f822014-01-24 08:11:42 -0800446 self.assertEqual(2, len(image_list),
447 'Unexpected number of images for '
448 'v2:test_create_backup; was the oldest backup not '
449 'yet deleted? Image list: %s' %
450 [image['name'] for image in image_list])
ivan-zhuccc89462013-10-31 16:53:12 +0800451 self.assertEqual((backup2, backup3),
452 (image_list[0]['name'], image_list[1]['name']))
453
Ken'ichi Ohmichi547a3062013-12-02 09:30:36 +0900454 def _get_output(self):
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000455 output = self.client.get_console_output(
Ken'ichi Ohmichibf4766a2015-12-09 07:48:43 +0000456 self.server_id, length=10)['output']
Ken'ichi Ohmichi547a3062013-12-02 09:30:36 +0900457 self.assertTrue(output, "Console output was empty.")
458 lines = len(output.split('\n'))
459 self.assertEqual(lines, 10)
460
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800461 @decorators.idempotent_id('4b8867e6-fffa-4d54-b1d1-6fdda57be2f3')
Adam Gandelmanc6eefb42014-07-15 16:44:08 -0700462 @testtools.skipUnless(CONF.compute_feature_enabled.console_output,
463 'Console output not supported.')
Attila Fazekas45c3fbc2013-02-24 16:39:52 +0100464 def test_get_console_output(self):
465 # Positive test:Should be able to GET the console output
466 # for a given server_id and number of lines
Ken'ichi Ohmichi851a1352013-11-26 18:42:52 +0900467
468 # This reboot is necessary for outputting some console log after
Takashi NATSUME6d5a2b42015-09-08 11:27:49 +0900469 # creating an instance backup. If an instance backup, the console
Ken'ichi Ohmichi851a1352013-11-26 18:42:52 +0900470 # log file is truncated and we cannot get any console log through
471 # "console-log" API.
472 # The detail is https://bugs.launchpad.net/nova/+bug/1251920
Ken'ichi Ohmichi36b714c2015-12-09 08:12:47 +0000473 self.client.reboot_server(self.server_id, type='HARD')
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000474 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
Ken'ichi Ohmichi547a3062013-12-02 09:30:36 +0900475 self.wait_for(self._get_output)
Attila Fazekas45c3fbc2013-02-24 16:39:52 +0100476
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800477 @decorators.idempotent_id('89104062-69d8-4b19-a71b-f47b7af093d7')
Adam Gandelmanc6eefb42014-07-15 16:44:08 -0700478 @testtools.skipUnless(CONF.compute_feature_enabled.console_output,
479 'Console output not supported.')
Ken'ichi Ohmichibd5df122014-09-18 08:03:52 +0000480 def test_get_console_output_with_unlimited_size(self):
David Kranz0fb14292015-02-11 15:55:20 -0500481 server = self.create_test_server(wait_until='ACTIVE')
Ken'ichi Ohmichibd5df122014-09-18 08:03:52 +0000482
483 def _check_full_length_console_log():
Ken'ichi Ohmichibf4766a2015-12-09 07:48:43 +0000484 output = self.client.get_console_output(server['id'])['output']
Ken'ichi Ohmichibd5df122014-09-18 08:03:52 +0000485 self.assertTrue(output, "Console output was empty.")
486 lines = len(output.split('\n'))
487
488 # NOTE: This test tries to get full length console log, and the
489 # length should be bigger than the one of test_get_console_output.
Béla Vancsics64862f72016-11-08 09:12:31 +0100490 self.assertGreater(lines, 10, "Cannot get enough console log "
491 "length. (lines: %s)" % lines)
Ken'ichi Ohmichibd5df122014-09-18 08:03:52 +0000492
493 self.wait_for(_check_full_length_console_log)
494
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800495 @decorators.idempotent_id('5b65d4e7-4ecd-437c-83c0-d6b79d927568')
Ken'ichi Ohmichibd5df122014-09-18 08:03:52 +0000496 @testtools.skipUnless(CONF.compute_feature_enabled.console_output,
497 'Console output not supported.')
Ken'ichi Ohmichi547a3062013-12-02 09:30:36 +0900498 def test_get_console_output_server_id_in_shutoff_status(self):
Attila Fazekas45c3fbc2013-02-24 16:39:52 +0100499 # Positive test:Should be able to GET the console output
Ken'ichi Ohmichi547a3062013-12-02 09:30:36 +0900500 # for a given server_id in SHUTOFF status
501
502 # NOTE: SHUTOFF is irregular status. To avoid test instability,
503 # one server is created only for this test without using
Ken'ichi Ohmichi4d8ba232017-03-03 18:28:05 -0800504 # the server that was created in setUpClass.
David Kranz0fb14292015-02-11 15:55:20 -0500505 server = self.create_test_server(wait_until='ACTIVE')
Ken'ichi Ohmichi547a3062013-12-02 09:30:36 +0900506 temp_server_id = server['id']
507
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000508 self.client.stop_server(temp_server_id)
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000509 waiters.wait_for_server_status(self.client, temp_server_id, 'SHUTOFF')
Ken'ichi Ohmichi547a3062013-12-02 09:30:36 +0900510 self.wait_for(self._get_output)
Dan Smithbc3bd242012-08-14 09:48:48 -0700511
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800512 @decorators.idempotent_id('bd61a9fd-062f-4670-972b-2d6c3e3b9e73')
Matt Riedemann992162e2014-03-24 09:11:06 -0700513 @testtools.skipUnless(CONF.compute_feature_enabled.pause,
514 'Pause is not available.')
Prem Karat6631f802013-07-04 12:07:33 +0530515 def test_pause_unpause_server(self):
David Kranzae99b9a2015-02-16 13:37:01 -0500516 self.client.pause_server(self.server_id)
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000517 waiters.wait_for_server_status(self.client, self.server_id, 'PAUSED')
David Kranzae99b9a2015-02-16 13:37:01 -0500518 self.client.unpause_server(self.server_id)
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000519 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
Prem Karat6631f802013-07-04 12:07:33 +0530520
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800521 @decorators.idempotent_id('0d8ee21e-b749-462d-83da-b85b41c86c7f')
Matt Riedemannf0cf6952014-03-24 09:08:52 -0700522 @testtools.skipUnless(CONF.compute_feature_enabled.suspend,
523 'Suspend is not available.')
Prem Karat6631f802013-07-04 12:07:33 +0530524 def test_suspend_resume_server(self):
David Kranzae99b9a2015-02-16 13:37:01 -0500525 self.client.suspend_server(self.server_id)
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000526 waiters.wait_for_server_status(self.client, self.server_id,
527 'SUSPENDED')
David Kranzae99b9a2015-02-16 13:37:01 -0500528 self.client.resume_server(self.server_id)
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000529 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
Prem Karat6631f802013-07-04 12:07:33 +0530530
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800531 @decorators.idempotent_id('77eba8e0-036e-4635-944b-f7a8f3b78dc9')
David Shrewsbury25f666f2014-07-22 12:17:59 -0400532 @testtools.skipUnless(CONF.compute_feature_enabled.shelve,
533 'Shelve is not available.')
Andrea Frittolicd368412017-08-14 21:37:56 +0100534 @utils.services('image')
Ken'ichi Ohmichi39437e22013-10-06 00:21:38 +0900535 def test_shelve_unshelve_server(self):
ghanshyam43a40652017-06-27 11:23:27 +0300536 if CONF.image_feature_enabled.api_v2:
537 glance_client = self.os_primary.image_client_v2
538 elif CONF.image_feature_enabled.api_v1:
539 glance_client = self.os_primary.image_client
540 else:
541 raise lib_exc.InvalidConfiguration(
542 'Either api_v1 or api_v2 must be True in '
543 '[image-feature-enabled].')
ghanshyam017b5fe2016-04-15 18:49:26 +0900544 compute.shelve_server(self.client, self.server_id,
545 force_shelve_offload=True)
Ken'ichi Ohmichi17b7f112014-02-20 06:33:49 +0900546
ghanshyam0f825252015-08-25 16:02:50 +0900547 server = self.client.show_server(self.server_id)['server']
Ken'ichi Ohmichi39437e22013-10-06 00:21:38 +0900548 image_name = server['name'] + '-shelved'
549 params = {'name': image_name}
ghanshyam43a40652017-06-27 11:23:27 +0300550 if CONF.image_feature_enabled.api_v2:
551 images = glance_client.list_images(params)['images']
552 elif CONF.image_feature_enabled.api_v1:
553 images = glance_client.list_images(
554 detail=True, **params)['images']
Ken'ichi Ohmichi39437e22013-10-06 00:21:38 +0900555 self.assertEqual(1, len(images))
556 self.assertEqual(image_name, images[0]['name'])
557
David Kranzae99b9a2015-02-16 13:37:01 -0500558 self.client.unshelve_server(self.server_id)
Ken'ichi Ohmichi0eb153c2015-07-13 02:18:25 +0000559 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
ghanshyam43a40652017-06-27 11:23:27 +0300560 glance_client.wait_for_resource_deletion(images[0]['id'])
rsritesh8fdfb1c2017-05-08 11:14:00 +0530561
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800562 @decorators.idempotent_id('af8eafd4-38a7-4a4b-bdbc-75145a580560')
Anju Tiwariaf98d222013-08-05 15:19:50 +0530563 def test_stop_start_server(self):
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000564 self.client.stop_server(self.server_id)
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000565 waiters.wait_for_server_status(self.client, self.server_id, 'SHUTOFF')
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000566 self.client.start_server(self.server_id)
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000567 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
Anju Tiwariaf98d222013-08-05 15:19:50 +0530568
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800569 @decorators.idempotent_id('80a8094c-211e-440a-ab88-9e59d556c7ee')
Zhu Zhu9643e512013-09-23 09:13:07 -0500570 def test_lock_unlock_server(self):
571 # Lock the server,try server stop(exceptions throw),unlock it and retry
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000572 self.client.lock_server(self.server_id)
Viktor Tikkanen3d063df2015-06-26 07:55:11 +0300573 self.addCleanup(self.client.unlock_server, self.server_id)
ghanshyam0f825252015-08-25 16:02:50 +0900574 server = self.client.show_server(self.server_id)['server']
Zhu Zhu9643e512013-09-23 09:13:07 -0500575 self.assertEqual(server['status'], 'ACTIVE')
576 # Locked server is not allowed to be stopped by non-admin user
Masayuki Igawad9388762015-01-20 14:56:42 +0900577 self.assertRaises(lib_exc.Conflict,
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000578 self.client.stop_server, self.server_id)
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000579 self.client.unlock_server(self.server_id)
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000580 self.client.stop_server(self.server_id)
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000581 waiters.wait_for_server_status(self.client, self.server_id, 'SHUTOFF')
Ken'ichi Ohmichib2631082015-08-27 01:31:00 +0000582 self.client.start_server(self.server_id)
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000583 waiters.wait_for_server_status(self.client, self.server_id, 'ACTIVE')
Zhu Zhu9643e512013-09-23 09:13:07 -0500584
Ghanshyam86d1a572014-03-05 10:19:25 +0900585 def _validate_url(self, url):
586 valid_scheme = ['http', 'https']
587 parsed_url = urlparse.urlparse(url)
588 self.assertNotEqual('None', parsed_url.port)
589 self.assertNotEqual('None', parsed_url.hostname)
590 self.assertIn(parsed_url.scheme, valid_scheme)
591
Ken'ichi Ohmichi14b0ae12017-01-27 17:18:52 -0800592 @decorators.idempotent_id('c6bc11bf-592e-4015-9319-1c98dc64daf5')
Ghanshyam86d1a572014-03-05 10:19:25 +0900593 @testtools.skipUnless(CONF.compute_feature_enabled.vnc_console,
594 'VNC Console feature is disabled.')
Ghanshyam86d1a572014-03-05 10:19:25 +0900595 def test_get_vnc_console(self):
596 # Get the VNC console of type 'novnc' and 'xvpvnc'
597 console_types = ['novnc', 'xvpvnc']
598 for console_type in console_types:
Ken'ichi Ohmichi32752ff2015-07-13 04:33:23 +0000599 body = self.client.get_vnc_console(self.server_id,
Ken'ichi Ohmichibf4766a2015-12-09 07:48:43 +0000600 type=console_type)['console']
Ghanshyam86d1a572014-03-05 10:19:25 +0900601 self.assertEqual(console_type, body['type'])
602 self.assertNotEqual('', body['url'])
603 self._validate_url(body['url'])