Stop handling 'InstanceUnknownCell' exception

This was only raised by cells v1 code and therefore no longer needs to be
handled. A future change will remove the exception itself.

Part of blueprint remove-cells-v1

Change-Id: I1dd6abcc2be17ff76f108e7ff3771314f33259c6
Signed-off-by: Stephen Finucane <[email protected]>

Author: stephenfin

nova/api/openstack/compute/admin_actions.py

@@ -44,8 +44,6 @@ def _reset_network(self, req, id, body):
         instance = common.get_instance(self.compute_api, context, id)
         try:
             self.compute_api.reset_network(context, instance)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
 
@@ -59,8 +57,6 @@ def _inject_network_info(self, req, id, body):
         instance = common.get_instance(self.compute_api, context, id)
         try:
             self.compute_api.inject_network_info(context, instance)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
 

nova/api/openstack/compute/admin_password.py

@@ -47,8 +47,6 @@ def change_password(self, req, id, body):
         password = body['changePassword']['adminPass']
         try:
             self.compute_api.set_admin_password(context, instance, password)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except (exception.InstancePasswordSetFailed,
                 exception.SetAdminPasswdNotSupported,
                 exception.InstanceAgentNotEnabled) as e:

nova/api/openstack/compute/create_backup.py

@@ -68,8 +68,6 @@ def _create_backup(self, req, id, body):
         try:
             image = self.compute_api.backup(context, instance, image_name,
                     backup_type, rotation, extra_properties=props)
-        except exception.InstanceUnknownCell as e:
-            raise webob.exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:
             common.raise_http_conflict_for_instance_invalid_state(state_error,
                     'createBackup', id)

nova/api/openstack/compute/deferred_delete.py

@@ -39,8 +39,6 @@ def _restore(self, req, id, body):
         instance = common.get_instance(self.compute_api, context, id)
         try:
             self.compute_api.restore(context, instance)
-        except exception.InstanceUnknownCell as error:
-            raise webob.exc.HTTPNotFound(explanation=error.format_message())
         except exception.QuotaError as error:
             raise webob.exc.HTTPForbidden(explanation=error.format_message())
         except exception.InstanceInvalidState as state_error:
@@ -59,8 +57,7 @@ def _force_delete(self, req, id, body):
                             'project_id': instance.project_id})
         try:
             self.compute_api.force_delete(context, instance)
-        except (exception.InstanceNotFound,
-                exception.InstanceUnknownCell) as e:
+        except exception.InstanceNotFound as e:
             raise webob.exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceIsLocked as e:
             raise webob.exc.HTTPConflict(explanation=e.format_message())

nova/api/openstack/compute/evacuate.py

@@ -132,8 +132,6 @@ def _evacuate(self, req, id, body):
         try:
             self.compute_api.evacuate(context, instance, host,
                                       on_shared_storage, password, force)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:
             common.raise_http_conflict_for_instance_invalid_state(state_error,
                     'evacuate', id)

nova/api/openstack/compute/floating_ips.py

@@ -272,8 +272,6 @@ def _add_floating_ip(self, req, id, body):
         except exception.NoFloatingIpInterface:
             msg = _('l3driver call to add floating IP failed')
             raise webob.exc.HTTPBadRequest(explanation=msg)
-        except exception.InstanceUnknownCell as e:
-            raise webob.exc.HTTPNotFound(explanation=e.format_message())
         except exception.FloatingIpNotFoundForAddress:
             msg = _('floating IP not found')
             raise webob.exc.HTTPNotFound(explanation=msg)

nova/api/openstack/compute/migrate_server.py

@@ -139,8 +139,6 @@ def _migrate_live(self, req, id, body):
             self.compute_api.live_migrate(context, instance, block_migration,
                                           disk_over_commit, host, force,
                                           async_)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except (exception.NoValidHost,
                 exception.ComputeServiceUnavailable,
                 exception.InvalidHypervisorType,

nova/api/openstack/compute/multinic.py

@@ -47,8 +47,6 @@ def _add_fixed_ip(self, req, id, body):
         network_id = body['addFixedIp']['networkId']
         try:
             self.compute_api.add_fixed_ip(context, instance, network_id)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.NoMoreFixedIps as e:
             raise exc.HTTPBadRequest(explanation=e.format_message())
 
@@ -67,7 +65,5 @@ def _remove_fixed_ip(self, req, id, body):
 
         try:
             self.compute_api.remove_fixed_ip(context, instance, address)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.FixedIpNotFoundForSpecificInstance as e:
             raise exc.HTTPBadRequest(explanation=e.format_message())

nova/api/openstack/compute/pause_server.py

@@ -44,8 +44,7 @@ def _pause(self, req, id, body):
         except exception.InstanceInvalidState as state_error:
             common.raise_http_conflict_for_instance_invalid_state(state_error,
                     'pause', id)
-        except (exception.InstanceUnknownCell,
-                     exception.InstanceNotFound) as e:
+        except exception.InstanceNotFound as e:
             raise exc.HTTPNotFound(explanation=e.format_message())
         except NotImplementedError:
             common.raise_feature_not_supported()
@@ -65,8 +64,7 @@ def _unpause(self, req, id, body):
         except exception.InstanceInvalidState as state_error:
             common.raise_http_conflict_for_instance_invalid_state(state_error,
                     'unpause', id)
-        except (exception.InstanceUnknownCell,
-                     exception.InstanceNotFound) as e:
+        except exception.InstanceNotFound as e:
             raise exc.HTTPNotFound(explanation=e.format_message())
         except NotImplementedError:
             common.raise_feature_not_supported()

nova/api/openstack/compute/remote_consoles.py

@@ -52,8 +52,7 @@ def get_vnc_console(self, req, id, body):
                                                       console_type)
         except exception.ConsoleTypeUnavailable as e:
             raise webob.exc.HTTPBadRequest(explanation=e.format_message())
