Comment 1 for bug 1534186

Revision history for this message
Augustina Ragwitz (auggy) wrote : Re: Improve error messaging for Nova remove-fixed-ip

I confirmed that the remove_fixed_ip_from_instance method in nova/network/neutronv2/api.py throws a fairly generic exception when an error is encountered. Improving the exception information could improve user experience. Additionally I audited the unit test coverage for this method and found exception states are not directly tested (eg, what happens when an invalid IP is provided as the argument to this particular method). I would suggest the fix for this bug is 2 parts, 1) improve exception information for remove_fixed_ip_from_instance and 2) improve unit test coverage for error states.