Code review comment for lp:~jtran/nova/lp702580

Revision history for this message
Vish Ishaya (vishvananda) wrote :

Yes the issue here is that we are wrapping the underlying exception with a useful error message so that it can be sent to the client. There is handling in the wsgi app for displaying NotFound, although we should perhaps call it ApiNotFound or something.

Vish

On Mar 28, 2011, at 12:08 PM, John Tran wrote:

> I figured it would be good to keep consistent w/ the prior code base. See nova/api/ec2/cloud.py:describe_images.
> --
> https://code.launchpad.net/~jtran/nova/lp702580/+merge/55209
> You are subscribed to branch lp:nova.

« Back to merge proposal