Comment 9 for bug 1191730

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to keystone (master)

Reviewed: https://review.openstack.org/33614
Committed: http://github.com/openstack/keystone/commit/f79ccf452b7a0cbf51c63adf259f46896586dc16
Submitter: Jenkins
Branch: master

commit f79ccf452b7a0cbf51c63adf259f46896586dc16
Author: Sergey Vilgelm <email address hidden>
Date: Mon Jun 24 13:41:07 2013 +0400

    Do not raise NEW exceptions

    Raising NEW exception is bad practice, because we lose TraceBack.
    So all places like:

    except SomeException as e:
        raise e

    should be replaced by

    except SomeException:
        raise

    If we are doing some other actions before reraising we should
    store information about exception then do all actions and then
    reraise it. This is caused by eventlet bug. It lost information
    about exception if it switch threads.

    fixes bug 1191730

    Change-Id: I8dffc36ba5780911dd57d7161d218d0324af60b3