From 2640f81754126c9d3ecd668eb99fb006f68b709b Mon Sep 17 00:00:00 2001 From: Vishvananda Ishaya Date: Fri, 3 Aug 2012 19:31:48 -0700 Subject: Key block_device_mapping off of volume extension partially implements blueprint disable-server-extensions Instead of creating a child controller for boot from volume, key the post parameter of of the os-volumes extension. Change-Id: I0dd233c2ae12a09d6fc59c49042c2f8b83f3fdba --- nova/tests/api/openstack/compute/test_servers.py | 41 +++++++++++++++++++----- 1 file changed, 33 insertions(+), 8 deletions(-) (limited to 'nova/tests') diff --git a/nova/tests/api/openstack/compute/test_servers.py b/nova/tests/api/openstack/compute/test_servers.py index e482ca812..edd53446a 100644 --- a/nova/tests/api/openstack/compute/test_servers.py +++ b/nova/tests/api/openstack/compute/test_servers.py @@ -1748,13 +1748,11 @@ class ServersControllerCreateTest(test.TestCase): fakes.stub_out_key_pair_funcs(self.stubs, have_key_pair=False) self._test_create_instance() - def _test_create_security_group(self, group): + def _test_create_extra(self, params): image_uuid = 'c905cedb-7281-47e4-8a62-f26bc5fc4c77' - body = dict(server=dict( - name='server_test', imageRef=image_uuid, flavorRef=2, - metadata={'hello': 'world', 'open': 'stack'}, - security_groups=[{'name': group}], - personality={})) + server = dict(name='server_test', imageRef=image_uuid, flavorRef=2) + server.update(params) + body = dict(server=server) req = fakes.HTTPRequest.blank('/v2/fake/servers') req.method = 'POST' req.body = jsonutils.dumps(body) @@ -1764,6 +1762,7 @@ class ServersControllerCreateTest(test.TestCase): def test_create_instance_with_security_group_enabled(self): self.ext_mgr.extensions = {'os-security-groups': 'fake'} group = 'foo' + params = {'security_groups': [{'name': group}]} old_create = nova.compute.api.API.create def create(*args, **kwargs): @@ -1771,10 +1770,11 @@ class ServersControllerCreateTest(test.TestCase): return old_create(*args, **kwargs) self.stubs.Set(nova.compute.api.API, 'create', create) - self._test_create_security_group(group) + self._test_create_extra(params) def test_create_instance_with_security_group_disabled(self): group = 'foo' + params = {'security_groups': [{'name': group}]} old_create = nova.compute.api.API.create def create(*args, **kwargs): @@ -1785,7 +1785,32 @@ class ServersControllerCreateTest(test.TestCase): return old_create(*args, **kwargs) self.stubs.Set(nova.compute.api.API, 'create', create) - self._test_create_security_group(group) + self._test_create_extra(params) + + def test_create_instance_with_volumes_enabled(self): + self.ext_mgr.extensions = {'os-volumes': 'fake'} + bdm = [{'device_name': 'foo'}] + params = {'block_device_mapping': bdm} + old_create = nova.compute.api.API.create + + def create(*args, **kwargs): + self.assertEqual(kwargs['block_device_mapping'], bdm) + return old_create(*args, **kwargs) + + self.stubs.Set(nova.compute.api.API, 'create', create) + self._test_create_extra(params) + + def test_create_instance_with_volumes_disabled(self): + bdm = [{'device_name': 'foo'}] + params = {'block_device_mapping': bdm} + old_create = nova.compute.api.API.create + + def create(*args, **kwargs): + self.assertEqual(kwargs['block_device_mapping'], None) + return old_create(*args, **kwargs) + + self.stubs.Set(nova.compute.api.API, 'create', create) + self._test_create_extra(params) def test_create_instance_with_access_ip(self): # proper local hrefs must start with 'http://localhost/v2/' -- cgit From 0bfbce95c8bb867d8e8d27cb7c7429465cc88f15 Mon Sep 17 00:00:00 2001 From: Vishvananda Ishaya Date: Tue, 7 Aug 2012 14:08:21 -0700 Subject: Key user_data in create server off of extension Implements part of blueprint disable-server-extensions Adds a new extension descriptor for user_data and makes sure that user_data is only accepted in the create server request if the extension is enabled. Change-Id: I7aef0ebe569da841adbf380d5695c9029fb3b002 --- .../tests/api/openstack/compute/test_extensions.py | 1 + nova/tests/api/openstack/compute/test_servers.py | 25 ++++++++++++++++++++++ 2 files changed, 26 insertions(+) (limited to 'nova/tests') diff --git a/nova/tests/api/openstack/compute/test_extensions.py b/nova/tests/api/openstack/compute/test_extensions.py index ebecb3ce6..48875dd3d 100644 --- a/nova/tests/api/openstack/compute/test_extensions.py +++ b/nova/tests/api/openstack/compute/test_extensions.py @@ -184,6 +184,7 @@ class ExtensionControllerTest(ExtensionTestCase): "ServerDiagnostics", "ServerStartStop", "SimpleTenantUsage", + "UserData", "VirtualInterfaces", "Volumes", "VolumeTypes", diff --git a/nova/tests/api/openstack/compute/test_servers.py b/nova/tests/api/openstack/compute/test_servers.py index edd53446a..6e81d99bc 100644 --- a/nova/tests/api/openstack/compute/test_servers.py +++ b/nova/tests/api/openstack/compute/test_servers.py @@ -1812,6 +1812,31 @@ class ServersControllerCreateTest(test.TestCase): self.stubs.Set(nova.compute.api.API, 'create', create) self._test_create_extra(params) + def test_create_instance_with_user_data_enabled(self): + self.ext_mgr.extensions = {'os-user-data': 'fake'} + user_data = 'fake' + params = {'user_data': user_data} + old_create = nova.compute.api.API.create + + def create(*args, **kwargs): + self.assertEqual(kwargs['user_data'], user_data) + return old_create(*args, **kwargs) + + self.stubs.Set(nova.compute.api.API, 'create', create) + self._test_create_extra(params) + + def test_create_instance_with_user_data_disabled(self): + user_data = 'fake' + params = {'user_data': user_data} + old_create = nova.compute.api.API.create + + def create(*args, **kwargs): + self.assertEqual(kwargs['user_data'], None) + return old_create(*args, **kwargs) + + self.stubs.Set(nova.compute.api.API, 'create', create) + self._test_create_extra(params) + def test_create_instance_with_access_ip(self): # proper local hrefs must start with 'http://localhost/v2/' image_uuid = '76fa36fc-c930-4bf3-8c8a-ea2a2420deb6' -- cgit