Brian: Made the fix you requested. Thanks for the exception refactor.
On Apr 28, 2011, at 9:45 AM, Brian Waldon wrote:
> Review: Needs Fixing
> 97: Can you replace exception.Error with an exception that makes more sense? nova/exception.py has been refactored with a ton of specific exception classes and there may be one that fits better in this case. If not, it would be great if you would add one for this case.
> --
> https://code.launchpad.net/~vishvananda/nova/has_role_cache/+merge/56449
> You are the owner of lp:~vishvananda/nova/has_role_cache.
Brian: Made the fix you requested. Thanks for the exception refactor.
On Apr 28, 2011, at 9:45 AM, Brian Waldon wrote:
> Review: Needs Fixing /code.launchpad .net/~vishvanan da/nova/ has_role_ cache/+ merge/56449
> 97: Can you replace exception.Error with an exception that makes more sense? nova/exception.py has been refactored with a ton of specific exception classes and there may be one that fits better in this case. If not, it would be great if you would add one for this case.
> --
> https:/
> You are the owner of lp:~vishvananda/nova/has_role_cache.