)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"1d9d2df2b9130e99ab04231853559e665cb985f6","unresolved":true,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"The deadlock happens because A-to-B waits forever for the lock held by"},{"line_number":35,"context_line":"the B-to-A operation, which in turn cannot proceed because it’s waiting"},{"line_number":36,"context_line":"for lock help by A-to-B."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"This patch fixes it using the new coordination.synchronized"},{"line_number":39,"context_line":"functionality that ensures that a series of locks are always acquired in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"4c69183e_2aea404e","line":36,"range":{"start_line":36,"start_character":9,"end_line":36,"end_character":13},"updated":"2023-05-23 08:57:54.000000000","message":"nit: held","commit_id":"1852eb0f412a8592f434fd1a3f31a0b0cba9cce1"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"86be75d5ea1eda68ed9c81cd496398e9faa02f5d","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"The deadlock happens because A-to-B waits forever for the lock held by"},{"line_number":35,"context_line":"the B-to-A operation, which in turn cannot proceed because it’s waiting"},{"line_number":36,"context_line":"for lock help by A-to-B."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"This patch fixes it using the new coordination.synchronized"},{"line_number":39,"context_line":"functionality that ensures that a series of locks are always acquired in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"ebfbaf59_cb0357b7","line":36,"range":{"start_line":36,"start_character":9,"end_line":36,"end_character":13},"in_reply_to":"4c69183e_2aea404e","updated":"2023-06-28 11:06:15.000000000","message":"Done","commit_id":"1852eb0f412a8592f434fd1a3f31a0b0cba9cce1"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"41a40d364493efbaa3c01e2683b6de2c71c74620","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9f5ee011_baa3e356","updated":"2022-07-07 08:48:37.000000000","message":"PowerMax CI job failure seems unrelated to this patch since it\u0027s also failing in the parent patch that doesn\u0027t touch the driver.\n\nThe failure is in starting the cinder-volume service itself as can be seen at the end of https://elab-os-logsrv.delllabs.net/00/848900/2/check/DellEMC_PowerMAX_FC/14d0321/DellEMC_PowerMAX_FC/683/logs/devstack-early.txt","commit_id":"52682e041c86a7b9f4b0af075799aafc44a1bd17"},{"author":{"_account_id":34290,"name":"Alexander Malashenko","email":"alexander.malashenko@dell.com","username":"amalashenko"},"change_message_id":"ca9f91b8803a80534165ccb31cb881e4e7eeaf51","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2407ea43_65b64fab","updated":"2022-07-12 11:26:26.000000000","message":"Looks good!","commit_id":"5a8bd158c47eedc4bc844904d759120d3af10803"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"dc2917cd5529f53fa4633fb16489e5ae17f89c75","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"230e78c8_13f5ff50","updated":"2022-07-20 19:58:47.000000000","message":"code looks good, CI is fine.","commit_id":"5a8bd158c47eedc4bc844904d759120d3af10803"},{"author":{"_account_id":33502,"name":"Oleg","display_name":"Oleg Nesterenkov","email":"Oleg_Nesterenkov@Dell.com","username":"olegnest"},"change_message_id":"c015853eef24e3c0ea54c99e37b7464adba1ddd4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0f73a904_d8da9dc8","updated":"2022-07-14 09:26:17.000000000","message":"recheck","commit_id":"5a8bd158c47eedc4bc844904d759120d3af10803"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"068b3de5014c521f4a3af0b9fddb3e64e97f7aa0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ddb52192_f3905233","updated":"2022-07-14 11:22:30.000000000","message":"recheck - tempest-slow-py3 fails with unrelated POST_FAILURE","commit_id":"5a8bd158c47eedc4bc844904d759120d3af10803"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"0831b32ff23648814012906c6d9c62bb1795791d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"401e6d75_839484a8","updated":"2022-07-21 18:51:55.000000000","message":"run-DellEMC PowerMAX CI","commit_id":"5a8bd158c47eedc4bc844904d759120d3af10803"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"f9a5a982fe4e04b99a41861d241d30fe87321293","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"15353b36_d4a79fed","updated":"2022-12-19 10:42:22.000000000","message":"Can you rebase your patch? Tried to do it for you, but gerrit rejected it...","commit_id":"4e6cbf1096f25e2c50178a7d2e1082d074b29111"},{"author":{"_account_id":33473,"name":"Atsushi Kawai","display_name":"akawai","email":"atsushi.kawai.bu@hitachi.com","username":"akawai"},"change_message_id":"cef577f7acce08f3b0b6d0977d997fec04e84437","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"37600f39_f4bdafbb","updated":"2023-01-19 11:36:18.000000000","message":"LGTM","commit_id":"1852eb0f412a8592f434fd1a3f31a0b0cba9cce1"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"1d9d2df2b9130e99ab04231853559e665cb985f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"06c41798_bc5a44be","updated":"2023-05-23 08:57:54.000000000","message":"LGTM.","commit_id":"1852eb0f412a8592f434fd1a3f31a0b0cba9cce1"},{"author":{"_account_id":35075,"name":"Alexander Deiter","email":"adeiter@infinidat.com","username":"adeiter"},"change_message_id":"dc5b7d33d4da0b0fe58ec976a534e340164524ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"bfb98665_29400153","updated":"2023-05-19 14:27:42.000000000","message":"Looks good to me - thank you!","commit_id":"1852eb0f412a8592f434fd1a3f31a0b0cba9cce1"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"17479a10bacbfe018fcaa5ac22471e9ad1ada14f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8398ca80_5a3119a1","updated":"2023-01-19 14:23:41.000000000","message":"lgtm!","commit_id":"1852eb0f412a8592f434fd1a3f31a0b0cba9cce1"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"bd89ce6c88e0eeaa1a85d9f289c1f4f0ca3854d2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d678d43d_127fbb16","updated":"2023-01-20 15:39:51.000000000","message":"thanks for the fix!","commit_id":"1852eb0f412a8592f434fd1a3f31a0b0cba9cce1"},{"author":{"_account_id":33473,"name":"Atsushi Kawai","display_name":"akawai","email":"atsushi.kawai.bu@hitachi.com","username":"akawai"},"change_message_id":"8335657d8dae5265a6209d398414baed2d37a6ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"4d664920_563227c3","updated":"2023-07-03 10:27:35.000000000","message":"LGTM","commit_id":"78407657ea9bf72aeab6b63958a2976ef25292f5"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"2fb265ec82ee730ddcd3f9fcbf4dec81abb7aa69","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5700fd9e_e95217dd","updated":"2023-06-28 11:48:46.000000000","message":"LGTM, thanks!","commit_id":"78407657ea9bf72aeab6b63958a2976ef25292f5"},{"author":{"_account_id":35063,"name":"Tony Saad","email":"tony.saad@dell.com","username":"tony-saad"},"change_message_id":"6693c28cdb29413eb48577d3a409c3cb4c8efe22","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"6c1e0ad4_faed4ed8","updated":"2023-06-29 15:59:18.000000000","message":"run-DellEMC PowerMAX CI","commit_id":"78407657ea9bf72aeab6b63958a2976ef25292f5"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"229879959b6269bff7c766d860585364dc5bed10","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"ddb5c875_f737fc10","updated":"2023-12-15 20:55:14.000000000","message":"There haven\u0027t been any substantive changes after the point where this patch had two +2s and a +W, so I feel OK about ninja-approving it now.","commit_id":"411852892d803b606110d0956a59764925c16ec6"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"c4340ab82e03d630069b9c9bf0d4da39ebcd12a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"e1895449_971c5484","updated":"2023-09-12 13:26:08.000000000","message":"recheck","commit_id":"411852892d803b606110d0956a59764925c16ec6"}],"cinder/volume/drivers/dell_emc/powermax/masking.py":[{"author":{"_account_id":5997,"name":"Walt","display_name":"Hemna","email":"waboring@hemna.com","username":"walter-boring","status":"SAP"},"change_message_id":"d6f9ac492a4962c2128b152382d1cc4cde1ae577","unresolved":true,"context_lines":[{"line_number":643,"context_line":""},{"line_number":644,"context_line":"    @coordination.synchronized("},{"line_number":645,"context_line":"        \"emc-sg-{source_storagegroup_name}-{serial_number}\","},{"line_number":646,"context_line":"        \"emc-sg-{target_storagegroup_name}-{serial_number}\")"},{"line_number":647,"context_line":"    def move_volume_between_storage_groups("},{"line_number":648,"context_line":"            self, serial_number, device_id, source_storagegroup_name,"},{"line_number":649,"context_line":"            target_storagegroup_name, extra_specs, force\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":2,"id":"e790c4bc_b9ee9fb2","line":646,"updated":"2022-07-06 20:28:23.000000000","message":"why are there 2 of the same exact locks?!","commit_id":"52682e041c86a7b9f4b0af075799aafc44a1bd17"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"b9a91214ffaf1e404f7722243d705980a686b593","unresolved":false,"context_lines":[{"line_number":643,"context_line":""},{"line_number":644,"context_line":"    @coordination.synchronized("},{"line_number":645,"context_line":"        \"emc-sg-{source_storagegroup_name}-{serial_number}\","},{"line_number":646,"context_line":"        \"emc-sg-{target_storagegroup_name}-{serial_number}\")"},{"line_number":647,"context_line":"    def move_volume_between_storage_groups("},{"line_number":648,"context_line":"            self, serial_number, device_id, source_storagegroup_name,"},{"line_number":649,"context_line":"            target_storagegroup_name, extra_specs, force\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":2,"id":"d9b5aa74_25108f10","line":646,"in_reply_to":"e790c4bc_b9ee9fb2","updated":"2022-07-07 08:44:29.000000000","message":"They are not the same, one is \"source_...\" and the other \"target_...\"","commit_id":"52682e041c86a7b9f4b0af075799aafc44a1bd17"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"f02365dc13ec169cd06fc5e978e0e2f8a6c925ed","unresolved":true,"context_lines":[{"line_number":643,"context_line":""},{"line_number":644,"context_line":"    @coordination.synchronized("},{"line_number":645,"context_line":"        \"emc-sg-{source_storagegroup_name}-{serial_number}\","},{"line_number":646,"context_line":"        \"emc-sg-{target_storagegroup_name}-{serial_number}\")"},{"line_number":647,"context_line":"    def move_volume_between_storage_groups("},{"line_number":648,"context_line":"            self, serial_number, device_id, source_storagegroup_name,"},{"line_number":649,"context_line":"            target_storagegroup_name, extra_specs, force\u003dFalse,"}],"source_content_type":"text/x-python","patch_set":2,"id":"fc39291a_f4a1ba01","line":646,"in_reply_to":"e790c4bc_b9ee9fb2","updated":"2022-07-07 08:43:57.000000000","message":"Those 2 are different, one is for the source, the second is for the target.","commit_id":"52682e041c86a7b9f4b0af075799aafc44a1bd17"}],"releasenotes/notes/powermax-deadlock-5fdcacb63ca87159.yaml":[{"author":{"_account_id":33502,"name":"Oleg","display_name":"Oleg Nesterenkov","email":"Oleg_Nesterenkov@Dell.com","username":"olegnest"},"change_message_id":"c1c16faca1b6f8c36a23f6c6eacc98528d52415a","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell EMC PowerMax driver `bug #1980870"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/1980870\u003e`_: Fixed potential"},{"line_number":6,"context_line":"    deadlock when moving volumes between Storage Group.s"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"e4e6044a_e5e42256","line":6,"range":{"start_line":6,"start_character":54,"end_line":6,"end_character":56},"updated":"2022-07-07 14:25:14.000000000","message":"Mistake here.","commit_id":"52682e041c86a7b9f4b0af075799aafc44a1bd17"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"6188bfca82734e19756d080e5d734560e7222a99","unresolved":true,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell EMC PowerMax driver `bug #1980870"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/1980870\u003e`_: Fixed potential"},{"line_number":6,"context_line":"    deadlock when moving volumes between Storage Group.s"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b162d519_a6f631ac","line":6,"updated":"2022-07-07 14:24:43.000000000","message":"nit: remove the extra dot in Group.s","commit_id":"52682e041c86a7b9f4b0af075799aafc44a1bd17"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"068b3de5014c521f4a3af0b9fddb3e64e97f7aa0","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell EMC PowerMax driver `bug #1980870"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/1980870\u003e`_: Fixed potential"},{"line_number":6,"context_line":"    deadlock when moving volumes between Storage Group.s"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b62fab51_e5a40ea4","line":6,"in_reply_to":"b162d519_a6f631ac","updated":"2022-07-14 11:22:30.000000000","message":"Done","commit_id":"52682e041c86a7b9f4b0af075799aafc44a1bd17"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"068b3de5014c521f4a3af0b9fddb3e64e97f7aa0","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell EMC PowerMax driver `bug #1980870"},{"line_number":5,"context_line":"    \u003chttps://bugs.launchpad.net/cinder/+bug/1980870\u003e`_: Fixed potential"},{"line_number":6,"context_line":"    deadlock when moving volumes between Storage Group.s"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"8a028e71_b01a7153","line":6,"range":{"start_line":6,"start_character":54,"end_line":6,"end_character":56},"in_reply_to":"e4e6044a_e5e42256","updated":"2022-07-14 11:22:30.000000000","message":"Done","commit_id":"52682e041c86a7b9f4b0af075799aafc44a1bd17"}]}