)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"72602b473769273335fab8aa1ef7327302940fc8","unresolved":false,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2015-07-11 03:12:40 -0400"},{"line_number":4,"context_line":"Commit:     Einst \u003cyu.changcai@99cloud.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-07-11 04:42:56 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fixbug for delete snapshot for attached volume failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"        When use gluster as cinder backing storage."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_edd62e4e","line":6,"updated":"2015-07-22 14:25:49.000000000","message":"please see https://wiki.openstack.org/wiki/GitCommitMessages","commit_id":"012825e31ce60dc33aca922bb8fcbb89fd389263"},{"author":{"_account_id":14084,"name":"Einst Crazy","email":"yu.changcai@99cloud.net","username":"Einst"},"change_message_id":"bee8a2f15e9e59649f4da5e99dd7e4ad398d2904","unresolved":false,"context_lines":[{"line_number":3,"context_line":"AuthorDate: 2015-07-11 03:12:40 -0400"},{"line_number":4,"context_line":"Commit:     Einst \u003cyu.changcai@99cloud.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-07-11 04:42:56 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fixbug for delete snapshot for attached volume failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"        When use gluster as cinder backing storage."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_30f7895a","line":6,"in_reply_to":"3a50d1a3_edd62e4e","updated":"2015-07-22 14:40:39.000000000","message":"Well， I would see it before the 2nd patch set","commit_id":"012825e31ce60dc33aca922bb8fcbb89fd389263"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"72602b473769273335fab8aa1ef7327302940fc8","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Einst \u003cyu.changcai@99cloud.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2015-07-11 04:42:56 -0400"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"fixbug for delete snapshot for attached volume failed"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"        When use gluster as cinder backing storage."},{"line_number":10,"context_line":"        Deleting snapshot for attached volume will cause"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3a50d1a3_4d22c253","line":7,"updated":"2015-07-22 14:25:49.000000000","message":"please prefix this with libvirt:","commit_id":"012825e31ce60dc33aca922bb8fcbb89fd389263"},{"author":{"_account_id":15524,"name":"dane-fichter","email":"dane.fichter@jhuapl.edu","username":"dane-fichter"},"change_message_id":"fe5ba81bdcdd515ebae36b8d12cb2ef27e2ebe70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3a50d1a3_82acda8c","updated":"2015-07-23 12:38:03.000000000","message":"libvirtd: Bug fix for deleting snapshot of attached volume\n\nWhen using gluster as cinder backing storage, deleting a snapshot of an\nattached volume will cause Nova to reference _get_snap_dev, which will \nhave a backing_store of None. This patch adds a simple check of\nbacking_store to prevent this error.","commit_id":"e14d816cc36c8a020cefe1a2b103d581c7bff764"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"1bd3fc6dbef220d3375fd597e30aabd9e5606752","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"libvirtd: Bug fix for deleting snapshot of attached volume"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"\tWhen using gluster as cinder backing storage,"},{"line_number":10,"context_line":"\tdeleting a snapshot of an attached volume will cause"},{"line_number":11,"context_line":"\tNova to reference _get_snap_dev, which will have"},{"line_number":12,"context_line":"\ta backing_store of None. This patch adds a simple check"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3a50d1a3_2fb9ca79","line":9,"updated":"2015-07-26 04:44:51.000000000","message":"please remove the white spaces","commit_id":"31854e6125c54ecc1f24ee74b31d16f99ff9c0a8"}],"nova/virt/libvirt/driver.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"72602b473769273335fab8aa1ef7327302940fc8","unresolved":false,"context_lines":[{"line_number":1908,"context_line":""},{"line_number":1909,"context_line":"            rebase_disk \u003d my_dev"},{"line_number":1910,"context_line":"            rebase_base \u003d delete_info[\u0027file_to_merge\u0027]  # often None"},{"line_number":1911,"context_line":"            if active_protocol is not None"},{"line_number":1912,"context_line":"                      and active_disk_object.backing_store is not None:"},{"line_number":1913,"context_line":"                rebase_base \u003d _get_snap_dev(delete_info[\u0027file_to_merge\u0027],"},{"line_number":1914,"context_line":"                                            active_disk_object.backing_store)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_6d71be75","line":1911,"updated":"2015-07-22 14:25:49.000000000","message":"if (active_protocol is not None and\n        and active_disk_object.backing_store is not None):","commit_id":"012825e31ce60dc33aca922bb8fcbb89fd389263"},{"author":{"_account_id":14084,"name":"Einst Crazy","email":"yu.changcai@99cloud.net","username":"Einst"},"change_message_id":"bee8a2f15e9e59649f4da5e99dd7e4ad398d2904","unresolved":false,"context_lines":[{"line_number":1908,"context_line":""},{"line_number":1909,"context_line":"            rebase_disk \u003d my_dev"},{"line_number":1910,"context_line":"            rebase_base \u003d delete_info[\u0027file_to_merge\u0027]  # often None"},{"line_number":1911,"context_line":"            if active_protocol is not None"},{"line_number":1912,"context_line":"                      and active_disk_object.backing_store is not None:"},{"line_number":1913,"context_line":"                rebase_base \u003d _get_snap_dev(delete_info[\u0027file_to_merge\u0027],"},{"line_number":1914,"context_line":"                                            active_disk_object.backing_store)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3a50d1a3_3046a94f","line":1911,"in_reply_to":"3a50d1a3_6d71be75","updated":"2015-07-22 14:40:39.000000000","message":"OK，I\u0027ll do","commit_id":"012825e31ce60dc33aca922bb8fcbb89fd389263"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"1bd3fc6dbef220d3375fd597e30aabd9e5606752","unresolved":false,"context_lines":[{"line_number":1908,"context_line":""},{"line_number":1909,"context_line":"            rebase_disk \u003d my_dev"},{"line_number":1910,"context_line":"            rebase_base \u003d delete_info[\u0027file_to_merge\u0027]  # often None"},{"line_number":1911,"context_line":"            if (active_protocol is not None"},{"line_number":1912,"context_line":"                      and active_disk_object.backing_store is not None):"},{"line_number":1913,"context_line":"                rebase_base \u003d _get_snap_dev(delete_info[\u0027file_to_merge\u0027],"},{"line_number":1914,"context_line":"                                            active_disk_object.backing_store)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a50d1a3_cfb0668e","line":1911,"updated":"2015-07-26 04:44:51.000000000","message":"you need to add in a unit test for this flow","commit_id":"31854e6125c54ecc1f24ee74b31d16f99ff9c0a8"}]}