)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d5feddf07f9be66972c6b082c79c7a47423e650b","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     b-prajakta \u003cprajaktabelapurkar11@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-05-04 17:07:08 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Volume Deletion rejection message  does not include \"awaiting-transfer\""},{"line_number":8,"context_line":"in invalid volume state"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"When user tries to delete a volume in \"awaiting-transfer\" state,"},{"line_number":11,"context_line":"the error message received does not include the \"awaiting-transfer\""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"a2f7471d_4e2023a9","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":23},"updated":"2022-05-05 07:35:27.000000000","message":"this should be less than 50 chars, please take a look at the guidelines for commit message\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":34807,"name":"Prajakta Swapnil Belapurkar","email":"prajaktabelapurkar11@gmail.com","username":"b-prajakta"},"change_message_id":"5f0a471d54d511dd8f149948c8f02cbca91efc88","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     b-prajakta \u003cprajaktabelapurkar11@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-05-04 17:07:08 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Volume Deletion rejection message  does not include \"awaiting-transfer\""},{"line_number":8,"context_line":"in invalid volume state"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"When user tries to delete a volume in \"awaiting-transfer\" state,"},{"line_number":11,"context_line":"the error message received does not include the \"awaiting-transfer\""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"bdca892f_b9e1cc0f","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":23},"in_reply_to":"a2f7471d_4e2023a9","updated":"2022-05-05 17:27:33.000000000","message":"Thanks for the link \nAddressed the review comment","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d5feddf07f9be66972c6b082c79c7a47423e650b","unresolved":true,"context_lines":[{"line_number":7,"context_line":"Volume Deletion rejection message  does not include \"awaiting-transfer\""},{"line_number":8,"context_line":"in invalid volume state"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"When user tries to delete a volume in \"awaiting-transfer\" state,"},{"line_number":11,"context_line":"the error message received does not include the \"awaiting-transfer\""},{"line_number":12,"context_line":"state in invalid states of Volume for deletion."},{"line_number":13,"context_line":"This leads to the user believing the volume-delete request is valid"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"46dd721e_14e224ba","line":10,"range":{"start_line":10,"start_character":5,"end_line":10,"end_character":9},"updated":"2022-05-05 07:35:27.000000000","message":"nit: a user","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":34807,"name":"Prajakta Swapnil Belapurkar","email":"prajaktabelapurkar11@gmail.com","username":"b-prajakta"},"change_message_id":"5f0a471d54d511dd8f149948c8f02cbca91efc88","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Volume Deletion rejection message  does not include \"awaiting-transfer\""},{"line_number":8,"context_line":"in invalid volume state"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"When user tries to delete a volume in \"awaiting-transfer\" state,"},{"line_number":11,"context_line":"the error message received does not include the \"awaiting-transfer\""},{"line_number":12,"context_line":"state in invalid states of Volume for deletion."},{"line_number":13,"context_line":"This leads to the user believing the volume-delete request is valid"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"aba00d73_68033b55","line":10,"range":{"start_line":10,"start_character":5,"end_line":10,"end_character":9},"in_reply_to":"46dd721e_14e224ba","updated":"2022-05-05 17:27:33.000000000","message":"Addressed the review comment","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d5feddf07f9be66972c6b082c79c7a47423e650b","unresolved":true,"context_lines":[{"line_number":14,"context_line":"and is not able to debug the reason for failure of the volume deletion"},{"line_number":15,"context_line":"for volume present in \"awaiting-transfer\" state."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes bug: Bug #1971603"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I78915c332169b26ffb2b97310efedec65bc25e4d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"d46bf3c0_20589900","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":24},"updated":"2022-05-05 07:35:27.000000000","message":"This should be Closes-Bug: #1971603\n\nSee this section for adding external references https://wiki.openstack.org/wiki/GitCommitMessages#Including_external_references","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":34807,"name":"Prajakta Swapnil Belapurkar","email":"prajaktabelapurkar11@gmail.com","username":"b-prajakta"},"change_message_id":"5f0a471d54d511dd8f149948c8f02cbca91efc88","unresolved":false,"context_lines":[{"line_number":14,"context_line":"and is not able to debug the reason for failure of the volume deletion"},{"line_number":15,"context_line":"for volume present in \"awaiting-transfer\" state."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Closes bug: Bug #1971603"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I78915c332169b26ffb2b97310efedec65bc25e4d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"25f229a0_723dbadf","line":17,"range":{"start_line":17,"start_character":0,"end_line":17,"end_character":24},"in_reply_to":"d46bf3c0_20589900","updated":"2022-05-05 17:27:33.000000000","message":"Thanks for sharing the relevant section.\nAddressed the review comments.","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ab6b97d5b1b5c890a1c199edcff4e8f0a3e0ea96","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Update Volume Delete rejection message"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When a user tries to delete a volume in \"awaiting-transfer\" state ,"},{"line_number":10,"context_line":"the error message received does not include the \"awaiting-transfer\""},{"line_number":11,"context_line":"state in invalid states of Volume for deletion."},{"line_number":12,"context_line":"This leads to the user believing the volume-delete request is valid"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"894d93c0_904a7679","line":9,"range":{"start_line":9,"start_character":65,"end_line":9,"end_character":66},"updated":"2022-05-06 08:15:58.000000000","message":"nit: extra space","commit_id":"2b75fa3021e1e6b4d635ab36a9084719c0e3d969"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"6cfef5da76fc94a1df900eb9519ba59bb0310627","unresolved":true,"context_lines":[{"line_number":14,"context_line":"for volume present in \"awaiting-transfer\" state."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-Bug: #1971603"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Change-Id: I78915c332169b26ffb2b97310efedec65bc25e4d"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"b824808c_a0404589","line":17,"updated":"2022-05-06 09:16:53.000000000","message":"nit: no empty line between footers","commit_id":"2b75fa3021e1e6b4d635ab36a9084719c0e3d969"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d5feddf07f9be66972c6b082c79c7a47423e650b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1c95e80c_17b75fd9","updated":"2022-05-05 07:35:27.000000000","message":"Thanks for your contribution Prajakta, few comments inline. Please follow commit message guidelines with the links provided.","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":34807,"name":"Prajakta Swapnil Belapurkar","email":"prajaktabelapurkar11@gmail.com","username":"b-prajakta"},"change_message_id":"5f0a471d54d511dd8f149948c8f02cbca91efc88","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"14c51608_ebe946c6","in_reply_to":"1c95e80c_17b75fd9","updated":"2022-05-05 17:27:33.000000000","message":"Thank you Rajat for the detailed review and helpful links.\nI have addressed the review comments and uploaded the updated patch .","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":34807,"name":"Prajakta Swapnil Belapurkar","email":"prajaktabelapurkar11@gmail.com","username":"b-prajakta"},"change_message_id":"c80dac4a5ed9b5f56b2c67df981442dfc6fa76d9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b5ed5dc8_1d60a38d","updated":"2022-05-07 08:33:39.000000000","message":"\u003e Patch Set 5:\n\u003e \n\u003e Build failed. Test completed on IBM PowerKVM platform. For rechecking only on the IBM PowerKVM CI, add a review comment with pkvm- recheck. For contact and more information, see https://wiki.openstack.org/wiki/PowerKVM\n\u003e \n\u003e - tempest-dsvm-full-focal-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/cinder/11/840511/5/check/tempest-dsvm-full-focal-py3/356e3e5/ : SUCCESS in 1h 55m 08s\n\u003e - devstack-plugin-ceph-dsvm-focal-py3 https://oplab9.parqtec.unicamp.br/pub/ppc64el/openstack/cinder/11/840511/5/check/devstack-plugin-ceph-dsvm-focal-py3/30bf858/ : TIMED_OUT in 3h 10m 21s\n\nrecheck","commit_id":"2b75fa3021e1e6b4d635ab36a9084719c0e3d969"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"6cfef5da76fc94a1df900eb9519ba59bb0310627","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"4a3d132e_7d123029","updated":"2022-05-06 09:16:53.000000000","message":"LGTM","commit_id":"2b75fa3021e1e6b4d635ab36a9084719c0e3d969"},{"author":{"_account_id":34807,"name":"Prajakta Swapnil Belapurkar","email":"prajaktabelapurkar11@gmail.com","username":"b-prajakta"},"change_message_id":"5f0a471d54d511dd8f149948c8f02cbca91efc88","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"2c775026_00148387","updated":"2022-05-05 17:27:33.000000000","message":"Thank you Rajat for the detailed review and helpful links.\nI have addressed all the review comments and uploaded the updated patch.\nThanks ","commit_id":"2b75fa3021e1e6b4d635ab36a9084719c0e3d969"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ab6b97d5b1b5c890a1c199edcff4e8f0a3e0ea96","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8f415dbd_f02b56d5","updated":"2022-05-06 08:15:58.000000000","message":"Thanks Prajakta. If you need to do another update then address the inline comments else it looks good to me.","commit_id":"2b75fa3021e1e6b4d635ab36a9084719c0e3d969"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"a16eacc450c2648ce87623d5b6226940eec889ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"d3fdde5f_a12a1501","updated":"2022-05-06 13:48:39.000000000","message":"recheck","commit_id":"2b75fa3021e1e6b4d635ab36a9084719c0e3d969"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"8072818f8dbd026757d4dc9911bc9a4367db6793","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1ff4fa67_f3b1cfae","updated":"2022-05-07 16:51:47.000000000","message":"recheck cinder-grenade-mn-sub-volbak failed during devstack setup","commit_id":"2b75fa3021e1e6b4d635ab36a9084719c0e3d969"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cd0776f2d76b655c005ee0aa33907fc43ae18cf2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"17b5c73d_be92df0e","updated":"2022-05-08 21:21:12.000000000","message":"recheck cinder-plugin-ceph-tempest - 159 failures, a lot of 500s from the identity service","commit_id":"2b75fa3021e1e6b4d635ab36a9084719c0e3d969"}],"cinder/volume/api.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d5feddf07f9be66972c6b082c79c7a47423e650b","unresolved":true,"context_lines":[{"line_number":507,"context_line":"        if not result:"},{"line_number":508,"context_line":"            status \u003d utils.build_or_str(expected.get(\u0027status\u0027),"},{"line_number":509,"context_line":"                                        _(\u0027status must be %s and\u0027))"},{"line_number":510,"context_line":"            msg \u003d _(\u0027Volume %s must not be migrating, attached,belong to a \u0027"},{"line_number":511,"context_line":"                    \u0027group, have snapshots , awaiting-transfer, \u0027"},{"line_number":512,"context_line":"                    \u0027or be disassociated from \u0027"},{"line_number":513,"context_line":"                    \u0027snapshots after volume transfer.\u0027) % status"}],"source_content_type":"text/x-python","patch_set":4,"id":"07ba5fe1_412449fc","line":510,"range":{"start_line":510,"start_character":62,"end_line":510,"end_character":64},"updated":"2022-05-05 07:35:27.000000000","message":"not related to the patch but a space is needed here","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":34807,"name":"Prajakta Swapnil Belapurkar","email":"prajaktabelapurkar11@gmail.com","username":"b-prajakta"},"change_message_id":"5f0a471d54d511dd8f149948c8f02cbca91efc88","unresolved":false,"context_lines":[{"line_number":507,"context_line":"        if not result:"},{"line_number":508,"context_line":"            status \u003d utils.build_or_str(expected.get(\u0027status\u0027),"},{"line_number":509,"context_line":"                                        _(\u0027status must be %s and\u0027))"},{"line_number":510,"context_line":"            msg \u003d _(\u0027Volume %s must not be migrating, attached,belong to a \u0027"},{"line_number":511,"context_line":"                    \u0027group, have snapshots , awaiting-transfer, \u0027"},{"line_number":512,"context_line":"                    \u0027or be disassociated from \u0027"},{"line_number":513,"context_line":"                    \u0027snapshots after volume transfer.\u0027) % status"}],"source_content_type":"text/x-python","patch_set":4,"id":"b98c4138_79d798f4","line":510,"range":{"start_line":510,"start_character":62,"end_line":510,"end_character":64},"in_reply_to":"07ba5fe1_412449fc","updated":"2022-05-05 17:27:33.000000000","message":"Resolved","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d5feddf07f9be66972c6b082c79c7a47423e650b","unresolved":true,"context_lines":[{"line_number":508,"context_line":"            status \u003d utils.build_or_str(expected.get(\u0027status\u0027),"},{"line_number":509,"context_line":"                                        _(\u0027status must be %s and\u0027))"},{"line_number":510,"context_line":"            msg \u003d _(\u0027Volume %s must not be migrating, attached,belong to a \u0027"},{"line_number":511,"context_line":"                    \u0027group, have snapshots , awaiting-transfer, \u0027"},{"line_number":512,"context_line":"                    \u0027or be disassociated from \u0027"},{"line_number":513,"context_line":"                    \u0027snapshots after volume transfer.\u0027) % status"},{"line_number":514,"context_line":"            LOG.info(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"cf0a292e_36007003","line":511,"range":{"start_line":511,"start_character":42,"end_line":511,"end_character":43},"updated":"2022-05-05 07:35:27.000000000","message":"nit: remove this space","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"d5feddf07f9be66972c6b082c79c7a47423e650b","unresolved":true,"context_lines":[{"line_number":508,"context_line":"            status \u003d utils.build_or_str(expected.get(\u0027status\u0027),"},{"line_number":509,"context_line":"                                        _(\u0027status must be %s and\u0027))"},{"line_number":510,"context_line":"            msg \u003d _(\u0027Volume %s must not be migrating, attached,belong to a \u0027"},{"line_number":511,"context_line":"                    \u0027group, have snapshots , awaiting-transfer, \u0027"},{"line_number":512,"context_line":"                    \u0027or be disassociated from \u0027"},{"line_number":513,"context_line":"                    \u0027snapshots after volume transfer.\u0027) % status"},{"line_number":514,"context_line":"            LOG.info(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"cbedbe50_17dd59b5","line":511,"range":{"start_line":511,"start_character":45,"end_line":511,"end_character":62},"updated":"2022-05-05 07:35:27.000000000","message":"since this is targeted for end users, we don\u0027t need to mention the exact volume state and can just say \"awaiting a transfer\"","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":34807,"name":"Prajakta Swapnil Belapurkar","email":"prajaktabelapurkar11@gmail.com","username":"b-prajakta"},"change_message_id":"5f0a471d54d511dd8f149948c8f02cbca91efc88","unresolved":false,"context_lines":[{"line_number":508,"context_line":"            status \u003d utils.build_or_str(expected.get(\u0027status\u0027),"},{"line_number":509,"context_line":"                                        _(\u0027status must be %s and\u0027))"},{"line_number":510,"context_line":"            msg \u003d _(\u0027Volume %s must not be migrating, attached,belong to a \u0027"},{"line_number":511,"context_line":"                    \u0027group, have snapshots , awaiting-transfer, \u0027"},{"line_number":512,"context_line":"                    \u0027or be disassociated from \u0027"},{"line_number":513,"context_line":"                    \u0027snapshots after volume transfer.\u0027) % status"},{"line_number":514,"context_line":"            LOG.info(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"3691d258_29505990","line":511,"range":{"start_line":511,"start_character":45,"end_line":511,"end_character":62},"in_reply_to":"cbedbe50_17dd59b5","updated":"2022-05-05 17:27:33.000000000","message":"addressed the review comment","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":34807,"name":"Prajakta Swapnil Belapurkar","email":"prajaktabelapurkar11@gmail.com","username":"b-prajakta"},"change_message_id":"5f0a471d54d511dd8f149948c8f02cbca91efc88","unresolved":false,"context_lines":[{"line_number":508,"context_line":"            status \u003d utils.build_or_str(expected.get(\u0027status\u0027),"},{"line_number":509,"context_line":"                                        _(\u0027status must be %s and\u0027))"},{"line_number":510,"context_line":"            msg \u003d _(\u0027Volume %s must not be migrating, attached,belong to a \u0027"},{"line_number":511,"context_line":"                    \u0027group, have snapshots , awaiting-transfer, \u0027"},{"line_number":512,"context_line":"                    \u0027or be disassociated from \u0027"},{"line_number":513,"context_line":"                    \u0027snapshots after volume transfer.\u0027) % status"},{"line_number":514,"context_line":"            LOG.info(msg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"69435635_841251ef","line":511,"range":{"start_line":511,"start_character":42,"end_line":511,"end_character":43},"in_reply_to":"cf0a292e_36007003","updated":"2022-05-05 17:27:33.000000000","message":"Addressed the review comments","commit_id":"89f5311e50a7a8540a9de70365a04db7ac28962b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"ab6b97d5b1b5c890a1c199edcff4e8f0a3e0ea96","unresolved":true,"context_lines":[{"line_number":508,"context_line":"            status \u003d utils.build_or_str(expected.get(\u0027status\u0027),"},{"line_number":509,"context_line":"                                        _(\u0027status must be %s and\u0027))"},{"line_number":510,"context_line":"            msg \u003d _(\u0027Volume %s must not be migrating, attached, belong to a \u0027"},{"line_number":511,"context_line":"                    \u0027group, have snapshots, awaiting a transfer, \u0027"},{"line_number":512,"context_line":"                    \u0027or be disassociated from \u0027"},{"line_number":513,"context_line":"                    \u0027snapshots after volume transfer.\u0027) % status"},{"line_number":514,"context_line":"            LOG.info(msg)"}],"source_content_type":"text/x-python","patch_set":5,"id":"b29c79fe_2f2ca58d","line":511,"range":{"start_line":511,"start_character":63,"end_line":511,"end_character":64},"updated":"2022-05-06 08:15:58.000000000","message":"nit: remove comma","commit_id":"2b75fa3021e1e6b4d635ab36a9084719c0e3d969"}]}