)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1849,"name":"Joe Gordon","email":"joe.gordon0@gmail.com","username":"jogo"},"change_message_id":"8d0b1ef91535d85573f8ed3583920473253e2cb6","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add tests to ensure retrying plug_vifs in pre_live_migration works."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"part of blueprint libvirt-driver-tests"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I80d4cdc27836511c55b2555556edd7a1964c7158"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAS3%2F%2Fw14%3D","line":11,"updated":"2013-09-19 18:34:44.000000000","message":"Because we are in Feature Freeze we aren\u0027t accepting BP patches, but because these are additional tests this is a good candidate for merging now.  So can you turn this into a bug.","commit_id":"2c3e49f3e6b9ca00c4533eb0330b1d003ab281f8"},{"author":{"_account_id":1970,"name":"Jacob Wen","email":"wenjianhn@gmail.com","username":"wenjianhn"},"change_message_id":"cefe5c6f54b8a12b7f7cf588f1b39e6aa9f4c94b","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add tests to ensure retrying plug_vifs in pre_live_migration works."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"part of blueprint libvirt-driver-tests"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I80d4cdc27836511c55b2555556edd7a1964c7158"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"AAAAS3%2F%2Frx8%3D","line":11,"in_reply_to":"AAAAS3%2F%2Fw14%3D","updated":"2013-09-22 04:25:03.000000000","message":"Done","commit_id":"2c3e49f3e6b9ca00c4533eb0330b1d003ab281f8"}],"nova/tests/virt/libvirt/test_libvirt.py":[{"author":{"_account_id":6722,"name":"ChangBo Guo(gcb)","email":"guochbo@cn.ibm.com","username":"guochbo"},"change_message_id":"9fbd4d999ea50d16adb6e476fb03a66ae6b3ead1","unresolved":false,"context_lines":[{"line_number":3080,"context_line":""},{"line_number":3081,"context_line":"    def test_pre_live_migration_plug_vifs_retry_works(self):"},{"line_number":3082,"context_line":"        self.flags(live_migration_retry_count\u003d3)"},{"line_number":3083,"context_line":"        called \u003d {\u0027count\u0027: 0}"},{"line_number":3084,"context_line":"        instance \u003d {\u0027name\u0027: \u0027test\u0027, \u0027uuid\u0027: \u0027uuid\u0027}"},{"line_number":3085,"context_line":""},{"line_number":3086,"context_line":"        def fake_plug_vifs(instance, network_info):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAS3%2F%2Frgs%3D","line":3083,"updated":"2013-09-22 09:35:07.000000000","message":"I think one interger variable is enough  :-),  I didn\u0027t see there is other key for dict called.\nhow about called_count \u003d0","commit_id":"b1ff4573ed477dd1c2505f96f93debce72304d88"},{"author":{"_account_id":6722,"name":"ChangBo Guo(gcb)","email":"guochbo@cn.ibm.com","username":"guochbo"},"change_message_id":"de4ff5f24b0afd08249a1826ced95121db569a20","unresolved":false,"context_lines":[{"line_number":3080,"context_line":""},{"line_number":3081,"context_line":"    def test_pre_live_migration_plug_vifs_retry_works(self):"},{"line_number":3082,"context_line":"        self.flags(live_migration_retry_count\u003d3)"},{"line_number":3083,"context_line":"        called \u003d {\u0027count\u0027: 0}"},{"line_number":3084,"context_line":"        instance \u003d {\u0027name\u0027: \u0027test\u0027, \u0027uuid\u0027: \u0027uuid\u0027}"},{"line_number":3085,"context_line":""},{"line_number":3086,"context_line":"        def fake_plug_vifs(instance, network_info):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAS3%2F%2FrVQ%3D","line":3083,"in_reply_to":"AAAAS3%2F%2FrV4%3D","updated":"2013-09-22 13:51:13.000000000","message":"I got it . ignore my comments :-)","commit_id":"b1ff4573ed477dd1c2505f96f93debce72304d88"},{"author":{"_account_id":1970,"name":"Jacob Wen","email":"wenjianhn@gmail.com","username":"wenjianhn"},"change_message_id":"fd1269b09f845bbac7ab33c5841023a815506384","unresolved":false,"context_lines":[{"line_number":3080,"context_line":""},{"line_number":3081,"context_line":"    def test_pre_live_migration_plug_vifs_retry_works(self):"},{"line_number":3082,"context_line":"        self.flags(live_migration_retry_count\u003d3)"},{"line_number":3083,"context_line":"        called \u003d {\u0027count\u0027: 0}"},{"line_number":3084,"context_line":"        instance \u003d {\u0027name\u0027: \u0027test\u0027, \u0027uuid\u0027: \u0027uuid\u0027}"},{"line_number":3085,"context_line":""},{"line_number":3086,"context_line":"        def fake_plug_vifs(instance, network_info):"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAS3%2F%2FrV4%3D","line":3083,"in_reply_to":"AAAAS3%2F%2Frgs%3D","updated":"2013-09-22 13:45:04.000000000","message":"I don\u0027t thinks so. Name binding in Python netsted scope is tricky. http://goo.gl/OYPWty","commit_id":"b1ff4573ed477dd1c2505f96f93debce72304d88"},{"author":{"_account_id":6722,"name":"ChangBo Guo(gcb)","email":"guochbo@cn.ibm.com","username":"guochbo"},"change_message_id":"9fbd4d999ea50d16adb6e476fb03a66ae6b3ead1","unresolved":false,"context_lines":[{"line_number":3084,"context_line":"        instance \u003d {\u0027name\u0027: \u0027test\u0027, \u0027uuid\u0027: \u0027uuid\u0027}"},{"line_number":3085,"context_line":""},{"line_number":3086,"context_line":"        def fake_plug_vifs(instance, network_info):"},{"line_number":3087,"context_line":"            called[\u0027count\u0027] +\u003d 1"},{"line_number":3088,"context_line":"            if called[\u0027count\u0027] \u003c CONF.live_migration_retry_count:"},{"line_number":3089,"context_line":"                raise processutils.ProcessExecutionError()"},{"line_number":3090,"context_line":"            else:"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAS3%2F%2Frgo%3D","line":3087,"updated":"2013-09-22 09:35:07.000000000","message":"same","commit_id":"b1ff4573ed477dd1c2505f96f93debce72304d88"},{"author":{"_account_id":6722,"name":"ChangBo Guo(gcb)","email":"guochbo@cn.ibm.com","username":"guochbo"},"change_message_id":"9fbd4d999ea50d16adb6e476fb03a66ae6b3ead1","unresolved":false,"context_lines":[{"line_number":3085,"context_line":""},{"line_number":3086,"context_line":"        def fake_plug_vifs(instance, network_info):"},{"line_number":3087,"context_line":"            called[\u0027count\u0027] +\u003d 1"},{"line_number":3088,"context_line":"            if called[\u0027count\u0027] \u003c CONF.live_migration_retry_count:"},{"line_number":3089,"context_line":"                raise processutils.ProcessExecutionError()"},{"line_number":3090,"context_line":"            else:"},{"line_number":3091,"context_line":"                return"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAS3%2F%2Frgk%3D","line":3088,"updated":"2013-09-22 09:35:07.000000000","message":"same","commit_id":"b1ff4573ed477dd1c2505f96f93debce72304d88"}]}