)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"887fd296d612ffa1fe0336adf5e97e1a8dcceb20","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"feb13aad_13a1c2a9","updated":"2024-02-04 09:21:00.000000000","message":"recheck devstack-plugin-nfs-tempest-full \ntempest.api.identity.admin.v3.test_services.ServicesTestJSON","commit_id":"dd428106f72abdcccae34a417df4229108fecf17"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"311e257e046705ed80afea5048c1419a814396ea","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7bae2c57_482ae830","updated":"2024-02-01 14:41:04.000000000","message":"recheck openstacksdk-functional-devstack cinder-plugin-ceph-tempest","commit_id":"dd428106f72abdcccae34a417df4229108fecf17"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"8ad6ec5346bec214871bbc3b796a85109ac2cca8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a79f86bd_84395726","updated":"2024-02-02 06:14:59.000000000","message":"recheck tempest-integrated-storage","commit_id":"dd428106f72abdcccae34a417df4229108fecf17"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"9820251469d3327a3928be623362fe2950dec8a7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"4809c45c_423f593b","updated":"2024-02-05 13:42:01.000000000","message":"Couple of comments inline.","commit_id":"404c60bb6c70782dc8e5e8e3ab0f88e3ce82090b"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"f2e0b980d6d8199068ab26db256636a40a3437cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"db775b84_6b73df53","updated":"2024-02-05 05:35:16.000000000","message":"run-DellEMC PowerStore CI","commit_id":"404c60bb6c70782dc8e5e8e3ab0f88e3ce82090b"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"830cc7ec1989b0526169d284b652e734b1085261","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"00bd761d_75d172a3","updated":"2024-08-30 06:26:01.000000000","message":"Code and tests looks good, powerstore CI is passing.\nWould be better if someone with active/active knowledge also reviews this to look for possible corner cases.","commit_id":"e661624e0d85ce30cf164b929c7f09de39b5601c"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"734aab1119a61537c3f20debb6a41c06609185d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8285ae40_7bd33a0a","updated":"2024-05-23 17:04:08.000000000","message":"LGTM","commit_id":"e661624e0d85ce30cf164b929c7f09de39b5601c"},{"author":{"_account_id":33612,"name":"yuval","email":"yuval@lightbitslabs.com","username":"yuval"},"change_message_id":"982c46d9054a65ca6b289c9548c1803a369d1d45","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ef3e10a6_5a7efa5c","updated":"2024-07-11 10:51:49.000000000","message":"LGTM","commit_id":"e661624e0d85ce30cf164b929c7f09de39b5601c"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"677fe7ed6d4a765ff9c6902ccc460f31d21c7e1e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7b8f5402_d6bbbe7f","updated":"2024-02-06 10:04:52.000000000","message":"My comments have been addressed. Thanks.","commit_id":"e661624e0d85ce30cf164b929c7f09de39b5601c"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"b92776be1910a22df45acf9380c1c15727969c33","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"415ed6e7_169f38d0","updated":"2024-09-03 16:27:00.000000000","message":"All I did was resolve the merge conflict, which Yian Zong seems OK with, so I think I can still vote on this patch.  (I resolved the conflict to speed things up, and then forgot to come back and look at the patch!)","commit_id":"2160b0e212feb49f6735b7ca9917081668e10a67"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"43d70266f7a18a1059609a928a2ebab9aec45c2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"33b65338_b3358476","updated":"2024-08-30 23:50:50.000000000","message":"Hi @rosmaita.fossdev@gmail.com and @rajatdhasmana@gmail.com,\nThank you very much for reviewing and resolving the merge conflict!","commit_id":"2160b0e212feb49f6735b7ca9917081668e10a67"},{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"c2984397f7e7cafd6978a31f596358e67ac9debc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"467a534c_1795ff00","updated":"2024-09-04 06:38:53.000000000","message":"Hi Yian,\nTo resolve tox pep8 errors, Rajat has submitted a patch:\nhttps://review.opendev.org/c/openstack/cinder/+/927825\n\nIt seems, we will have to wait till patch 927825 gets merged.","commit_id":"2160b0e212feb49f6735b7ca9917081668e10a67"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"3be817cef6c8756b66ac8c22a904ac7fc73bec87","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6ce832de_4260fc89","updated":"2024-09-01 17:09:07.000000000","message":"PowerStore CI is passing.","commit_id":"2160b0e212feb49f6735b7ca9917081668e10a67"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cfa65ef22c9513850a68283cebcf2b33c92d8040","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"990f8066_bfac8ca2","updated":"2024-08-30 20:22:34.000000000","message":"Resolved the merge conflict that happened when the QoS patch merged first.  If all tests pass, I\u0027m willing to carry over Rajat\u0027s +2 to this patch.  As far as A/A goes, our docs [0] say that \u0027it’s “self-certification”. Vendors must do their own testing until they are satisfied with their testing\u0027.\n\n[0] https://docs.openstack.org/cinder/latest/contributor/high_availability.html#enabling-active-active-on-drivers","commit_id":"2160b0e212feb49f6735b7ca9917081668e10a67"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"d07f846d4aea83bc211653f2abebdbf1d301ad30","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0caa8869_1fce9a31","updated":"2024-09-04 16:02:03.000000000","message":"recheck - cinder-tempest-plugin-protection-functional failed.","commit_id":"2160b0e212feb49f6735b7ca9917081668e10a67"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"1cfaccde9f006c0634124cc80b984d834f12e2e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"902df632_798627d4","updated":"2024-09-04 13:41:20.000000000","message":"recheck openstack-tox-pep8 fixed","commit_id":"2160b0e212feb49f6735b7ca9917081668e10a67"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"8618f97d9e56b6fd9cb3826abb4880a16f68108b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"fc51468d_2ded409b","updated":"2024-09-04 06:34:44.000000000","message":"recheck openstack-tox-pep8 seems fixed","commit_id":"2160b0e212feb49f6735b7ca9917081668e10a67"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"235450fdd68fd13cfb6cd640e5c25d4f8979379b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f8504ea2_13bead7c","in_reply_to":"0caa8869_1fce9a31","updated":"2024-09-05 00:52:05.000000000","message":"Thanks, Nilesh!","commit_id":"2160b0e212feb49f6735b7ca9917081668e10a67"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"e7640b4ea3b915e788f3a0c2cb092c5d8bd61611","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"895b6711_abb4768d","in_reply_to":"467a534c_1795ff00","updated":"2024-09-04 06:51:27.000000000","message":"Got it. Thanks, Raghavendra!","commit_id":"2160b0e212feb49f6735b7ca9917081668e10a67"}],"cinder/volume/drivers/dell_emc/powerstore/driver.py":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"9820251469d3327a3928be623362fe2950dec8a7","unresolved":true,"context_lines":[{"line_number":221,"context_line":"            # failover operation"},{"line_number":222,"context_line":"            self.active_backend_id \u003d self.replication_devices[0][\"backend_id\"]"},{"line_number":223,"context_line":"        else:"},{"line_number":224,"context_line":"            msg \u003d f\"Target {active_backend_id} is not valid.\""},{"line_number":225,"context_line":"            LOG.error(msg)"},{"line_number":226,"context_line":"            raise exception.InvalidReplicationTarget(reason\u003dmsg)"},{"line_number":227,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"90bbaefd_b367d9cf","line":224,"updated":"2024-02-05 13:42:01.000000000","message":"Query: Can this msg be formatted same as line 194 above.\nIt would be help during translation.","commit_id":"404c60bb6c70782dc8e5e8e3ab0f88e3ce82090b"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"0ed4309b00cdf6ea66bb77f330431c2220ad2695","unresolved":false,"context_lines":[{"line_number":221,"context_line":"            # failover operation"},{"line_number":222,"context_line":"            self.active_backend_id \u003d self.replication_devices[0][\"backend_id\"]"},{"line_number":223,"context_line":"        else:"},{"line_number":224,"context_line":"            msg \u003d f\"Target {active_backend_id} is not valid.\""},{"line_number":225,"context_line":"            LOG.error(msg)"},{"line_number":226,"context_line":"            raise exception.InvalidReplicationTarget(reason\u003dmsg)"},{"line_number":227,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"e3c77235_5b284aa0","line":224,"in_reply_to":"90bbaefd_b367d9cf","updated":"2024-02-06 08:12:39.000000000","message":"Done","commit_id":"404c60bb6c70782dc8e5e8e3ab0f88e3ce82090b"}],"releasenotes/notes/bp-dell-powerstore-aa-ca7b2e9355a1e5a5.yaml":[{"author":{"_account_id":29122,"name":"Raghavendra Tilay","email":"raghavendra-uddhav.tilay@hpe.com","username":"raghavendrat"},"change_message_id":"9820251469d3327a3928be623362fe2950dec8a7","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell PoweStore driver: Enabled cinder volume active/active support."},{"line_number":5,"context_line":"    This allows users to configure Dell PowerStore backends in cinder"},{"line_number":6,"context_line":"    clustered environments."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"cff88607_4fe15387","line":4,"range":{"start_line":4,"start_character":9,"end_line":4,"end_character":18},"updated":"2024-02-05 13:42:01.000000000","message":"typo: PowerStore","commit_id":"404c60bb6c70782dc8e5e8e3ab0f88e3ce82090b"},{"author":{"_account_id":35759,"name":"Yian Zong","display_name":"Yian Zong","email":"yian.zong@dell.com","username":"yianzong"},"change_message_id":"0ed4309b00cdf6ea66bb77f330431c2220ad2695","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"features:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Dell PoweStore driver: Enabled cinder volume active/active support."},{"line_number":5,"context_line":"    This allows users to configure Dell PowerStore backends in cinder"},{"line_number":6,"context_line":"    clustered environments."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3048d715_689cb290","line":4,"range":{"start_line":4,"start_character":9,"end_line":4,"end_character":18},"in_reply_to":"cff88607_4fe15387","updated":"2024-02-06 08:12:39.000000000","message":"Done","commit_id":"404c60bb6c70782dc8e5e8e3ab0f88e3ce82090b"}]}