-        except (exception.InstanceUnknownCell,
-                     exception.InstanceNotFound) as e:
+        except exception.InstanceNotFound as e:
             raise webob.exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceNotReady as e:
             raise webob.exc.HTTPConflict(explanation=e.format_message())
@@ -81,8 +80,7 @@ def get_spice_console(self, req, id, body):
                                                         console_type)
         except exception.ConsoleTypeUnavailable as e:
             raise webob.exc.HTTPBadRequest(explanation=e.format_message())
-        except (exception.InstanceUnknownCell,
-                     exception.InstanceNotFound) as e:
+        except exception.InstanceNotFound as e:
             raise webob.exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceNotReady as e:
             raise webob.exc.HTTPConflict(explanation=e.format_message())
@@ -112,8 +110,7 @@ def get_rdp_console(self, req, id, body):
                                                       console_type)
         except exception.ConsoleTypeUnavailable as e:
             raise webob.exc.HTTPBadRequest(explanation=e.format_message())
-        except (exception.InstanceUnknownCell,
-                     exception.InstanceNotFound) as e:
+        except exception.InstanceNotFound as e:
             raise webob.exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceNotReady as e:
             raise webob.exc.HTTPConflict(explanation=e.format_message())
@@ -138,8 +135,7 @@ def get_serial_console(self, req, id, body):
             output = self.compute_api.get_serial_console(context,
                                                          instance,
                                                          console_type)
-        except (exception.InstanceUnknownCell,
-                     exception.InstanceNotFound) as e:
+        except exception.InstanceNotFound as e:
             raise webob.exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceNotReady as e:
             raise webob.exc.HTTPConflict(explanation=e.format_message())

nova/api/openstack/compute/rescue.py

@@ -61,8 +61,6 @@ def _rescue(self, req, id, body):
             self.compute_api.rescue(context, instance,
                                     rescue_password=password,
                                     rescue_image_ref=rescue_image_ref)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:
@@ -89,8 +87,6 @@ def _unrescue(self, req, id, body):
         instance = common.get_instance(self.compute_api, context, id)
         try:
             self.compute_api.unrescue(context, instance)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:

nova/api/openstack/compute/server_metadata.py

@@ -108,16 +108,10 @@ def _update_instance_metadata(self, context, server_id, metadata,
                                                              server,
                                                              metadata,
                                                              delete)
-
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
-
         except exception.QuotaError as error:
             raise exc.HTTPForbidden(explanation=error.format_message())
-
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
-
         except exception.InstanceInvalidState as state_error:
             common.raise_http_conflict_for_instance_invalid_state(state_error,
                     'update metadata', server_id)
@@ -150,13 +144,8 @@ def delete(self, req, server_id, id):
         server = common.get_instance(self.compute_api, context, server_id)
         try:
             self.compute_api.delete_instance_metadata(context, server, id)
-
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
-
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
-
         except exception.InstanceInvalidState as state_error:
             common.raise_http_conflict_for_instance_invalid_state(state_error,
                     'delete metadata', server_id)

nova/api/openstack/compute/servers.py

@@ -852,8 +852,6 @@ def _action_confirm_resize(self, req, id, body):
         instance = self._get_server(context, req, id)
         try:
             self.compute_api.confirm_resize(context, instance)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.MigrationNotFound:
             msg = _("Instance has not been resized.")
             raise exc.HTTPBadRequest(explanation=msg)
@@ -872,8 +870,6 @@ def _action_revert_resize(self, req, id, body):
         instance = self._get_server(context, req, id)
         try:
             self.compute_api.revert_resize(context, instance)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.MigrationNotFound:
             msg = _("Instance has not been resized.")
             raise exc.HTTPBadRequest(explanation=msg)
@@ -927,8 +923,6 @@ def _resize(self, req, instance_id, flavor_id, **kwargs):
 
         try:
             self.compute_api.resize(context, instance, flavor_id, **kwargs)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.QuotaError as error:
             raise exc.HTTPForbidden(
                 explanation=error.format_message())
