)]}'
{"nova/tests/unit/objects/test_instance.py":[{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"63687775a875633d1d15919564082618ea474bf5","unresolved":false,"context_lines":[{"line_number":1486,"context_line":""},{"line_number":1487,"context_line":"    def test_obj_clone(self):"},{"line_number":1488,"context_line":"        # Make sure clone shows no changes when no metadata is set"},{"line_number":1489,"context_line":"        inst1 \u003d objects.Instance(uuid\u003duuids.instance)"},{"line_number":1490,"context_line":"        inst1.obj_reset_changes()"},{"line_number":1491,"context_line":"        inst1 \u003d inst1.obj_clone()"},{"line_number":1492,"context_line":"        self.assertEqual(len(inst1.obj_what_changed()), 0)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_07a3079e","line":1489,"updated":"2016-11-10 08:03:08.000000000","message":"If there is only one variable \u0027inst1\u0027, is the \u0027inst\u0027 naming more proper?","commit_id":"9345ed40e85e291d2f9e9aac24d468f40140ae0e"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"9ddefc50f711fc87d6b6717c07f07e76257114d6","unresolved":false,"context_lines":[{"line_number":1486,"context_line":""},{"line_number":1487,"context_line":"    def test_obj_clone(self):"},{"line_number":1488,"context_line":"        # Make sure clone shows no changes when no metadata is set"},{"line_number":1489,"context_line":"        inst1 \u003d objects.Instance(uuid\u003duuids.instance)"},{"line_number":1490,"context_line":"        inst1.obj_reset_changes()"},{"line_number":1491,"context_line":"        inst1 \u003d inst1.obj_clone()"},{"line_number":1492,"context_line":"        self.assertEqual(len(inst1.obj_what_changed()), 0)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_fbb779d5","line":1489,"in_reply_to":"9a629dbe_07a3079e","updated":"2016-11-10 14:12:27.000000000","message":"seems like a nit, bit harsh to -1...","commit_id":"9345ed40e85e291d2f9e9aac24d468f40140ae0e"},{"author":{"_account_id":23950,"name":"Eric Xie","email":"eric_xiett@163.com","username":"ericxie"},"change_message_id":"8a8cc4fdaaed4c98d09dfa959703256a11cea456","unresolved":false,"context_lines":[{"line_number":1486,"context_line":""},{"line_number":1487,"context_line":"    def test_obj_clone(self):"},{"line_number":1488,"context_line":"        # Make sure clone shows no changes when no metadata is set"},{"line_number":1489,"context_line":"        inst1 \u003d objects.Instance(uuid\u003duuids.instance)"},{"line_number":1490,"context_line":"        inst1.obj_reset_changes()"},{"line_number":1491,"context_line":"        inst1 \u003d inst1.obj_clone()"},{"line_number":1492,"context_line":"        self.assertEqual(len(inst1.obj_what_changed()), 0)"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_8c154e9d","line":1489,"in_reply_to":"9a629dbe_fbb779d5","updated":"2016-11-11 02:04:23.000000000","message":"Sorry，i change it.","commit_id":"9345ed40e85e291d2f9e9aac24d468f40140ae0e"},{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"824941128532a1b873b7559fe12abd81a7345d37","unresolved":false,"context_lines":[{"line_number":1495,"context_line":"        inst1.metadata \u003d dict(key1\u003d\u0027val1\u0027)"},{"line_number":1496,"context_line":"        inst1.system_metadata \u003d dict(key1\u003d\u0027val1\u0027)"},{"line_number":1497,"context_line":"        inst1.obj_reset_changes()"},{"line_number":1498,"context_line":"        inst1 \u003d inst1.obj_clone()"},{"line_number":1499,"context_line":"        self.assertEqual(len(inst1.obj_what_changed()), 0)"},{"line_number":1500,"context_line":""},{"line_number":1501,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_c54d42c5","line":1498,"updated":"2016-11-11 16:22:18.000000000","message":"Nit: I think calling this cloned might help readability.","commit_id":"9345ed40e85e291d2f9e9aac24d468f40140ae0e"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"4e1587c9f51ec5691c3342fc7bb0a6f6de7ede45","unresolved":false,"context_lines":[{"line_number":1496,"context_line":"        inst1.system_metadata \u003d dict(key1\u003d\u0027val1\u0027)"},{"line_number":1497,"context_line":"        inst1.obj_reset_changes()"},{"line_number":1498,"context_line":"        inst1 \u003d inst1.obj_clone()"},{"line_number":1499,"context_line":"        self.assertEqual(len(inst1.obj_what_changed()), 0)"},{"line_number":1500,"context_line":""},{"line_number":1501,"context_line":""},{"line_number":1502,"context_line":"class TestInstanceObject(test_objects._LocalTest,"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a629dbe_44650f1c","line":1499,"updated":"2016-11-09 20:05:41.000000000","message":"This isn\u0027t super easy to understand, but I pulled it down and it seems to do the right thing.","commit_id":"9345ed40e85e291d2f9e9aac24d468f40140ae0e"}]}