)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"064cf3bc4aa42ef7aa7f31b05732fe6e902ffca8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     liuming \u003cliumingxiyou@126.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-09-28 15:21:56 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Deletes evacuated instance files when source host is ok"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Image_backend is Rdb and the instance is evacuated due to the host power down.After the source host is powered on,the instance\u0027s files are not deleted."},{"line_number":10,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_27333963","line":7,"updated":"2018-10-02 18:11:24.000000000","message":"Please format the commit message according to the guidelines:\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"89ca1c9e600940074e8197b8dcce55a5eed33db1"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"064cf3bc4aa42ef7aa7f31b05732fe6e902ffca8","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Deletes evacuated instance files when source host is ok"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Image_backend is Rdb and the instance is evacuated due to the host power down.After the source host is powered on,the instance\u0027s files are not deleted."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"So this patch delete  evacuated instance files when source host is ok."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_c7260517","line":9,"updated":"2018-10-02 18:11:24.000000000","message":"This sounds like a bug. If so, there should be a launchpad bug record associated with it. If one doesn\u0027t already exist, please open one [1]. When you have a bug number, add to your commit message:\n\n Closes-Bug: #XXXXXXX\n\nThis allows cross-tracking of bugs with the changes that fix them.\n\n[1] https://bugs.launchpad.net/nova/+filebug","commit_id":"89ca1c9e600940074e8197b8dcce55a5eed33db1"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"a0213ba94e28db02c6a847b142804e8ba235cee6","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Deletes evacuated instance files when source host is ok"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Image_backend is Rdb and the instance is evacuated due to the "},{"line_number":10,"context_line":"host power down.After the source host is powered on,the instance\u0027s "},{"line_number":11,"context_line":"files are not deleted."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_4d905650","line":9,"range":{"start_line":9,"start_character":61,"end_line":9,"end_character":62},"updated":"2018-10-09 07:38:20.000000000","message":"extra space","commit_id":"b37e84c6e020bb6b017d681a087ad4c29fb1bda3"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"a0213ba94e28db02c6a847b142804e8ba235cee6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Deletes evacuated instance files when source host is ok"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Image_backend is Rdb and the instance is evacuated due to the "},{"line_number":10,"context_line":"host power down.After the source host is powered on,the instance\u0027s "},{"line_number":11,"context_line":"files are not deleted."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"So this patch delete  evacuated instance files when source host is ok."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_ed884222","line":10,"range":{"start_line":10,"start_character":15,"end_line":10,"end_character":16},"updated":"2018-10-09 07:38:20.000000000","message":"add a space","commit_id":"b37e84c6e020bb6b017d681a087ad4c29fb1bda3"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"a0213ba94e28db02c6a847b142804e8ba235cee6","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Deletes evacuated instance files when source host is ok"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Image_backend is Rdb and the instance is evacuated due to the "},{"line_number":10,"context_line":"host power down.After the source host is powered on,the instance\u0027s "},{"line_number":11,"context_line":"files are not deleted."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"So this patch delete  evacuated instance files when source host is ok."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_0d9a5e6e","line":10,"range":{"start_line":10,"start_character":66,"end_line":10,"end_character":67},"updated":"2018-10-09 07:38:20.000000000","message":"ditto","commit_id":"b37e84c6e020bb6b017d681a087ad4c29fb1bda3"},{"author":{"_account_id":15888,"name":"Zhenyu Zheng","email":"zheng.zhenyu@outlook.com","username":"Kevin_Zheng"},"change_message_id":"a0213ba94e28db02c6a847b142804e8ba235cee6","unresolved":false,"context_lines":[{"line_number":10,"context_line":"host power down.After the source host is powered on,the instance\u0027s "},{"line_number":11,"context_line":"files are not deleted."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"So this patch delete  evacuated instance files when source host is ok."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes-Bug: #1794685"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"3f79a3b5_8d7f4e16","line":13,"range":{"start_line":13,"start_character":21,"end_line":13,"end_character":22},"updated":"2018-10-09 07:38:20.000000000","message":"extra space","commit_id":"b37e84c6e020bb6b017d681a087ad4c29fb1bda3"}],"nova/virt/libvirt/driver.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"064cf3bc4aa42ef7aa7f31b05732fe6e902ffca8","unresolved":false,"context_lines":[{"line_number":1064,"context_line":"        is_shared_block_storage \u003d False"},{"line_number":1065,"context_line":"        if migrate_data and \u0027is_shared_block_storage\u0027 in migrate_data:"},{"line_number":1066,"context_line":"            is_shared_block_storage \u003d migrate_data.is_shared_block_storage"},{"line_number":1067,"context_line":"        if (destroy_disks or is_shared_block_storage or"},{"line_number":1068,"context_line":"                (migrate_data is None and CONF.libvirt.images_type \u003d\u003d \u0027rdb\u0027)):"},{"line_number":1069,"context_line":"            attempts \u003d int(instance.system_metadata.get(\u0027clean_attempts\u0027,"},{"line_number":1070,"context_line":"                                                        \u00270\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_2781f9c2","line":1067,"updated":"2018-10-02 18:11:24.000000000","message":"There should be tests for this change.\n\nThe fact that this passes existing tests indicates that there\u0027s a gap in coverage. Much of the time this leads to writing a separate change to introduce a test case, and then this change will fix it and adjust the test accordingly.","commit_id":"89ca1c9e600940074e8197b8dcce55a5eed33db1"},{"author":{"_account_id":9610,"name":"Tao Li","email":"litao3721@126.com","username":"Litao"},"change_message_id":"8636cbb68e439830e84b515e83d895dd55d8fadd","unresolved":false,"context_lines":[{"line_number":1065,"context_line":"        if migrate_data and \u0027is_shared_block_storage\u0027 in migrate_data:"},{"line_number":1066,"context_line":"            is_shared_block_storage \u003d migrate_data.is_shared_block_storage"},{"line_number":1067,"context_line":"        if (destroy_disks or is_shared_block_storage or"},{"line_number":1068,"context_line":"                (migrate_data is None and CONF.libvirt.images_type \u003d\u003d \u0027rdb\u0027)):"},{"line_number":1069,"context_line":"            attempts \u003d int(instance.system_metadata.get(\u0027clean_attempts\u0027,"},{"line_number":1070,"context_line":"                                                        \u00270\u0027))"},{"line_number":1071,"context_line":"            success \u003d self.delete_instance_files(instance)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f79a3b5_9f065421","line":1068,"range":{"start_line":1068,"start_character":71,"end_line":1068,"end_character":74},"updated":"2018-10-24 01:50:42.000000000","message":"This should be \u0027rbd\u0027 not \u0027rdb\u0027","commit_id":"9a5068507d893234713c83e6cda1d5fa86469128"}]}