)]}'
{"nova/compute/manager.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"700b0ae399c1815c9716b3b2bcf538d00b9ff530","unresolved":false,"context_lines":[{"line_number":1351,"context_line":"        \"\"\"Rescue an instance on this host.\"\"\""},{"line_number":1352,"context_line":"        context \u003d context.elevated()"},{"line_number":1353,"context_line":"        if not instance:"},{"line_number":1354,"context_line":"            instance \u003d self.db.instance_get_by_uuid(context, instance_uuid)"},{"line_number":1355,"context_line":""},{"line_number":1356,"context_line":"        LOG.audit(_(\u0027Unrescuing\u0027), context\u003dcontext, instance\u003dinstance)"},{"line_number":1357,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2F7vM%3D","line":1354,"updated":"2012-07-31 20:32:16.000000000","message":"Hmm, maybe I\u0027d know this if I was paying attention to previous patches, but what happens if neither instance nor instance_uuid is provided? Should we catch this and raise something meaningful instead of whatever happens when the core instance_get_by_uuid() function gets None?","commit_id":"6a6fe2f6bf736583d9e2cde0733c6568877b7979"},{"author":{"_account_id":1561,"name":"Russell Bryant","email":"rbryant@redhat.com","username":"russellb"},"change_message_id":"b845e26ebd120d1db7bb167506268ba0929f181c","unresolved":false,"context_lines":[{"line_number":1351,"context_line":"        \"\"\"Rescue an instance on this host.\"\"\""},{"line_number":1352,"context_line":"        context \u003d context.elevated()"},{"line_number":1353,"context_line":"        if not instance:"},{"line_number":1354,"context_line":"            instance \u003d self.db.instance_get_by_uuid(context, instance_uuid)"},{"line_number":1355,"context_line":""},{"line_number":1356,"context_line":"        LOG.audit(_(\u0027Unrescuing\u0027), context\u003dcontext, instance\u003dinstance)"},{"line_number":1357,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALH%2F%2F7uo%3D","line":1354,"in_reply_to":"AAAALH%2F%2F7vM%3D","updated":"2012-07-31 20:39:36.000000000","message":"It\u0027s going to raise exception.InstanceNotFound(), which seems pretty reasonable.  The code won\u0027t continue past instance_get_by_uuid().","commit_id":"6a6fe2f6bf736583d9e2cde0733c6568877b7979"}]}