)]}'
{"cinder/scheduler/rpcapi.py":[{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"7f865cad30d7d155490da0d7bef3e61191eab854","unresolved":false,"context_lines":[{"line_number":75,"context_line":"            \u0027filter_properties_list\u0027: filter_properties_list,"},{"line_number":76,"context_line":"        }"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        cctxt.cast(ctxt, \u0027create_consistencygroup\u0027, **msg_args)"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    def create_group(self, ctxt, group, group_spec\u003dNone,"},{"line_number":81,"context_line":"                     request_spec_list\u003dNone, group_filter_properties\u003dNone,"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_c0648bba","line":78,"updated":"2016-11-17 19:00:58.000000000","message":"Why do we need to remove these lines?","commit_id":"7802645cc8c2a05e8eba21698e7269442fafec02"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"290a49d9e4b46568f3b62622597c3157feba21eb","unresolved":false,"context_lines":[{"line_number":75,"context_line":"            \u0027filter_properties_list\u0027: filter_properties_list,"},{"line_number":76,"context_line":"        }"},{"line_number":77,"context_line":""},{"line_number":78,"context_line":"        cctxt.cast(ctxt, \u0027create_consistencygroup\u0027, **msg_args)"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    def create_group(self, ctxt, group, group_spec\u003dNone,"},{"line_number":81,"context_line":"                     request_spec_list\u003dNone, group_filter_properties\u003dNone,"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_4b5797c9","line":78,"in_reply_to":"7a77a97e_c0648bba","updated":"2016-11-18 14:51:03.000000000","message":"Oh, that\u0027s for consistency - casts doesn\u0027t return anything. I can revert these changes if you prefer.","commit_id":"7802645cc8c2a05e8eba21698e7269442fafec02"}],"cinder/test.py":[{"author":{"_account_id":23602,"name":"Michael Dovgal","email":"dovgalmichael@gmail.com","username":"mdovgal"},"change_message_id":"85e9fb897abb38eba6f7855988efb04c01db1c4f","unresolved":false,"context_lines":[{"line_number":499,"context_line":"                return retval"},{"line_number":500,"context_line":""},{"line_number":501,"context_line":"        # Enable mocks that will check everything and run RPC method."},{"line_number":502,"context_line":"        with mock.patch.object(rpcapi.client, \"prepare\","},{"line_number":503,"context_line":"                               side_effect\u003d_fake_prepare_method):"},{"line_number":504,"context_line":"            with mock.patch.object(rpcapi.client, rpc_method,"},{"line_number":505,"context_line":"                                   side_effect\u003d_fake_rpc_method):"},{"line_number":506,"context_line":"                real_retval \u003d getattr(rpcapi, method)(self.context, **kwargs)"},{"line_number":507,"context_line":"                self.assertEqual(expected_retval, real_retval)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_f3540eff","line":505,"range":{"start_line":502,"start_character":0,"end_line":505,"end_character":65},"updated":"2016-11-17 15:48:23.000000000","message":"Maybe we can use only one with? or you think your variant is better?\n\nwith mock.patch.object(rpcapi.client, \"prepare\",\n                       side_effect\u003d_fake_prepare_method), \\\n     mock.patch.object(rpcapi.client, rpc_method,\n                       side_effect\u003d_fake_rpc_method)","commit_id":"7802645cc8c2a05e8eba21698e7269442fafec02"},{"author":{"_account_id":11600,"name":"Michał Dulko","email":"michal.dulko@gmail.com","username":"dulek"},"change_message_id":"b3fb959ae64efc52782f3566d76ee4a423ca5f04","unresolved":false,"context_lines":[{"line_number":499,"context_line":"                return retval"},{"line_number":500,"context_line":""},{"line_number":501,"context_line":"        # Enable mocks that will check everything and run RPC method."},{"line_number":502,"context_line":"        with mock.patch.object(rpcapi.client, \"prepare\","},{"line_number":503,"context_line":"                               side_effect\u003d_fake_prepare_method):"},{"line_number":504,"context_line":"            with mock.patch.object(rpcapi.client, rpc_method,"},{"line_number":505,"context_line":"                                   side_effect\u003d_fake_rpc_method):"},{"line_number":506,"context_line":"                real_retval \u003d getattr(rpcapi, method)(self.context, **kwargs)"},{"line_number":507,"context_line":"                self.assertEqual(expected_retval, real_retval)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a77a97e_51e5b735","line":505,"range":{"start_line":502,"start_character":0,"end_line":505,"end_character":65},"in_reply_to":"7a77a97e_f3540eff","updated":"2016-11-17 17:09:26.000000000","message":"I wanted to avoid using \\ to wrap lines and remember some problems with using parentheses to wrap lines, so I\u0027ve went this way.","commit_id":"7802645cc8c2a05e8eba21698e7269442fafec02"}]}