)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"8cf962a5d00f2bfcf7d4612661830f3df4dc8ce2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"74e29e27_edad23fd","updated":"2022-07-11 10:43:59.000000000","message":"LGTM","commit_id":"ae1a1a2ea9f5a28bb0fcfe34814c2deb4e480273"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"e96e304832e8c6ff30cee1cf66ded7cc39211271","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d029c8e0_33271517","updated":"2022-06-23 15:38:08.000000000","message":"Thank you for the review!\n","commit_id":"ae1a1a2ea9f5a28bb0fcfe34814c2deb4e480273"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"1255b7078b8c7e5942568523ea2c682b2b044a8b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"34303120_8072078a","updated":"2022-05-19 18:56:16.000000000","message":"recheck\n","commit_id":"ae1a1a2ea9f5a28bb0fcfe34814c2deb4e480273"}],"cinder/volume/manager.py":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"a401f3d703ecacc22854fe7c0eda4a974e0c0268","unresolved":true,"context_lines":[{"line_number":799,"context_line":"            locked_action \u003d None"},{"line_number":800,"context_line":""},{"line_number":801,"context_line":"        # Make sure the created volume is not deleted while connected for"},{"line_number":802,"context_line":"        # copying data such as downloading an image."},{"line_number":803,"context_line":"        locked_volume_action \u003d \"%s-%s\" % (volume.id, \u0027delete_volume\u0027)"},{"line_number":804,"context_line":""},{"line_number":805,"context_line":"        def _run_flow() -\u003e None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"bc10bda5_02332efb","line":802,"range":{"start_line":802,"start_character":23,"end_line":802,"end_character":51},"updated":"2022-05-24 13:07:01.000000000","message":"That specific case is also handled by [1]. However, adding the additional lock action here is a good failsafe that ensure there are no other circumstances where a volume is accidentally deleted while it\u0027s being created.\n\n[1] https://review.opendev.org/c/openstack/cinder/+/830901","commit_id":"ae1a1a2ea9f5a28bb0fcfe34814c2deb4e480273"},{"author":{"_account_id":13689,"name":"Hironori Shiina","email":"Hironori.Shiina@fujitsu.com","username":"shiina"},"change_message_id":"e96e304832e8c6ff30cee1cf66ded7cc39211271","unresolved":false,"context_lines":[{"line_number":799,"context_line":"            locked_action \u003d None"},{"line_number":800,"context_line":""},{"line_number":801,"context_line":"        # Make sure the created volume is not deleted while connected for"},{"line_number":802,"context_line":"        # copying data such as downloading an image."},{"line_number":803,"context_line":"        locked_volume_action \u003d \"%s-%s\" % (volume.id, \u0027delete_volume\u0027)"},{"line_number":804,"context_line":""},{"line_number":805,"context_line":"        def _run_flow() -\u003e None:"}],"source_content_type":"text/x-python","patch_set":1,"id":"b0b2678b_51c33ef1","line":802,"range":{"start_line":802,"start_character":23,"end_line":802,"end_character":51},"in_reply_to":"bc10bda5_02332efb","updated":"2022-06-23 15:38:08.000000000","message":"This patch covers a case where the volume status gets \u0027available\u0027 while it is actually downloading an image. This case can be reproduced by a procedure in the bug report.","commit_id":"ae1a1a2ea9f5a28bb0fcfe34814c2deb4e480273"}]}