Improve rpc RemoteError to return original exception args

Bug #920705 reported by Dean Troyer
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Wishlist
Alex Meade

Bug Description

The original exception information returned in an rpc.RemoteError is reduced to strings. We need a way to return the original exception args intact to the calling routines.

Revision history for this message
Brian Waldon (bcwaldon) wrote :

Being able to return enough information to reconstruct the same exception on the other side of an rpc connection would be fantastic.

Changed in nova:
status: New → Triaged
importance: Undecided → Wishlist
Brian Waldon (bcwaldon)
Changed in nova:
assignee: nobody → Brian Waldon (bcwaldon)
Brian Waldon (bcwaldon)
Changed in nova:
status: Triaged → In Progress
Brian Waldon (bcwaldon)
Changed in nova:
assignee: Brian Waldon (bcwaldon) → Alex Meade (alex-meade)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/5749

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

Reviewed: https://review.openstack.org/5749
Committed: http://github.com/openstack/nova/commit/a0150a4d9e751ec222221558dfe89a66b0c118ab
Submitter: Jenkins
Branch: master

commit a0150a4d9e751ec222221558dfe89a66b0c118ab
Author: Alex Meade <email address hidden>
Date: Wed Mar 21 19:13:16 2012 +0000

    Add the serialization of exceptions for RPC calls.

    This change uses json to serialize an exception so that it can be sent
    through RPC calls to be reconstructed on the other side. The traceback is added
    to the exception message. If recreating the exception fails for whatever reason
    then a RemoteError is created containing all of the exception information.

    Adds flag 'allowed_rpc_exception_modules' to prevent dangerous modules from
    being accessed and allowing arbitrary code to be run.

    Fixes bug 920705
    Fixes bug 940500

    Change-Id: Ife3b64b19fe8abbc730184d4ee7d9fcabfd29db3

Changed in nova:
status: In Progress → Fix Committed
Devin Carlen (devcamcar)
Changed in nova:
milestone: none → folsom-1
Thierry Carrez (ttx)
Changed in nova:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: folsom-1 → 2012.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.