)]}'
{"nova/tests/unit/cells/test_cells_messaging.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8fdc182870100381d4b0353122c1123d57a0d0c1","unresolved":false,"context_lines":[{"line_number":981,"context_line":""},{"line_number":982,"context_line":"        self.mox.StubOutWithMock(self.tgt_db_inst, \u0027actions_get\u0027)"},{"line_number":983,"context_line":"        self.tgt_db_inst.actions_get(self.ctxt,"},{"line_number":984,"context_line":"                                     uuids.instance_1).AndReturn([fake_act])"},{"line_number":985,"context_line":"        self.mox.ReplayAll()"},{"line_number":986,"context_line":""},{"line_number":987,"context_line":"        response \u003d self.src_msg_runner.actions_get(self.ctxt,"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a061dce_ba7d6470","line":984,"range":{"start_line":984,"start_character":43,"end_line":984,"end_character":53},"updated":"2016-04-05 20:46:03.000000000","message":"nit: If the test only has a single instance in it, I\u0027d just use:\n\n uuids.instance","commit_id":"51a0410314fc3268da739defa8deb0ba1627b57d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8115cbc495656fe0c36c4192ce0b57d98aed9535","unresolved":false,"context_lines":[{"line_number":981,"context_line":""},{"line_number":982,"context_line":"        self.mox.StubOutWithMock(self.tgt_db_inst, \u0027actions_get\u0027)"},{"line_number":983,"context_line":"        self.tgt_db_inst.actions_get(self.ctxt,"},{"line_number":984,"context_line":"                                     uuids.instance).AndReturn([fake_act])"},{"line_number":985,"context_line":"        self.mox.ReplayAll()"},{"line_number":986,"context_line":""},{"line_number":987,"context_line":"        response \u003d self.src_msg_runner.actions_get(self.ctxt,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a061dce_0b6c449b","line":984,"range":{"start_line":984,"start_character":37,"end_line":984,"end_character":51},"updated":"2016-04-08 20:48:56.000000000","message":"This should maybe be fake_uuid.","commit_id":"46f8f6ee2fe0a9ad2797b381abe1a1267fd88006"},{"author":{"_account_id":20181,"name":"Srushti Gadadare","email":"srushti391993@gmail.com","username":"srushti"},"change_message_id":"dd870b482b7a5447acc86acd842abeea1e7b2627","unresolved":false,"context_lines":[{"line_number":981,"context_line":""},{"line_number":982,"context_line":"        self.mox.StubOutWithMock(self.tgt_db_inst, \u0027actions_get\u0027)"},{"line_number":983,"context_line":"        self.tgt_db_inst.actions_get(self.ctxt,"},{"line_number":984,"context_line":"                                     uuids.instance).AndReturn([fake_act])"},{"line_number":985,"context_line":"        self.mox.ReplayAll()"},{"line_number":986,"context_line":""},{"line_number":987,"context_line":"        response \u003d self.src_msg_runner.actions_get(self.ctxt,"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a18252c_a8c5283d","line":984,"range":{"start_line":984,"start_character":37,"end_line":984,"end_character":51},"in_reply_to":"7a1b2928_aa03b70c","updated":"2016-04-13 13:00:14.000000000","message":"Hi,\n\nI have made changes to use fake_uuid as per review comments.","commit_id":"46f8f6ee2fe0a9ad2797b381abe1a1267fd88006"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"fa508fab2fac70c497ca2711d10b9dabdb86008f","unresolved":false,"context_lines":[{"line_number":981,"context_line":""},{"line_number":982,"context_line":"        self.mox.StubOutWithMock(self.tgt_db_inst, \u0027actions_get\u0027)"},{"line_number":983,"context_line":"        self.tgt_db_inst.actions_get(self.ctxt,"},{"line_number":984,"context_line":"                                     uuids.instance).AndReturn([fake_act])"},{"line_number":985,"context_line":"        self.mox.ReplayAll()"},{"line_number":986,"context_line":""},{"line_number":987,"context_line":"        response \u003d self.src_msg_runner.actions_get(self.ctxt,"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a1b2928_aa03b70c","line":984,"range":{"start_line":984,"start_character":37,"end_line":984,"end_character":51},"in_reply_to":"9a061dce_0b6c449b","updated":"2016-04-10 23:10:30.000000000","message":"Yeah, FAKE_UUID is defined as a server id on fake_server_actions. So it is not necessary to use uuidsentinel for creating different id.","commit_id":"46f8f6ee2fe0a9ad2797b381abe1a1267fd88006"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8115cbc495656fe0c36c4192ce0b57d98aed9535","unresolved":false,"context_lines":[{"line_number":986,"context_line":""},{"line_number":987,"context_line":"        response \u003d self.src_msg_runner.actions_get(self.ctxt,"},{"line_number":988,"context_line":"                                                   self.tgt_cell_name,"},{"line_number":989,"context_line":"                                                   uuids.instance)"},{"line_number":990,"context_line":"        result \u003d response.value_or_raise()"},{"line_number":991,"context_line":"        self.assertEqual([jsonutils.to_primitive(fake_act)], result)"},{"line_number":992,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a061dce_4b047cc5","line":989,"range":{"start_line":989,"start_character":51,"end_line":989,"end_character":65},"updated":"2016-04-08 20:48:56.000000000","message":"And here.","commit_id":"46f8f6ee2fe0a9ad2797b381abe1a1267fd88006"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8115cbc495656fe0c36c4192ce0b57d98aed9535","unresolved":false,"context_lines":[{"line_number":997,"context_line":""},{"line_number":998,"context_line":"        self.mox.StubOutWithMock(self.tgt_db_inst, \u0027action_get_by_request_id\u0027)"},{"line_number":999,"context_line":"        self.tgt_db_inst.action_get_by_request_id(self.ctxt,"},{"line_number":1000,"context_line":"                uuids.instance, \u0027req-fake\u0027).AndReturn(fake_act)"},{"line_number":1001,"context_line":"        self.mox.ReplayAll()"},{"line_number":1002,"context_line":""},{"line_number":1003,"context_line":"        response \u003d self.src_msg_runner.action_get_by_request_id(self.ctxt,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a061dce_2b0948de","line":1000,"updated":"2016-04-08 20:48:56.000000000","message":"and here","commit_id":"46f8f6ee2fe0a9ad2797b381abe1a1267fd88006"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8115cbc495656fe0c36c4192ce0b57d98aed9535","unresolved":false,"context_lines":[{"line_number":1001,"context_line":"        self.mox.ReplayAll()"},{"line_number":1002,"context_line":""},{"line_number":1003,"context_line":"        response \u003d self.src_msg_runner.action_get_by_request_id(self.ctxt,"},{"line_number":1004,"context_line":"                self.tgt_cell_name, uuids.instance, \u0027req-fake\u0027)"},{"line_number":1005,"context_line":"        result \u003d response.value_or_raise()"},{"line_number":1006,"context_line":"        self.assertEqual(jsonutils.to_primitive(fake_act), result)"},{"line_number":1007,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a061dce_6b13400e","line":1004,"updated":"2016-04-08 20:48:56.000000000","message":"And here","commit_id":"46f8f6ee2fe0a9ad2797b381abe1a1267fd88006"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8115cbc495656fe0c36c4192ce0b57d98aed9535","unresolved":false,"context_lines":[{"line_number":1497,"context_line":"    def _test_instance_update_at_top(self, exists\u003dTrue):"},{"line_number":1498,"context_line":"        fake_uuid \u003d fake_server_actions.FAKE_UUID"},{"line_number":1499,"context_line":"        fake_info_cache \u003d objects.InstanceInfoCache("},{"line_number":1500,"context_line":"            instance_uuid\u003duuids.instance)"},{"line_number":1501,"context_line":"        fake_sys_metadata \u003d {\u0027key1\u0027: \u0027value1\u0027,"},{"line_number":1502,"context_line":"                             \u0027key2\u0027: \u0027value2\u0027}"},{"line_number":1503,"context_line":"        fake_attrs \u003d {\u0027uuid\u0027: fake_uuid,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a061dce_4b9abcc6","line":1500,"range":{"start_line":1500,"start_character":26,"end_line":1500,"end_character":40},"updated":"2016-04-08 20:48:56.000000000","message":"Seems this should be fake_uuid which is used on L1503 to create the instance object on L1507.","commit_id":"46f8f6ee2fe0a9ad2797b381abe1a1267fd88006"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8115cbc495656fe0c36c4192ce0b57d98aed9535","unresolved":false,"context_lines":[{"line_number":1519,"context_line":"            if exists:"},{"line_number":1520,"context_line":"                mock_save.side_effect \u003d fake_save"},{"line_number":1521,"context_line":"            else:"},{"line_number":1522,"context_line":"                error \u003d exception.InstanceNotFound(instance_id\u003d\u0027fake_uuid\u0027)"},{"line_number":1523,"context_line":"                mock_save.side_effect \u003d error"},{"line_number":1524,"context_line":""},{"line_number":1525,"context_line":"            self.src_msg_runner.instance_update_at_top(self.ctxt,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a061dce_10aab119","line":1522,"range":{"start_line":1522,"start_character":63,"end_line":1522,"end_character":74},"updated":"2016-04-08 20:48:56.000000000","message":"Use fake_uuid here?","commit_id":"46f8f6ee2fe0a9ad2797b381abe1a1267fd88006"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8115cbc495656fe0c36c4192ce0b57d98aed9535","unresolved":false,"context_lines":[{"line_number":1543,"context_line":"    def test_instance_update_at_top_with_building_state(self):"},{"line_number":1544,"context_line":"        fake_uuid \u003d fake_server_actions.FAKE_UUID"},{"line_number":1545,"context_line":"        fake_info_cache \u003d objects.InstanceInfoCache("},{"line_number":1546,"context_line":"            instance_uuid\u003duuids.instance)"},{"line_number":1547,"context_line":"        fake_sys_metadata \u003d {\u0027key1\u0027: \u0027value1\u0027,"},{"line_number":1548,"context_line":"                             \u0027key2\u0027: \u0027value2\u0027}"},{"line_number":1549,"context_line":"        fake_attrs \u003d {\u0027uuid\u0027: fake_uuid,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a061dce_ebb59031","line":1546,"range":{"start_line":1546,"start_character":26,"end_line":1546,"end_character":40},"updated":"2016-04-08 20:48:56.000000000","message":"I think this is supposed to be fake_uuid.","commit_id":"46f8f6ee2fe0a9ad2797b381abe1a1267fd88006"}]}