openstack/nova
Code Issues Proposed changes

Merge "Fix get_vnc_console race."

Browse Source
This commit is contained in:
Jenkins 2012-08-17 20:14:21 +00:00 committed by Gerrit Code Review
commit a10be151ad
2 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
nova/compute/api.py
View File

@ -1627,6 +1627,9 @@ class API(base.Base):
@wrap_check_policy
def get_vnc_console(self, context, instance, console_type):
"""Get a url to an instance Console."""
if not instance['host']:
raise exception.InstanceNotReady(instance=instance)
connect_info = self.compute_rpcapi.get_vnc_console(context,
instance=instance, console_type=console_type)

9
nova/tests/compute/test_compute.py
View File

@ -3899,6 +3899,15 @@ class ComputeAPITestCase(BaseTestCase):
fake_instance, fake_console_type)
self.assertEqual(console, {'url': 'fake_console_url'})
def test_get_vnc_console_no_host(self):
instance = self._create_fake_instance(params={'host': ''})
self.assertRaises(exception.InstanceNotReady,
self.compute_api.get_vnc_console,
self.context, instance, 'novnc')
db.instance_destroy(self.context, instance['uuid'])
def test_console_output(self):
fake_instance = {'uuid': 'fake_uuid',
'host': 'fake_compute_host'}