)]}'
{"magnumclient/v1/pods_shell.py":[{"author":{"_account_id":11536,"name":"hongbin","email":"hongbin034@gmail.com","username":"hongbin"},"change_message_id":"6c755e12001f281406a2e0dffb26e7f74f1acef3","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    \"\"\"Print a list of registered pods.\"\"\""},{"line_number":30,"context_line":"    bay \u003d cs.bays.get(args.bay)"},{"line_number":31,"context_line":"    if bay.status not in [\u0027CREATE_COMPLETE\u0027, \u0027UPDATE_COMPLETE\u0027]:"},{"line_number":32,"context_line":"        print(\u0027Bay status for %s is: %s. We can not list pods in there\u0027"},{"line_number":33,"context_line":"              \u0027 until the status is CREATE_COMPLETE or UPDATE_COMPLETE.\u0027 %"},{"line_number":34,"context_line":"              (bay.uuid, bay.status))"},{"line_number":35,"context_line":"        return"},{"line_number":36,"context_line":"    pods \u003d cs.pods.list(args.bay)"},{"line_number":37,"context_line":"    columns \u003d (\u0027uuid\u0027, \u0027name\u0027)"},{"line_number":38,"context_line":"    utils.print_list(pods, columns,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa69d971_ab14719f","line":35,"range":{"start_line":32,"start_character":8,"end_line":35,"end_character":14},"updated":"2016-01-11 21:19:24.000000000","message":"It should raise an exception instead of returning, so that the main method will catch the exception and do the corresponding processing. Check here https://github.com/openstack/python-magnumclient/blob/master/magnumclient/shell.py#L588 .","commit_id":"4d7264a9fb5c5d176b90c4a711414abf19550e83"},{"author":{"_account_id":10263,"name":"Vilobh Meshram","email":"vilobhmm@yahoo-inc.com","username":"vilobhmm"},"change_message_id":"e9819efe4b0459c054af8b24ff7f70aaaef1db0e","unresolved":false,"context_lines":[{"line_number":29,"context_line":"    \"\"\"Print a list of registered pods.\"\"\""},{"line_number":30,"context_line":"    bay \u003d cs.bays.get(args.bay)"},{"line_number":31,"context_line":"    if bay.status not in [\u0027CREATE_COMPLETE\u0027, \u0027UPDATE_COMPLETE\u0027]:"},{"line_number":32,"context_line":"        print(\u0027Bay status for %s is: %s. We can not list pods in there\u0027"},{"line_number":33,"context_line":"              \u0027 until the status is CREATE_COMPLETE or UPDATE_COMPLETE.\u0027 %"},{"line_number":34,"context_line":"              (bay.uuid, bay.status))"},{"line_number":35,"context_line":"        return"},{"line_number":36,"context_line":"    pods \u003d cs.pods.list(args.bay)"},{"line_number":37,"context_line":"    columns \u003d (\u0027uuid\u0027, \u0027name\u0027)"},{"line_number":38,"context_line":"    utils.print_list(pods, columns,"}],"source_content_type":"text/x-python","patch_set":2,"id":"da6ed579_f2dc591a","line":35,"range":{"start_line":32,"start_character":8,"end_line":35,"end_character":14},"in_reply_to":"fa69d971_ab14719f","updated":"2016-01-12 02:51:03.000000000","message":"+1.\n\nAlso please make sure you dump the bay.uuid and bay.status in exception as well.","commit_id":"4d7264a9fb5c5d176b90c4a711414abf19550e83"}]}