)]}'
{"nova/api/openstack/compute/contrib/floating_ips.py":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"6c13c761158884978065527d29dc091bab0d4649","unresolved":false,"context_lines":[{"line_number":296,"context_line":"        else:"},{"line_number":297,"context_line":"            msg \u003d _(\"Floating ip %(address)s is not associated with an \""},{"line_number":298,"context_line":"                    \"instance.\") % locals()"},{"line_number":299,"context_line":"            raise webob.exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":""},{"line_number":302,"context_line":"class Floating_ips(extensions.ExtensionDescriptor):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Bwls%3D","line":299,"updated":"2013-02-19 20:38:41.000000000","message":"Won\u0027t this cover other errors to?   such as if id isn\u0027t a proper uuid?","commit_id":"ea39e56f4197394feacb4933c69d46dbdb830977"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"698120f4fea2e3cb8307e380bcaa71e227c882fb","unresolved":false,"context_lines":[{"line_number":296,"context_line":"        else:"},{"line_number":297,"context_line":"            msg \u003d _(\"Floating ip %(address)s is not associated with an \""},{"line_number":298,"context_line":"                    \"instance.\") % locals()"},{"line_number":299,"context_line":"            raise webob.exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":""},{"line_number":302,"context_line":"class Floating_ips(extensions.ExtensionDescriptor):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Bn7U%3D","line":299,"in_reply_to":"AAAAM3%2F%2Bqjg%3D","updated":"2013-02-25 14:29:10.000000000","message":"That seems better to me.  Including the \u0027id\u0027 passed in is helpful.  Not quite as helpful as splitting the checks and having multiple exception messages, but definitely better.","commit_id":"ea39e56f4197394feacb4933c69d46dbdb830977"},{"author":{"_account_id":4468,"name":"Wangpan","email":"hzwangpan@corp.netease.com","username":"hzwangpan"},"change_message_id":"f36df3c3f476c987f453bcea02d64b5ab0236046","unresolved":false,"context_lines":[{"line_number":296,"context_line":"        else:"},{"line_number":297,"context_line":"            msg \u003d _(\"Floating ip %(address)s is not associated with an \""},{"line_number":298,"context_line":"                    \"instance.\") % locals()"},{"line_number":299,"context_line":"            raise webob.exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":""},{"line_number":302,"context_line":"class Floating_ips(extensions.ExtensionDescriptor):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Bqjg%3D","line":299,"in_reply_to":"AAAAM3%2F%2Bu10%3D","updated":"2013-02-22 09:32:44.000000000","message":"how about this: \"Floating ip %(address)s is not associated with instance %(id)s\" % locals() ?","commit_id":"ea39e56f4197394feacb4933c69d46dbdb830977"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"4cf86426988b17b5851e735971b7bbf9d839086e","unresolved":false,"context_lines":[{"line_number":296,"context_line":"        else:"},{"line_number":297,"context_line":"            msg \u003d _(\"Floating ip %(address)s is not associated with an \""},{"line_number":298,"context_line":"                    \"instance.\") % locals()"},{"line_number":299,"context_line":"            raise webob.exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":""},{"line_number":302,"context_line":"class Floating_ips(extensions.ExtensionDescriptor):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAM3%2F%2Bu10%3D","line":299,"in_reply_to":"AAAAM3%2F%2Bwls%3D","updated":"2013-02-20 14:06:39.000000000","message":"+1.  There are four different types of checking occurring which could use a different message.  Or a more generic message that encompasses them all.","commit_id":"ea39e56f4197394feacb4933c69d46dbdb830977"},{"author":{"_account_id":5174,"name":"Mauro Sergio Martins Rodrigues","email":"maurosr@linux.vnet.ibm.com","username":"maurosr"},"change_message_id":"0c0224f00f07e735601e22ab76c5c6fcce99257b","unresolved":false,"context_lines":[{"line_number":296,"context_line":"        else:"},{"line_number":297,"context_line":"            msg \u003d _(\"Floating ip %(address)s is not associated with instance \""},{"line_number":298,"context_line":"                    \"%(id)s.\") % locals()"},{"line_number":299,"context_line":"            raise webob.exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":""},{"line_number":302,"context_line":"class Floating_ips(extensions.ExtensionDescriptor):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAM3%2F%2BjtI%3D","line":299,"updated":"2013-02-27 14:15:42.000000000","message":"it\u0027s not a bad request, an unprocessable entity sounds more reasonable to me... http://restpatterns.org/HTTP_Status_Codes/422_-_Unprocessable_Entity","commit_id":"5c55c615e150800786da38e96937f0edbad1a284"},{"author":{"_account_id":4468,"name":"Wangpan","email":"hzwangpan@corp.netease.com","username":"hzwangpan"},"change_message_id":"2e695605e9980186629c1fe31fff1f8740c2cf3e","unresolved":false,"context_lines":[{"line_number":296,"context_line":"        else:"},{"line_number":297,"context_line":"            msg \u003d _(\"Floating ip %(address)s is not associated with instance \""},{"line_number":298,"context_line":"                    \"%(id)s.\") % locals()"},{"line_number":299,"context_line":"            raise webob.exc.HTTPBadRequest(explanation\u003dmsg)"},{"line_number":300,"context_line":""},{"line_number":301,"context_line":""},{"line_number":302,"context_line":"class Floating_ips(extensions.ExtensionDescriptor):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAANH%2F%2F%2Fsg%3D","line":299,"in_reply_to":"AAAAM3%2F%2BjtI%3D","updated":"2013-02-28 02:03:41.000000000","message":"yes, sounds better, I\u0027ll change to it, thanks!","commit_id":"5c55c615e150800786da38e96937f0edbad1a284"}]}