)]}'
{"dragonflow/tests/fullstack/test_object_version.py":[{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"d9a128bac66f6cc532bcb87c9061ab79579268ff","unresolved":false,"context_lines":[{"line_number":210,"context_line":"            fip_obj \u003d fip.get_floatingip()"},{"line_number":211,"context_line":"            self.assertEqual(fip_obj.get_lport_id(), port_id)"},{"line_number":212,"context_line":"            version \u003d self.nb_api.get_floatingip(fip_id).get_version()"},{"line_number":213,"context_line":"            self.assertEqual(first_version + 1, version)"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"            fip.update({})"},{"line_number":216,"context_line":"            fip_obj \u003d fip.get_floatingip()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_9c716247","line":213,"range":{"start_line":213,"start_character":0,"end_line":213,"end_character":56},"updated":"2016-09-12 02:13:05.000000000","message":"I think assertGreater will be more suitable here. The version is not controller by df now. We can\u0027t make sure new version equals old version +1. The revision will increase 1 for each db transaction. Ideally, each api will use one db transaction, but this is no guarantee for that.","commit_id":"f79d6b61f179967b4cedf53639c9dc3be6e2c733"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"8dc005f4b1aa27e7c3ae8bb6fa4fc04c89e5ff74","unresolved":false,"context_lines":[{"line_number":210,"context_line":"            fip_obj \u003d fip.get_floatingip()"},{"line_number":211,"context_line":"            self.assertEqual(fip_obj.get_lport_id(), port_id)"},{"line_number":212,"context_line":"            version \u003d self.nb_api.get_floatingip(fip_id).get_version()"},{"line_number":213,"context_line":"            self.assertEqual(first_version + 1, version)"},{"line_number":214,"context_line":""},{"line_number":215,"context_line":"            fip.update({})"},{"line_number":216,"context_line":"            fip_obj \u003d fip.get_floatingip()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_97d69def","line":213,"range":{"start_line":213,"start_character":0,"end_line":213,"end_character":56},"in_reply_to":"9a89bdaa_9c716247","updated":"2016-09-12 02:55:28.000000000","message":"I agree. It is also unimportant that the version increases by exactly one. What\u0027s important is that the version has increased.","commit_id":"f79d6b61f179967b4cedf53639c9dc3be6e2c733"},{"author":{"_account_id":11159,"name":"Hong Hui Xiao","email":"honghui_xiao@yeah.net","username":"HongHuiXiao"},"change_message_id":"d9a128bac66f6cc532bcb87c9061ab79579268ff","unresolved":false,"context_lines":[{"line_number":216,"context_line":"            fip_obj \u003d fip.get_floatingip()"},{"line_number":217,"context_line":"            self.assertIsNone(fip_obj.get_lport_id())"},{"line_number":218,"context_line":"            version \u003d self.nb_api.get_floatingip(fip_id).get_version()"},{"line_number":219,"context_line":"            self.assertEqual(first_version + 2, version)"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"            fip.close()"},{"line_number":222,"context_line":"            self.assertFalse(fip.exists())"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_dc912a48","line":219,"updated":"2016-09-12 02:13:05.000000000","message":"Ditto","commit_id":"f79d6b61f179967b4cedf53639c9dc3be6e2c733"},{"author":{"_account_id":20229,"name":"Omer Anson","email":"omer.anson@toganetworks.com","username":"oanson"},"change_message_id":"8dc005f4b1aa27e7c3ae8bb6fa4fc04c89e5ff74","unresolved":false,"context_lines":[{"line_number":216,"context_line":"            fip_obj \u003d fip.get_floatingip()"},{"line_number":217,"context_line":"            self.assertIsNone(fip_obj.get_lport_id())"},{"line_number":218,"context_line":"            version \u003d self.nb_api.get_floatingip(fip_id).get_version()"},{"line_number":219,"context_line":"            self.assertEqual(first_version + 2, version)"},{"line_number":220,"context_line":""},{"line_number":221,"context_line":"            fip.close()"},{"line_number":222,"context_line":"            self.assertFalse(fip.exists())"}],"source_content_type":"text/x-python","patch_set":1,"id":"9a89bdaa_b703a175","line":219,"in_reply_to":"9a89bdaa_dc912a48","updated":"2016-09-12 02:55:28.000000000","message":"Same response :)","commit_id":"f79d6b61f179967b4cedf53639c9dc3be6e2c733"}]}
