)]}'
{"novaclient/tests/v1_1/fakes.py":[{"author":{"_account_id":5652,"name":"David Ripton","email":"dripton@redhat.com","username":"dripton"},"change_message_id":"b70328cf4a3ddf7dce5d663ccf88721385853c57","unresolved":false,"context_lines":[{"line_number":1707,"context_line":"        return (202, {}, {})"},{"line_number":1708,"context_line":""},{"line_number":1709,"context_line":"    def post_servers_uuid2_action(self, **kw):"},{"line_number":1710,"context_line":"            return (202, {}, {})"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2FdnE%3D","line":1710,"updated":"2013-06-03 13:02:50.000000000","message":"Weird indentation here, at least in Gerrit.","commit_id":"413fa1d1e329b41d1fabe07cb134ac408987a517"}],"novaclient/tests/v1_1/test_shell.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a7ad6744d82b5972c5d2e90ca9ef54dd9eb25f2a","unresolved":false,"context_lines":[{"line_number":943,"context_line":"            \u0027POST\u0027, \u0027/os-hosts/sample-host/action\u0027, {\u0027reboot\u0027: None})"},{"line_number":944,"context_line":""},{"line_number":945,"context_line":"    def test_host_evacuate(self):"},{"line_number":946,"context_line":"        self.run_command(\u0027host-evacuate hyper target_hyper\u0027)"},{"line_number":947,"context_line":"        self.assert_called(\u0027GET\u0027, \u0027/os-hypervisors/hyper/servers\u0027, None, 0)"},{"line_number":948,"context_line":"        self.assert_called(\u0027POST\u0027, \u0027/servers/uuid1/action\u0027,"},{"line_number":949,"context_line":"                           {\u0027evacuate\u0027: {\u0027host\u0027: \u0027target_hyper\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPH%2F%2FdRQ%3D","line":946,"updated":"2013-06-03 15:32:03.000000000","message":"Would be nice to see a test for the other argument, to make sure True gets passed through.","commit_id":"aa2e721c718df91ab8e4b3f260f5672e035174f2"}],"novaclient/v1_1/contrib/host_evacuate.py":[{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"65bc69d2de02ee6cd6305bb3dc0a064c2f99f017","unresolved":false,"context_lines":[{"line_number":32,"context_line":"                                      args.on_shared_storage)[1]"},{"line_number":33,"context_line":"            print res"},{"line_number":34,"context_line":"        except Exception as e:"},{"line_number":35,"context_line":"            print \"Error while evacuating instance: %s\" % e"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAPH%2F%2FmUA%3D","line":35,"updated":"2013-05-29 16:18:02.000000000","message":"Instead of printing as you go, you should gather all of this data and print a nice table with success/error information for each instance.  Also, what does this data look like?","commit_id":"cb0e2a0f7db3a05c6900e2299736ff17d73c2d0c"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a7ad6744d82b5972c5d2e90ca9ef54dd9eb25f2a","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"@utils.arg(\u0027host\u0027, metavar\u003d\u0027\u003chost\u003e\u0027, help\u003d\u0027Name of host.\u0027)"},{"line_number":25,"context_line":"@utils.arg(\u0027target_host\u0027, metavar\u003d\u0027\u003ctarget_host\u003e\u0027, help\u003d\u0027Name of target host.\u0027)"},{"line_number":26,"context_line":"@utils.arg(\u0027--on-shared-storage\u0027,"},{"line_number":27,"context_line":"           dest\u003d\u0027on_shared_storage\u0027,"},{"line_number":28,"context_line":"           action\u003d\"store_true\","}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPH%2F%2FdSQ%3D","line":25,"updated":"2013-06-03 15:32:03.000000000","message":"I wondered about this before, it might be good to make this optional now, to save breaking the compatibility later. But I guess we have not let the nova cli change to make that callable through yet.","commit_id":"aa2e721c718df91ab8e4b3f260f5672e035174f2"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"7678cc874862fe2718930b406f557a80e9aefa31","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":""},{"line_number":24,"context_line":"@utils.arg(\u0027host\u0027, metavar\u003d\u0027\u003chost\u003e\u0027, help\u003d\u0027Name of host.\u0027)"},{"line_number":25,"context_line":"@utils.arg(\u0027target_host\u0027, metavar\u003d\u0027\u003ctarget_host\u003e\u0027, help\u003d\u0027Name of target host.\u0027)"},{"line_number":26,"context_line":"@utils.arg(\u0027--on-shared-storage\u0027,"},{"line_number":27,"context_line":"           dest\u003d\u0027on_shared_storage\u0027,"},{"line_number":28,"context_line":"           action\u003d\"store_true\","}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPH%2F%2Fc9s%3D","line":25,"in_reply_to":"AAAAPH%2F%2FdSQ%3D","updated":"2013-06-03 16:37:04.000000000","message":"I agree on making this optional.  Maybe for now you can just check if it wasn\u0027t provided and raise an exception explaining that this currently isn\u0027t supported.","commit_id":"aa2e721c718df91ab8e4b3f260f5672e035174f2"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"a7ad6744d82b5972c5d2e90ca9ef54dd9eb25f2a","unresolved":false,"context_lines":[{"line_number":47,"context_line":"                                              \"error_message\": error_message}))"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    utils.print_list(response,"},{"line_number":50,"context_line":"                     [\"Instance ID\", \"Rebuild Accepted\", \"Error Message\"])"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAPH%2F%2FdQE%3D","line":50,"updated":"2013-06-03 15:32:03.000000000","message":"Maybe \"Evacuate Accepted\" would match the host_evacuate method better, rather than \"Rebuild Accepted\".\n\nAlso, should it not be called \"Server UUID\" to match the other API calls?","commit_id":"aa2e721c718df91ab8e4b3f260f5672e035174f2"}]}