)]}'
{"novaclient/v1_1/shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"830d4dcf510be46e8cc643192e8552479a7fe1a7","unresolved":false,"context_lines":[{"line_number":1019,"context_line":"        try:"},{"line_number":1020,"context_line":"            _find_server(cs, server).delete()"},{"line_number":1021,"context_line":"        except Exception, e:"},{"line_number":1022,"context_line":"            print e"},{"line_number":1023,"context_line":""},{"line_number":1024,"context_line":""},{"line_number":1025,"context_line":"def _find_server(cs, server):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALn%2F%2F1X4%3D","line":1022,"updated":"2012-11-06 16:44:40.000000000","message":"Hmmm…what if the exception raised is an authentication error?  I think you need to try to only catch \"not found\" equivalent exceptions in this loop.","commit_id":"33b2830e9c81f87fb37e51eca65f6e98143d1e06"},{"author":{"_account_id":4340,"name":"Sulochan Acharya","email":"sulochan@gmail.com","username":"sulochan-acharya"},"change_message_id":"62efd85e4d7ba3f07e77dc1148661f4dd0048c7b","unresolved":false,"context_lines":[{"line_number":1019,"context_line":"        try:"},{"line_number":1020,"context_line":"            _find_server(cs, server).delete()"},{"line_number":1021,"context_line":"        except Exception, e:"},{"line_number":1022,"context_line":"            print e"},{"line_number":1023,"context_line":""},{"line_number":1024,"context_line":""},{"line_number":1025,"context_line":"def _find_server(cs, server):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALn%2F%2Fw3w%3D","line":1022,"in_reply_to":"AAAALn%2F%2F1X4%3D","updated":"2012-11-07 21:42:45.000000000","message":"I am not sure if I understand this concern correctly...but auth error would be caught before it gets to a subcommand..so it would have raised before it gets here ? Let me know if I am missing anything. thanks Kevin.","commit_id":"33b2830e9c81f87fb37e51eca65f6e98143d1e06"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"b7094c81057d2bf486e0cb43b2ce5a1af331c5cb","unresolved":false,"context_lines":[{"line_number":1019,"context_line":"        try:"},{"line_number":1020,"context_line":"            _find_server(cs, server).delete()"},{"line_number":1021,"context_line":"        except Exception, e:"},{"line_number":1022,"context_line":"            print e"},{"line_number":1023,"context_line":""},{"line_number":1024,"context_line":""},{"line_number":1025,"context_line":"def _find_server(cs, server):"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAALn%2F%2Fw20%3D","line":1022,"in_reply_to":"AAAALn%2F%2Fw3w%3D","updated":"2012-11-07 21:47:32.000000000","message":"No, I can see a situation where an auth error could get raised during the loop, but it\u0027s probably a contrived example.  Honestly, after further reflection, I can\u0027t see any reason to delay this patch any further, so…","commit_id":"33b2830e9c81f87fb37e51eca65f6e98143d1e06"}]}