)]}'
{"nova/tests/unit/objects/test_bandwidth_usage.py":[{"author":{"_account_id":19619,"name":"Michal Pryc","email":"mpryc@redhat.com","username":"migi"},"change_message_id":"4e2cad2ed75c3edd7f3de1553951cce38a90ac28","unresolved":false,"context_lines":[{"line_number":115,"context_line":"                        start_period\u003dself.expected_bw_usage[\u0027start_period\u0027])"},{"line_number":116,"context_line":"        self._compare(self, expected_bw_usage1, bw_usage,"},{"line_number":117,"context_line":"                ignored_fields\u003d[\u0027last_refreshed\u0027, \u0027created_at\u0027])"},{"line_number":118,"context_line":"        bw_usage.create(uuids.instance, \u0027fake_mac1\u0027,"},{"line_number":119,"context_line":"                        100, 200, 12345, 67890,"},{"line_number":120,"context_line":"                        start_period\u003dself.expected_bw_usage[\u0027start_period\u0027])"},{"line_number":121,"context_line":"        self._compare(self, self.expected_bw_usage, bw_usage,"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a122d0e_e5fa94bc","line":118,"range":{"start_line":118,"start_character":38,"end_line":118,"end_character":39},"updated":"2016-04-19 07:08:51.000000000","message":"Shouldn\u0027t here be another instance? In the original test this one was fake_uuid1, why now it\u0027s same uuid?","commit_id":"80a8dbe541162bedb651c33be3f3b99ebec4b511"},{"author":{"_account_id":20181,"name":"Srushti Gadadare","email":"srushti391993@gmail.com","username":"srushti"},"change_message_id":"a146b3cce9c255f4f3f15c5de8a878f601cdc326","unresolved":false,"context_lines":[{"line_number":115,"context_line":"                        start_period\u003dself.expected_bw_usage[\u0027start_period\u0027])"},{"line_number":116,"context_line":"        self._compare(self, expected_bw_usage1, bw_usage,"},{"line_number":117,"context_line":"                ignored_fields\u003d[\u0027last_refreshed\u0027, \u0027created_at\u0027])"},{"line_number":118,"context_line":"        bw_usage.create(uuids.instance, \u0027fake_mac1\u0027,"},{"line_number":119,"context_line":"                        100, 200, 12345, 67890,"},{"line_number":120,"context_line":"                        start_period\u003dself.expected_bw_usage[\u0027start_period\u0027])"},{"line_number":121,"context_line":"        self._compare(self, self.expected_bw_usage, bw_usage,"}],"source_content_type":"text/x-python","patch_set":5,"id":"1a122d0e_2c4f476c","line":118,"range":{"start_line":118,"start_character":38,"end_line":118,"end_character":39},"in_reply_to":"1a122d0e_e5fa94bc","updated":"2016-04-20 06:48:19.000000000","message":"Hi Michal,\n\nThere is no need for this change.\nSame instance uuid needs to be passed otherwise it causes testcase to fail with mismatch error.","commit_id":"80a8dbe541162bedb651c33be3f3b99ebec4b511"}],"nova/tests/unit/objects/test_block_device.py":[{"author":{"_account_id":19619,"name":"Michal Pryc","email":"mpryc@redhat.com","username":"migi"},"change_message_id":"4e2cad2ed75c3edd7f3de1553951cce38a90ac28","unresolved":false,"context_lines":[{"line_number":473,"context_line":"        ]"},{"line_number":474,"context_line":"        get_all_by_inst.return_value \u003d fakes"},{"line_number":475,"context_line":"        bdm_list \u003d objects.BlockDeviceMappingList.get_by_instance_uuid("},{"line_number":476,"context_line":"            self.context, uuids.instance_1)"},{"line_number":477,"context_line":"        self.assertRaises(exception.UndefinedRootBDM, bdm_list.root_bdm)"},{"line_number":478,"context_line":""},{"line_number":479,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1a122d0e_e5cf347d","line":476,"updated":"2016-04-19 07:08:51.000000000","message":"Interesting that here is different instance then in original test see \"fake-instance\" and then it doesn\u0027t really refer to the uuid_X.\n\nAlso ideal would be to name it more meaningful like uuid.bdm_instance_1 so it\u0027s easier in the future to debug if required.","commit_id":"80a8dbe541162bedb651c33be3f3b99ebec4b511"},{"author":{"_account_id":20181,"name":"Srushti Gadadare","email":"srushti391993@gmail.com","username":"srushti"},"change_message_id":"a146b3cce9c255f4f3f15c5de8a878f601cdc326","unresolved":false,"context_lines":[{"line_number":473,"context_line":"        ]"},{"line_number":474,"context_line":"        get_all_by_inst.return_value \u003d fakes"},{"line_number":475,"context_line":"        bdm_list \u003d objects.BlockDeviceMappingList.get_by_instance_uuid("},{"line_number":476,"context_line":"            self.context, uuids.instance_1)"},{"line_number":477,"context_line":"        self.assertRaises(exception.UndefinedRootBDM, bdm_list.root_bdm)"},{"line_number":478,"context_line":""},{"line_number":479,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1a122d0e_37283086","line":476,"in_reply_to":"1a122d0e_e5cf347d","updated":"2016-04-20 06:48:19.000000000","message":"Done","commit_id":"80a8dbe541162bedb651c33be3f3b99ebec4b511"}],"nova/tests/unit/objects/test_ec2.py":[{"author":{"_account_id":19619,"name":"Michal Pryc","email":"mpryc@redhat.com","username":"migi"},"change_message_id":"4e2cad2ed75c3edd7f3de1553951cce38a90ac28","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \u0027deleted_at\u0027: None,"},{"line_number":28,"context_line":"    \u0027deleted\u0027: 0,"},{"line_number":29,"context_line":"    \u0027id\u0027: 1,"},{"line_number":30,"context_line":"    \u0027uuid\u0027: uuids.uuid,"},{"line_number":31,"context_line":"}"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1a122d0e_c5915854","line":30,"updated":"2016-04-19 07:08:51.000000000","message":"Please rename to be more meaningful like uuids.ec2_map_uuid or whatever you think fits here.","commit_id":"80a8dbe541162bedb651c33be3f3b99ebec4b511"},{"author":{"_account_id":20181,"name":"Srushti Gadadare","email":"srushti391993@gmail.com","username":"srushti"},"change_message_id":"a146b3cce9c255f4f3f15c5de8a878f601cdc326","unresolved":false,"context_lines":[{"line_number":27,"context_line":"    \u0027deleted_at\u0027: None,"},{"line_number":28,"context_line":"    \u0027deleted\u0027: 0,"},{"line_number":29,"context_line":"    \u0027id\u0027: 1,"},{"line_number":30,"context_line":"    \u0027uuid\u0027: uuids.uuid,"},{"line_number":31,"context_line":"}"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""}],"source_content_type":"text/x-python","patch_set":5,"id":"1a122d0e_57232ca7","line":30,"in_reply_to":"1a122d0e_c5915854","updated":"2016-04-20 06:48:19.000000000","message":"Done","commit_id":"80a8dbe541162bedb651c33be3f3b99ebec4b511"}]}