diff --git a/ironic/api/controllers/v1/chassis.py b/ironic/api/controllers/v1/chassis.py index ea8bba7d4e..da8c6e65a1 100644 --- a/ironic/api/controllers/v1/chassis.py +++ b/ironic/api/controllers/v1/chassis.py @@ -32,10 +32,6 @@ from ironic.api.controllers.v1 import types from ironic.api.controllers.v1 import utils as api_utils from ironic.common import exception from ironic import objects -from ironic.openstack.common import excutils -from ironic.openstack.common import log - -LOG = log.getLogger(__name__) class ChassisPatchType(types.JsonPatchType): @@ -210,11 +206,8 @@ class ChassisController(rest.RestController): :param chassis: a chassis within the request body. """ - try: - new_chassis = pecan.request.dbapi.create_chassis(chassis.as_dict()) - except Exception as e: - with excutils.save_and_reraise_exception(): - LOG.exception(e) + new_chassis = pecan.request.dbapi.create_chassis(chassis.as_dict()) + return Chassis.convert_with_links(new_chassis) @wsme.validate(types.uuid, [ChassisPatchType]) diff --git a/ironic/api/controllers/v1/driver.py b/ironic/api/controllers/v1/driver.py index e9f0163463..93f8626b26 100644 --- a/ironic/api/controllers/v1/driver.py +++ b/ironic/api/controllers/v1/driver.py @@ -23,9 +23,6 @@ import wsmeext.pecan as wsme_pecan from ironic.api.controllers.v1 import base from ironic.api.controllers.v1 import link from ironic.common import exception -from ironic.openstack.common import log - -LOG = log.getLogger(__name__) class Driver(base.APIBase): diff --git a/ironic/api/controllers/v1/node.py b/ironic/api/controllers/v1/node.py index f650a19d68..036a0d63b9 100644 --- a/ironic/api/controllers/v1/node.py +++ b/ironic/api/controllers/v1/node.py @@ -34,16 +34,12 @@ from ironic.common import exception from ironic.common import states as ir_states from ironic.common import utils from ironic import objects -from ironic.openstack.common import excutils -from ironic.openstack.common import log CONF = cfg.CONF CONF.import_opt('heartbeat_timeout', 'ironic.conductor.manager', group='conductor') -LOG = log.getLogger(__name__) - class NodePatchType(types.JsonPatchType): @@ -205,7 +201,6 @@ class NodeStatesController(rest.RestController): if target == rpc_node.provision_state: msg = (_("Node %(node)s is already in the '%(state)s' state.") % {'node': rpc_node['uuid'], 'state': target}) - LOG.exception(msg) raise wsme.exc.ClientSideError(msg, status_code=400) if target == ir_states.ACTIVE: @@ -219,7 +214,6 @@ class NodeStatesController(rest.RestController): if processing: msg = (_('Node %s is already being provisioned or decommissioned.') % rpc_node.uuid) - LOG.exception(msg) raise wsme.exc.ClientSideError(msg, status_code=409) # Conflict # Note that there is a race condition. The node state(s) could change @@ -624,11 +618,8 @@ class NodesController(rest.RestController): e.code = 400 raise e - try: - new_node = pecan.request.dbapi.create_node(node.as_dict()) - except Exception as e: - with excutils.save_and_reraise_exception(): - LOG.exception(e) + new_node = pecan.request.dbapi.create_node(node.as_dict()) + return Node.convert_with_links(new_node) @wsme.validate(types.uuid, [NodePatchType]) @@ -675,12 +666,8 @@ class NodesController(rest.RestController): e.code = 400 raise e - try: - new_node = pecan.request.rpcapi.update_node( - pecan.request.context, rpc_node, topic) - except Exception as e: - with excutils.save_and_reraise_exception(): - LOG.exception(e) + new_node = pecan.request.rpcapi.update_node( + pecan.request.context, rpc_node, topic) return Node.convert_with_links(new_node) diff --git a/ironic/api/controllers/v1/port.py b/ironic/api/controllers/v1/port.py index 958a6f9391..6e43bab879 100644 --- a/ironic/api/controllers/v1/port.py +++ b/ironic/api/controllers/v1/port.py @@ -30,10 +30,6 @@ from ironic.api.controllers.v1 import types from ironic.api.controllers.v1 import utils as api_utils from ironic.common import exception from ironic import objects -from ironic.openstack.common import excutils -from ironic.openstack.common import log - -LOG = log.getLogger(__name__) class PortPatchType(types.JsonPatchType): @@ -252,11 +248,8 @@ class PortsController(rest.RestController): if self._from_nodes: raise exception.OperationNotPermitted - try: - new_port = pecan.request.dbapi.create_port(port.as_dict()) - except Exception as e: - with excutils.save_and_reraise_exception(): - LOG.exception(e) + new_port = pecan.request.dbapi.create_port(port.as_dict()) + return Port.convert_with_links(new_port) @wsme.validate(types.uuid, [PortPatchType])