)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"6470ef00680f8e97338513f52642190f7d241344","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"48c83b58_8e59bea8","updated":"2026-06-09 05:14:53.000000000","message":"Please do a review.","commit_id":"ea21f6fa172c0faa3f8f7bc317d9bfd513878320"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"16ac5fc50690f761e15a6a8c1f66588157ea726a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"feb47fe5_7bc82a17","updated":"2026-04-30 05:54:39.000000000","message":"run-DellEMC PowerStore CI","commit_id":"ea21f6fa172c0faa3f8f7bc317d9bfd513878320"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"7ec95772d0841631572d7da93bd83a51555de54c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2a0f90bf_9676cffe","updated":"2026-08-28 11:36:15.000000000","message":"Please resolve merge conflicts","commit_id":"55269ffa5f59237eb4d1bf7b1b7cf314264cac1d"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"001def506e0d4ef34da88494740270d942bbb1e0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"076427c5_059f7300","updated":"2026-09-02 06:46:17.000000000","message":"run-DellEMC PowerStore CI","commit_id":"55269ffa5f59237eb4d1bf7b1b7cf314264cac1d"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"ae9d6fc4383f59b9ea7a5f9c5dfa38b8215838a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"49a4e8eb_6d207cc8","updated":"2026-09-02 08:41:12.000000000","message":"run-DellEMC PowerStore CI","commit_id":"55269ffa5f59237eb4d1bf7b1b7cf314264cac1d"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"eab3589ad0c334066daecde2d6bbac9e4da373ec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8c0f7bd8_65cf50b2","updated":"2026-09-02 21:49:23.000000000","message":"run-DellEMC PowerStore CI","commit_id":"55269ffa5f59237eb4d1bf7b1b7cf314264cac1d"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"b078ff204400e060bdcfab31534adfa0e8f70ab4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9a76931b_a4c18665","updated":"2026-09-02 11:54:25.000000000","message":"run-DellEMC PowerStore CI","commit_id":"55269ffa5f59237eb4d1bf7b1b7cf314264cac1d"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"1c8d7e0a91fda7f1bf4c79df9687e25361a9f811","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ab6092a3_6f3093c0","updated":"2026-09-02 11:03:29.000000000","message":"run-DellEMC PowerStore CI","commit_id":"55269ffa5f59237eb4d1bf7b1b7cf314264cac1d"}],"cinder/tests/unit/volume/drivers/dell_emc/powerstore/test_metro_volume.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"864a6dae867d63535e0de8bcd5b8e8f7b39e0192","unresolved":true,"context_lines":[{"line_number":349,"context_line":"        mock_clone.assert_not_called()"},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"    @mock.patch(\"cinder.volume.drivers.dell_emc.powerstore.client.\""},{"line_number":352,"context_line":"                \"PowerStoreClient.get_replication_session_state\")"},{"line_number":353,"context_line":"    @mock.patch(\"cinder.volume.drivers.dell_emc.powerstore.client.\""},{"line_number":354,"context_line":"                \"PowerStoreClient.get_volume_replication_session_id\")"},{"line_number":355,"context_line":"    def test_revert_to_snapshot_not_paused(self,"}],"source_content_type":"text/x-python","patch_set":2,"id":"e377c6bb_e209b191","line":352,"updated":"2026-07-24 19:48:07.000000000","message":"The four tests cover the main paths well (clone + snapshot, supported + unsupported version). One gap: there\u0027s no test for the case where volume.size \u003e source_size on a metro clone. In that scenario the code does clone → extend → configure metro. Since extending metro volumes has special handling, it would be good to verify this sequence with a test where the target volume is larger than the source.","commit_id":"a73937b39fc16a174145d507cf633726602bce91"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"3df974257cc8741a93d8cff87c48b4c4561ca509","unresolved":false,"context_lines":[{"line_number":349,"context_line":"        mock_clone.assert_not_called()"},{"line_number":350,"context_line":""},{"line_number":351,"context_line":"    @mock.patch(\"cinder.volume.drivers.dell_emc.powerstore.client.\""},{"line_number":352,"context_line":"                \"PowerStoreClient.get_replication_session_state\")"},{"line_number":353,"context_line":"    @mock.patch(\"cinder.volume.drivers.dell_emc.powerstore.client.\""},{"line_number":354,"context_line":"                \"PowerStoreClient.get_volume_replication_session_id\")"},{"line_number":355,"context_line":"    def test_revert_to_snapshot_not_paused(self,"}],"source_content_type":"text/x-python","patch_set":2,"id":"89447ffe_84064b67","line":352,"in_reply_to":"e377c6bb_e209b191","updated":"2026-08-04 13:41:39.000000000","message":"Done","commit_id":"a73937b39fc16a174145d507cf633726602bce91"}],"cinder/volume/drivers/dell_emc/powerstore/adapter.py":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"864a6dae867d63535e0de8bcd5b8e8f7b39e0192","unresolved":false,"context_lines":[{"line_number":393,"context_line":"                       % {\"metro_clone_version\":"},{"line_number":394,"context_line":"                          POWERSTORE_METRO_CLONE_VERSION_SUPPORT,"},{"line_number":395,"context_line":"                          \"current_version\": array_version})"},{"line_number":396,"context_line":"                LOG.warning(msg)"},{"line_number":397,"context_line":"                raise exception.InvalidInput(reason\u003dmsg)"},{"line_number":398,"context_line":"        if isinstance(source, Snapshot):"},{"line_number":399,"context_line":"            entity \u003d \"snapshot\""}],"source_content_type":"text/x-python","patch_set":2,"id":"c1ad9436_d6a7f238","line":396,"updated":"2026-07-24 19:48:07.000000000","message":"Minor: The LOG.warning here is redundant — the InvalidInput exception will be caught and logged by the API/manager layer, resulting in the same message appearing in logs twice. Consider removing the LOG.warning. Not a blocker.","commit_id":"a73937b39fc16a174145d507cf633726602bce91"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"864a6dae867d63535e0de8bcd5b8e8f7b39e0192","unresolved":true,"context_lines":[{"line_number":815,"context_line":"            if self.client.get_replication_session_state("},{"line_number":816,"context_line":"                    metro_replication_session_id) not in [\"Paused\","},{"line_number":817,"context_line":"                                                          \"Fractured\"]:"},{"line_number":818,"context_line":"                msg \u003d _(\"Error reverting PowerStore metro \""},{"line_number":819,"context_line":"                        \"volume %(volume_id)s. The replication \""},{"line_number":820,"context_line":"                        \"session %(id)s must be paused in order to \""},{"line_number":821,"context_line":"                        \"revert metro volumes.\" % {"}],"source_content_type":"text/x-python","patch_set":2,"id":"5e06cd98_66ed9330","line":818,"updated":"2026-07-24 19:48:07.000000000","message":"Good catch fixing the error message from \"extending\" to \"reverting\" — this was clearly a copy-paste bug from the extend code.\n\nHowever, two things:\n\n1. This fix is unrelated to the metro clone feature. Consider splitting it into a separate commit for clean git history (helps with bisection and cherry-picks).\n\n2. Pre-existing i18n bug: the % string substitution happens *before* _(), so the translation function receives an already-interpolated string that will never match a translation catalog entry.\n\nCurrent (wrong):\n  msg \u003d _(\"Error reverting ... %(volume_id)s ...\" % {\"volume_id\": volume.id, ...})\n\nCorrect:\n  msg \u003d _(\"Error reverting ... %(volume_id)s ...\") % {\"volume_id\": volume.id, ...}\n\nSince you\u0027re already touching this line, it would be good to fix the i18n pattern too.","commit_id":"a73937b39fc16a174145d507cf633726602bce91"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"3df974257cc8741a93d8cff87c48b4c4561ca509","unresolved":false,"context_lines":[{"line_number":815,"context_line":"            if self.client.get_replication_session_state("},{"line_number":816,"context_line":"                    metro_replication_session_id) not in [\"Paused\","},{"line_number":817,"context_line":"                                                          \"Fractured\"]:"},{"line_number":818,"context_line":"                msg \u003d _(\"Error reverting PowerStore metro \""},{"line_number":819,"context_line":"                        \"volume %(volume_id)s. The replication \""},{"line_number":820,"context_line":"                        \"session %(id)s must be paused in order to \""},{"line_number":821,"context_line":"                        \"revert metro volumes.\" % {"}],"source_content_type":"text/x-python","patch_set":2,"id":"003f7c5b_658e3a98","line":818,"in_reply_to":"5e06cd98_66ed9330","updated":"2026-08-04 13:41:39.000000000","message":"Done","commit_id":"a73937b39fc16a174145d507cf633726602bce91"}],"releasenotes/notes/bp-dell-powerstore-clone-metro-volume-964b6590c4556465.yaml":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"cfa3537e29a4ff82e00a5de040b1cc02a6956743","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 PowerStore driver: Added support for clone metro volume."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"bdea167c_92536e7c","line":4,"updated":"2026-05-11 15:33:40.000000000","message":"can you elaborate about feature?","commit_id":"ea21f6fa172c0faa3f8f7bc317d9bfd513878320"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"e8f10317e4c25d4b93d2c71fcb6e9a227e002407","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 PowerStore driver: Added support for clone metro volume."}],"source_content_type":"text/x-yaml","patch_set":1,"id":"4e26d918_55e49e72","line":4,"in_reply_to":"bdea167c_92536e7c","updated":"2026-07-24 05:54:45.000000000","message":"Done","commit_id":"ea21f6fa172c0faa3f8f7bc317d9bfd513878320"}]}