)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"599d5167ba4b19d7153ceda2777745b88a747953","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"1. Check that both sides of the RDF pair are in the same storage group"},{"line_number":10,"context_line":"   on a delete."},{"line_number":11,"context_line":"2. Wait if a metro or async rdf group is not fully synched in certain"},{"line_number":12,"context_line":"   operations (create, extend and migrate)."},{"line_number":13,"context_line":"3. Allow for a clean delete when creates fail."},{"line_number":14,"context_line":"4. Catch RDF exceptions in delete and rollback code to see if cleanup"},{"line_number":15,"context_line":"   can proceed further."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"3625ee8a_ae840160","line":12,"range":{"start_line":11,"start_character":3,"end_line":12,"end_character":43},"updated":"2022-05-27 14:47:57.000000000","message":"assuming the rdf group facilitates replication (which i could be wrong about), did we have an issue regarding the volumes in source and target groups being out of sync?","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"603beac0c58eefe21fc2c9ed686d08e1da259025","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"1. Check that both sides of the RDF pair are in the same storage group"},{"line_number":10,"context_line":"   on a delete."},{"line_number":11,"context_line":"2. Wait if a metro or async rdf group is not fully synched in certain"},{"line_number":12,"context_line":"   operations (create, extend and migrate)."},{"line_number":13,"context_line":"3. Allow for a clean delete when creates fail."},{"line_number":14,"context_line":"4. Catch RDF exceptions in delete and rollback code to see if cleanup"},{"line_number":15,"context_line":"   can proceed further."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"6735b5a8_40a287a9","line":12,"range":{"start_line":11,"start_character":3,"end_line":12,"end_character":43},"in_reply_to":"1c8734ca_622d6788","updated":"2022-12-19 10:16:43.000000000","message":"Done","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"b463ce169e6729ed1e76af1b4cae7bbff80a1960","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"1. Check that both sides of the RDF pair are in the same storage group"},{"line_number":10,"context_line":"   on a delete."},{"line_number":11,"context_line":"2. Wait if a metro or async rdf group is not fully synched in certain"},{"line_number":12,"context_line":"   operations (create, extend and migrate)."},{"line_number":13,"context_line":"3. Allow for a clean delete when creates fail."},{"line_number":14,"context_line":"4. Catch RDF exceptions in delete and rollback code to see if cleanup"},{"line_number":15,"context_line":"   can proceed further."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"1c8734ca_622d6788","line":12,"range":{"start_line":11,"start_character":3,"end_line":12,"end_character":43},"in_reply_to":"3625ee8a_ae840160","updated":"2022-09-07 08:37:34.000000000","message":"In a RDF pair relationship, both source and target need to be in sync. Although we\u0027re making any disaster recovery testing, it\u0027s the expected status.","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"599d5167ba4b19d7153ceda2777745b88a747953","unresolved":true,"context_lines":[{"line_number":10,"context_line":"   on a delete."},{"line_number":11,"context_line":"2. Wait if a metro or async rdf group is not fully synched in certain"},{"line_number":12,"context_line":"   operations (create, extend and migrate)."},{"line_number":13,"context_line":"3. Allow for a clean delete when creates fail."},{"line_number":14,"context_line":"4. Catch RDF exceptions in delete and rollback code to see if cleanup"},{"line_number":15,"context_line":"   can proceed further."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":12,"id":"02c6934d_cfe4df9b","line":13,"range":{"start_line":13,"start_character":3,"end_line":13,"end_character":46},"updated":"2022-05-27 14:47:57.000000000","message":"wasn\u0027t this the functionality before? did users had to manually remove the volume/LUN after a create request fails?","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ffdac519014c8fd85f1ea5a32b4febbd6bd2162b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"1fc12e3c_bf0bf9e9","updated":"2022-05-27 13:24:26.000000000","message":"-1: This is a pretty big change in terms of LOC, and from what\u0027s being changed, it sounds like your customers would want to know if their driver code has this fix or not.  So there should be a bug tracking this change.  If there is one, please add it to the commit message; if there isn\u0027t, please file one and add it to the commit message.  May be worth adding a release note, too.\n\nCode and tests look good, and the third-party CI is green, though I don\u0027t think there are any tempest tests covering this change.  I assume you\u0027ve tested it carefully locally.\n\n-1 is for the lack of bug/release note; otherwise the patch LGTM.","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"},{"author":{"_account_id":34298,"name":"Marco Li","email":"liijuun@163.com"},"change_message_id":"99b486db9d2487be1e78def0f9045af46b6b8cda","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"7474a606_0f72d806","updated":"2022-04-24 02:29:01.000000000","message":"run-DellEMC PowerMAX CI","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"},{"author":{"_account_id":34298,"name":"Marco Li","email":"liijuun@163.com"},"change_message_id":"464d7f2e2cd7bcee2444e9d76e5f98b7aa1cd6d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"88ef7a07_5bd45a85","updated":"2022-04-22 02:11:27.000000000","message":"run-DellEMC PowerMAX CI","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"},{"author":{"_account_id":34298,"name":"Marco Li","email":"liijuun@163.com"},"change_message_id":"d1a36d92a0927c7ef62ca4e89708c0a0d464f7b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"a963aff4_95efe0f3","updated":"2022-04-25 06:08:06.000000000","message":"run-DellEMC PowerMAX CI","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"ae9a54df49e8a24fcc283d9d834537d9de64ec3d","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":12,"id":"2997da1b_bcdac8dc","in_reply_to":"1abf3630_2c252185","updated":"2023-03-01 14:53:29.000000000","message":"Are any plans to proposed a tempest tests as a follow up?","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"599d5167ba4b19d7153ceda2777745b88a747953","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"1abf3630_2c252185","in_reply_to":"1fc12e3c_bf0bf9e9","updated":"2022-05-27 14:47:57.000000000","message":"I agree with Brian here. With the set of changes introduced, the description in the commit message seems incomplete and does not specify why these changes are introduced. I\u0027m not super familiar with PowerMax terminologies but the changes seems to be made based on some issue that a customer would have faced. If True, we need a proper bug report and if there are multiple issues, it\u0027s better to fix them in different patches with different releasenotes so it\u0027s easy to track and easier to look for relevant fixes in releasenotes.","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"fef1bcaaaf5b2ae5bf1fbdbbc92014c9046d36c9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":12,"id":"4f807199_b71fafc9","in_reply_to":"2997da1b_bcdac8dc","updated":"2023-03-03 07:58:53.000000000","message":"We can take it to our backlog work and work on it.","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"0921c225f70ce5989046853465c4d7e28f704ea3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"f9b327b8_c12fe695","updated":"2022-06-29 00:14:36.000000000","message":"The CI failures are related to this work. Please update. Thanks!","commit_id":"0e41434ada3f25c656a10aba9dd90b885d140da8"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"8f8d9011b2825401b29c45df61db285504b22065","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"fe6ed369_e3db1e9d","updated":"2022-06-29 07:18:19.000000000","message":"recheck","commit_id":"0e41434ada3f25c656a10aba9dd90b885d140da8"},{"author":{"_account_id":34298,"name":"Marco Li","email":"liijuun@163.com"},"change_message_id":"62e358ad6945e13b482dcb37cf95870bfdcdeb42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"32c2d42d_8a71ef64","updated":"2022-06-29 06:00:35.000000000","message":"run-DellEMC PowerMAX CI","commit_id":"0e41434ada3f25c656a10aba9dd90b885d140da8"},{"author":{"_account_id":33502,"name":"Oleg","display_name":"Oleg Nesterenkov","email":"Oleg_Nesterenkov@Dell.com","username":"olegnest"},"change_message_id":"540fe80385ab95f97e457383eabb3f903b881886","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"1ea134f4_67058c65","in_reply_to":"f9b327b8_c12fe695","updated":"2022-06-29 17:42:23.000000000","message":"Thanks for a comment! Done.","commit_id":"0e41434ada3f25c656a10aba9dd90b885d140da8"},{"author":{"_account_id":33502,"name":"Oleg","display_name":"Oleg Nesterenkov","email":"Oleg_Nesterenkov@Dell.com","username":"olegnest"},"change_message_id":"b60e1f88c39e95530eec351dfa4cdfcc6553dc9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"bdc14e0a_e8dfc38a","updated":"2022-06-29 08:43:49.000000000","message":"recheck","commit_id":"8f20984efa0d46fad5b75a184f3546b642832943"},{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"ae9a54df49e8a24fcc283d9d834537d9de64ec3d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"cf58218e_60d9936c","updated":"2023-03-01 14:53:29.000000000","message":"Code had been updated and CI passed. One question in line:","commit_id":"a9b94b67f5b5a14e241b357937b66ecf3624c363"},{"author":{"_account_id":34290,"name":"Alexander Malashenko","email":"alexander.malashenko@dell.com","username":"amalashenko"},"change_message_id":"3e5aa50fdb99818b1df64f45f180135297280a4c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"cd38bb49_3ab501ac","updated":"2022-06-29 11:48:07.000000000","message":"LGTM!","commit_id":"a9b94b67f5b5a14e241b357937b66ecf3624c363"},{"author":{"_account_id":33502,"name":"Oleg","display_name":"Oleg Nesterenkov","email":"Oleg_Nesterenkov@Dell.com","username":"olegnest"},"change_message_id":"7ae6c333ec483644942cef3fd02d4b927aee104b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"920931f4_9d7adc62","updated":"2022-06-29 14:01:51.000000000","message":"run-DellEMC PowerMAX CI","commit_id":"a9b94b67f5b5a14e241b357937b66ecf3624c363"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"9e6cdd005be18e46adbbb00805670a0891af520e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":16,"id":"c26cd483_28103219","updated":"2023-01-04 14:05:46.000000000","message":"run-DellEMC PowerMAX CI","commit_id":"a9b94b67f5b5a14e241b357937b66ecf3624c363"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"d8743581df2af4215d5592fae96a57fa54cc3da3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"041f4e83_ec9487d2","updated":"2023-03-02 19:47:32.000000000","message":"recheck","commit_id":"c6d804b78ed77af92426aefcae7528f8006c2f70"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"3009a743451f630d222e1de798d1887179653e8c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"cc6018ee_47751419","updated":"2023-03-03 07:55:27.000000000","message":"recheck tempest-integrated-storage","commit_id":"c6d804b78ed77af92426aefcae7528f8006c2f70"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"6ccc00e97e4c0322647d559a2fca7e17199ba75c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"a6875d13_67195b57","updated":"2023-03-02 14:20:00.000000000","message":"recheck tempest-integrated-storage failure","commit_id":"c6d804b78ed77af92426aefcae7528f8006c2f70"}],"cinder/volume/drivers/dell_emc/powermax/common.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ffdac519014c8fd85f1ea5a32b4febbd6bd2162b","unresolved":true,"context_lines":[{"line_number":2352,"context_line":"                        LOG.debug(\u0027Waiting for RDFG %(rdfg_no)s to finishing \u0027"},{"line_number":2353,"context_line":"                                  \u0027synchronising.\u0027,"},{"line_number":2354,"context_line":"                                  {\u0027rdfg_no\u0027: rdfg_no})"},{"line_number":2355,"context_line":"                        self.rest.wait_for_rdf_group_sync("},{"line_number":2356,"context_line":"                            array, mgmt_sg_name, rdfg_no, rep_extra_specs)"},{"line_number":2357,"context_line":"                    return True, mgmt_sg_name"},{"line_number":2358,"context_line":"                else:"}],"source_content_type":"text/x-python","patch_set":12,"id":"80d9a4be_02a67a29","line":2355,"range":{"start_line":2355,"start_character":24,"end_line":2355,"end_character":57},"updated":"2022-05-27 13:24:26.000000000","message":"Took me a while to find it, but here\u0027s the wait.","commit_id":"0afadb3effe7c974844ed019d2e52728e4c3f90f"}],"releasenotes/notes/powermax-rdf-checks-c0420d99e22e79df.yaml":[{"author":{"_account_id":20813,"name":"Sofia Enriquez","email":"lsofia.enriquez@gmail.com","username":"enriquetaso"},"change_message_id":"ae9a54df49e8a24fcc283d9d834537d9de64ec3d","unresolved":true,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    `Bug #1979667 \u003chttps://bugs.launchpad.net/cinder/+bug/1979667\u003e`_: Fixed"},{"line_number":5,"context_line":"    creating async bootable volumes by adding validation in RDF groups.    "}],"source_content_type":"text/x-yaml","patch_set":16,"id":"5e93aae6_e03ded6a","line":5,"range":{"start_line":5,"start_character":71,"end_line":5,"end_character":75},"updated":"2023-03-01 14:53:29.000000000","message":":nit: whitespace","commit_id":"a9b94b67f5b5a14e241b357937b66ecf3624c363"},{"author":{"_account_id":35063,"name":"Tony Saad","email":"tony.saad@dell.com","username":"tony-saad"},"change_message_id":"947fe208763fe52148af7e18d5e640e33b0ac964","unresolved":false,"context_lines":[{"line_number":2,"context_line":"fixes:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    `Bug #1979667 \u003chttps://bugs.launchpad.net/cinder/+bug/1979667\u003e`_: Fixed"},{"line_number":5,"context_line":"    creating async bootable volumes by adding validation in RDF groups.    "}],"source_content_type":"text/x-yaml","patch_set":16,"id":"fc6adac8_07e27722","line":5,"range":{"start_line":5,"start_character":71,"end_line":5,"end_character":75},"in_reply_to":"5e93aae6_e03ded6a","updated":"2023-03-01 20:52:24.000000000","message":"Done","commit_id":"a9b94b67f5b5a14e241b357937b66ecf3624c363"}]}