)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"fb04f15f87606c309bce052e16637bcf44e7b6c4","unresolved":false,"context_lines":[{"line_number":13,"context_line":"volume is exitsing any more after init host."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Fix this bug by using cascade deleting volume"},{"line_number":16,"context_line":"when init host, as any volme that is in deleting"},{"line_number":17,"context_line":"status has already passed API checks and should"},{"line_number":18,"context_line":"be deleted with all its snapshots."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"7aa08908_7e9f9203","line":16,"range":{"start_line":16,"start_character":23,"end_line":16,"end_character":28},"updated":"2016-06-06 12:41:05.000000000","message":"please fix this if you post one more patch.","commit_id":"83c579903a2cc0eebb14bc813ba42652342e0ae4"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"349e2c02a6a1c9f5561d547d5b688b307af904e2","unresolved":false,"context_lines":[{"line_number":13,"context_line":"volume is exitsing any more after init host."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Fix this bug by using cascade deleting volume"},{"line_number":16,"context_line":"when init host, as any volme that is in deleting"},{"line_number":17,"context_line":"status has already passed API checks and should"},{"line_number":18,"context_line":"be deleted with all its snapshots."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"7aa08908_696df397","line":16,"range":{"start_line":16,"start_character":23,"end_line":16,"end_character":28},"in_reply_to":"7aa08908_7e9f9203","updated":"2016-06-07 01:23:38.000000000","message":"sure","commit_id":"83c579903a2cc0eebb14bc813ba42652342e0ae4"},{"author":{"_account_id":9171,"name":"Vipin Balachandran","email":"vipin.bl@gmail.com","username":"vbala"},"change_message_id":"9cc295b61d894fd4442948f4c1c0cf33ab303c5f","unresolved":false,"context_lines":[{"line_number":13,"context_line":"volume is exitsing any more after init host."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Fix this bug by using cascade deleting volume"},{"line_number":16,"context_line":"when init host, as any volme that is in deleting"},{"line_number":17,"context_line":"status has already passed API checks and should"},{"line_number":18,"context_line":"be deleted with all its snapshots."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"5a9d85d2_b61ca978","line":16,"range":{"start_line":16,"start_character":23,"end_line":16,"end_character":28},"updated":"2016-06-19 16:46:45.000000000","message":"You promised to fix this typo in patch set 7 :)","commit_id":"fb8da1b765078e79a48ce7208e7c4ec4c26e41d0"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"b4b5854448eda17fe3a7970ad4e9b29b0e300d36","unresolved":false,"context_lines":[{"line_number":13,"context_line":"volume is exitsing any more after init host."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Fix this bug by using cascade deleting volume"},{"line_number":16,"context_line":"when init host, as any volme that is in deleting"},{"line_number":17,"context_line":"status has already passed API checks and should"},{"line_number":18,"context_line":"be deleted with all its snapshots."},{"line_number":19,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"5a9d85d2_ce1b5a1a","line":16,"range":{"start_line":16,"start_character":23,"end_line":16,"end_character":28},"in_reply_to":"5a9d85d2_b61ca978","updated":"2016-06-21 06:44:24.000000000","message":"Ops.... I can\u0027t believe I miss this. Sorry :(","commit_id":"fb8da1b765078e79a48ce7208e7c4ec4c26e41d0"}],"cinder/tests/unit/test_volume.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"929f8b3ed61e16e195b511a7e99b7e381c1a81f1","unresolved":false,"context_lines":[{"line_number":4596,"context_line":"        snapshot \u003d tests_utils.create_snapshot(self.context,"},{"line_number":4597,"context_line":"                                               volume.id, status\u003d\u0027deleting\u0027)"},{"line_number":4598,"context_line":"        self.volume.init_host()"},{"line_number":4599,"context_line":"        self.assertRaises(exception.SnapshotNotFound, snapshot.refresh)"},{"line_number":4600,"context_line":""},{"line_number":4601,"context_line":""},{"line_number":4602,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":5,"id":"9abb7d3a_d96ac3b2","line":4599,"updated":"2016-05-31 10:47:23.000000000","message":"nit: You should probably check that the volume has also been deleted.","commit_id":"903c26e583d2739c51fac2487c516371f08326b1"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"c51c0ee8759e96f047b950df597c6056edd017e1","unresolved":false,"context_lines":[{"line_number":4596,"context_line":"        snapshot \u003d tests_utils.create_snapshot(self.context,"},{"line_number":4597,"context_line":"                                               volume.id, status\u003d\u0027deleting\u0027)"},{"line_number":4598,"context_line":"        self.volume.init_host()"},{"line_number":4599,"context_line":"        self.assertRaises(exception.SnapshotNotFound, snapshot.refresh)"},{"line_number":4600,"context_line":""},{"line_number":4601,"context_line":""},{"line_number":4602,"context_line":"@ddt.ddt"}],"source_content_type":"text/x-python","patch_set":5,"id":"9abb7d3a_b9da5fe6","line":4599,"in_reply_to":"9abb7d3a_d96ac3b2","updated":"2016-05-31 10:55:13.000000000","message":"Sure, will update this soon.","commit_id":"903c26e583d2739c51fac2487c516371f08326b1"}],"cinder/volume/manager.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"7a1a4c8c25b78d11c4a75b36f743eafb98fee0b8","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    cfg.BoolOpt(\u0027volume_service_inithost_offload\u0027,"},{"line_number":104,"context_line":"                default\u003dFalse,"},{"line_number":105,"context_line":"                help\u003d\u0027Offload pending volume/snapshot delete during \u0027"},{"line_number":106,"context_line":"                     \u0027volume service startup\u0027),"},{"line_number":107,"context_line":"    cfg.StrOpt(\u0027zoning_mode\u0027,"},{"line_number":108,"context_line":"               help\u003d\u0027FC Zoning mode configured\u0027),"},{"line_number":109,"context_line":"    cfg.StrOpt(\u0027extra_capabilities\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"9abb7d3a_34e14c9b","line":106,"updated":"2016-05-31 13:31:17.000000000","message":"This may be rather confusing since it doesn\u0027t actually do anything for snapshot deletes that were done with the standard snapshot-delete method.","commit_id":"69b86aad044e8bf4324e48d16afef7dd1e5f75a0"},{"author":{"_account_id":8846,"name":"Hao Wang","display_name":"Hao Wang","email":"sxmatch1986@gmail.com","username":"sxmatch"},"change_message_id":"5869af968f979abefd4176606b444641fdd2a84b","unresolved":false,"context_lines":[{"line_number":103,"context_line":"    cfg.BoolOpt(\u0027volume_service_inithost_offload\u0027,"},{"line_number":104,"context_line":"                default\u003dFalse,"},{"line_number":105,"context_line":"                help\u003d\u0027Offload pending volume/snapshot delete during \u0027"},{"line_number":106,"context_line":"                     \u0027volume service startup\u0027),"},{"line_number":107,"context_line":"    cfg.StrOpt(\u0027zoning_mode\u0027,"},{"line_number":108,"context_line":"               help\u003d\u0027FC Zoning mode configured\u0027),"},{"line_number":109,"context_line":"    cfg.StrOpt(\u0027extra_capabilities\u0027,"}],"source_content_type":"text/x-python","patch_set":6,"id":"9abb7d3a_8604a9c6","line":106,"in_reply_to":"9abb7d3a_34e14c9b","updated":"2016-06-03 00:45:52.000000000","message":"Ops, it shouldn\u0027t be changed anymore, thanks Eric.","commit_id":"69b86aad044e8bf4324e48d16afef7dd1e5f75a0"}]}