@@ -967,8 +961,6 @@ def delete(self, req, id):
         except exception.InstanceNotFound:
             msg = _("Instance could not be found")
             raise exc.HTTPNotFound(explanation=msg)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except (exception.InstanceIsLocked,
                 exception.AllocationDeleteFailed) as e:
             raise exc.HTTPConflict(explanation=e.format_message())
@@ -1093,8 +1085,6 @@ def _action_rebuild(self, req, id, body):
         except exception.InstanceNotFound:
             msg = _("Instance could not be found")
             raise exc.HTTPNotFound(explanation=msg)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.ImageNotFound:
             msg = _("Cannot find image for rebuild")
             raise exc.HTTPBadRequest(explanation=msg)
@@ -1188,8 +1178,6 @@ def _action_create_image(self, req, id, body):
                                                   instance,
                                                   image_name,
                                                   extra_properties=metadata)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:
             common.raise_http_conflict_for_instance_invalid_state(state_error,
                         'createImage', id)
@@ -1262,8 +1250,6 @@ def _start_server(self, req, id, body):
             self.compute_api.start(context, instance)
         except (exception.InstanceNotReady, exception.InstanceIsLocked) as e:
             raise webob.exc.HTTPConflict(explanation=e.format_message())
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:
             common.raise_http_conflict_for_instance_invalid_state(state_error,
                 'start', id)
@@ -1282,8 +1268,6 @@ def _stop_server(self, req, id, body):
             self.compute_api.stop(context, instance)
         except (exception.InstanceNotReady, exception.InstanceIsLocked) as e:
             raise webob.exc.HTTPConflict(explanation=e.format_message())
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:
             common.raise_http_conflict_for_instance_invalid_state(state_error,
                 'stop', id)

nova/api/openstack/compute/shelve.py

@@ -45,8 +45,6 @@ def _shelve(self, req, id, body):
                             'project_id': instance.project_id})
         try:
             self.compute_api.shelve(context, instance)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:
@@ -64,8 +62,6 @@ def _shelve_offload(self, req, id, body):
         instance = common.get_instance(self.compute_api, context, id)
         try:
             self.compute_api.shelve_offload(context, instance)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:
@@ -98,8 +94,6 @@ def _unshelve(self, req, id, body):
 
         try:
             self.compute_api.unshelve(context, instance)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:

nova/api/openstack/compute/suspend_server.py

@@ -38,8 +38,6 @@ def _suspend(self, req, id, body):
                         target={'user_id': server.user_id,
                                 'project_id': server.project_id})
             self.compute_api.suspend(context, server)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:
@@ -56,8 +54,6 @@ def _resume(self, req, id, body):
         server = common.get_instance(self.compute_api, context, id)
         try:
             self.compute_api.resume(context, server)
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InstanceIsLocked as e:
             raise exc.HTTPConflict(explanation=e.format_message())
         except exception.InstanceInvalidState as state_error:

nova/api/openstack/compute/volumes.py

@@ -335,8 +335,7 @@ def create(self, req, server_id, body):
             device = self.compute_api.attach_volume(
                 context, instance, volume_id, device, tag=tag,
                 supports_multiattach=supports_multiattach)
-        except (exception.InstanceUnknownCell,
-                exception.VolumeNotFound) as e:
+        except exception.VolumeNotFound as e:
             raise exc.HTTPNotFound(explanation=e.format_message())
         except (exception.InstanceIsLocked,
                 exception.DevicePathInUse,
@@ -448,8 +447,6 @@ def delete(self, req, server_id, id):
             self.compute_api.detach_volume(context, instance, volume)
         except exception.InvalidVolume as e:
             raise exc.HTTPBadRequest(explanation=e.format_message())
-        except exception.InstanceUnknownCell as e:
-            raise exc.HTTPNotFound(explanation=e.format_message())
         except exception.InvalidInput as e:
             raise exc.HTTPBadRequest(explanation=e.format_message())
         except exception.InstanceIsLocked as e:

nova/tests/unit/api/openstack/compute/test_deferred_delete.py

@@ -104,19 +104,6 @@ def test_force_delete_instance_notfound(self, mock_force_delete, mock_get):
         self.assertIn('Instance fake_uuid could not be found',
                       ex.explanation)
 
-    @mock.patch.object(compute_api.API, 'get')
-    @mock.patch.object(compute_api.API, 'force_delete',
-                side_effect=exception.InstanceUnknownCell(
-                    instance_uuid='fake_uuid'))
-    def test_force_delete_instance_cellunknown(self, mock_force_delete,
-                                               mock_get):
-        req = fakes.HTTPRequest.blank('/v2/fake/servers/fake_uuid/action')
-        ex = self.assertRaises(webob.exc.HTTPNotFound,
-                            self.extension._force_delete,
-                            req, 'fake_uuid', '')
-        self.assertIn('Cell is not known for instance fake_uuid',
-                      ex.explanation)
-
     @mock.patch.object(compute_api.API, 'get')
     @mock.patch.object(compute_api.API, 'restore')
     def test_restore(self, mock_restore, mock_get):