)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"c442481f90643a4d71082bade919cd292e68490b","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     bingyan \u003chuangbingyan@huawei.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-11-25 11:11:13 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    fix \u0027huawei dorado v6\u0027 create new volume form snapshot error"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    change the luncopy mode,in dorado v6 we use clone_pair"},{"line_number":10,"context_line":"    modified some interface to call dorado v6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"a4d67964_10699bd8","line":7,"updated":"2021-11-29 15:45:43.000000000","message":"typo: should be \"from\"\n\nHere is how this line could be written so that it follows the convention for patches that fix driver bugs:\n\nHuawei - Fix dorado v6 creating volume from snapshot error","commit_id":"7482cbbf305a93488bedca1acd8ce1cfbff25da2"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"b05c50f09b3d4b9ece22c262e8670ed29d7ad759","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     bingyan \u003chuangbingyan@huawei.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-11-25 11:11:13 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    fix \u0027huawei dorado v6\u0027 create new volume form snapshot error"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    change the luncopy mode,in dorado v6 we use clone_pair"},{"line_number":10,"context_line":"    modified some interface to call dorado v6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"ea6d5a76_51c43ce3","line":7,"in_reply_to":"83bbf5c4_f1da84e4","updated":"2021-12-07 13:57:17.000000000","message":"The patch on master has a different Change-Id, so I agree this review should be closed. To do that, click the \"Abandon\" button near the top of the window on the right side.","commit_id":"7482cbbf305a93488bedca1acd8ce1cfbff25da2"},{"author":{"_account_id":34247,"name":"huangbingyan","email":"huangbingyan@huawei.com","username":"bingyan"},"change_message_id":"563e3438252a1359e02ab057757c738088501002","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     bingyan \u003chuangbingyan@huawei.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-11-25 11:11:13 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"    fix \u0027huawei dorado v6\u0027 create new volume form snapshot error"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    change the luncopy mode,in dorado v6 we use clone_pair"},{"line_number":10,"context_line":"    modified some interface to call dorado v6"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"83bbf5c4_f1da84e4","line":7,"in_reply_to":"a4d67964_10699bd8","updated":"2021-12-07 09:57:55.000000000","message":"thanks for the reply\ni resubmitted to master\nhttps://review.opendev.org/c/openstack/cinder/+/819790\n\nthe commit message is modifies.Do i have to close this request?","commit_id":"7482cbbf305a93488bedca1acd8ce1cfbff25da2"},{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"c442481f90643a4d71082bade919cd292e68490b","unresolved":true,"context_lines":[{"line_number":12,"context_line":"    DocImpact"},{"line_number":13,"context_line":"    Closes-Bug: #1942342"},{"line_number":14,"context_line":"    Implements:"},{"line_number":15,"context_line":"    Change-Id:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I760cbd0467305f25fc29b5f9818749ae9007b40e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"316e00e4_21728191","line":15,"updated":"2021-11-29 15:45:43.000000000","message":"I have several comments:\n- Do not indent L7..L15\n- Delete L12, L14 (they are not relevant for this patch)\n- Delete L15 (the actual change ID is L17, and is automatically added)","commit_id":"7482cbbf305a93488bedca1acd8ce1cfbff25da2"},{"author":{"_account_id":34247,"name":"huangbingyan","email":"huangbingyan@huawei.com","username":"bingyan"},"change_message_id":"563e3438252a1359e02ab057757c738088501002","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    DocImpact"},{"line_number":13,"context_line":"    Closes-Bug: #1942342"},{"line_number":14,"context_line":"    Implements:"},{"line_number":15,"context_line":"    Change-Id:"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I760cbd0467305f25fc29b5f9818749ae9007b40e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"1521ecd7_6cbacd39","line":15,"in_reply_to":"316e00e4_21728191","updated":"2021-12-07 09:57:55.000000000","message":"i resubmitted to master","commit_id":"7482cbbf305a93488bedca1acd8ce1cfbff25da2"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":21129,"name":"Alan Bishop","email":"abishopsweng@gmail.com","username":"ASBishop","status":"ex Red Hat"},"change_message_id":"c442481f90643a4d71082bade919cd292e68490b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"02d79534_1929023e","updated":"2021-11-29 15:45:43.000000000","message":"I haven\u0027t reviewed the actual code changes, but the biggest issue (in addition to my comments about the commit message) is that you cannot submit a patch directly to the stable/ussuri branch. Patches must be submitted to master, and after they merge they can be proposed to stable branches.","commit_id":"7482cbbf305a93488bedca1acd8ce1cfbff25da2"},{"author":{"_account_id":34247,"name":"huangbingyan","email":"huangbingyan@huawei.com","username":"bingyan"},"change_message_id":"563e3438252a1359e02ab057757c738088501002","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dd9d84a0_c1bf24e4","updated":"2021-12-07 09:57:55.000000000","message":"thanks for the reply\ni resubmitted to master\nhttps://review.opendev.org/c/openstack/cinder/+/819790\n\nthe commit message is modifies.Do i have to close this request?","commit_id":"7482cbbf305a93488bedca1acd8ce1cfbff25da2"},{"author":{"_account_id":34247,"name":"huangbingyan","email":"huangbingyan@huawei.com","username":"bingyan"},"change_message_id":"563e3438252a1359e02ab057757c738088501002","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c05f41f9_7846267a","in_reply_to":"02d79534_1929023e","updated":"2021-12-07 09:57:55.000000000","message":"thanks for the reply\ni resubmitted to master\nhttps://review.opendev.org/c/openstack/cinder/+/819790\n\nthe commit message is modifies.Do i have to close this request?","commit_id":"7482cbbf305a93488bedca1acd8ce1cfbff25da2"}]}