)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"7c712089443d45583492668fe3c51f538df868ba","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9c0df90d_df6700cc","updated":"2026-01-28 23:33:52.000000000","message":"Prasant, can you take a look at the Zuul failures and either update the code or request a recheck?","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":38284,"name":"Prasant Padhi","display_name":"Prasant Padhi","email":"prasant.padhi@dell.com","username":"prasant.padhi@dell.com"},"change_message_id":"5bfbc72f8bc9f32e271a26bf67aef26cade8d7b7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"a48193cd_af99066d","updated":"2026-02-12 04:56:27.000000000","message":"recheck","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":38284,"name":"Prasant Padhi","display_name":"Prasant Padhi","email":"prasant.padhi@dell.com","username":"prasant.padhi@dell.com"},"change_message_id":"2fde86f160aededd8575d831cc593b1b758e2705","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"b59c5339_a6c57d52","updated":"2026-01-19 07:24:34.000000000","message":"retry","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"9926b316c6ff0d4afbb78526a602aa49be67ee71","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"07ce11b6_5d45c29d","updated":"2026-02-10 04:55:03.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"9b1c18ae78b240bfe33d480804afd2214ac5c5cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"0af100f0_a90649bc","updated":"2026-02-14 05:48:39.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":38284,"name":"Prasant Padhi","display_name":"Prasant Padhi","email":"prasant.padhi@dell.com","username":"prasant.padhi@dell.com"},"change_message_id":"1ed47e1bcb10cb2e6a5bb86528019e7862cf92a3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"222b0d76_47bb1b86","updated":"2026-02-03 10:19:26.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"628e5ccc675ebd77cddcfb8779f6532a057bc20b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"43b3a462_f4c2a93e","updated":"2026-02-11 12:28:08.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"2ce01a026b1232b90975ecf3e7e61f2e87533429","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"4574405f_8bc75936","updated":"2026-02-12 11:53:13.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"432dc4d696923897c5205207bd34033a69832918","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"79c900b6_bc04c71f","updated":"2026-02-13 17:58:58.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"d0e5ef109d880e8387a084d5471bca1e5d87afd5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"83d0aac2_7237b7cb","updated":"2026-02-13 16:09:44.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":38284,"name":"Prasant Padhi","display_name":"Prasant Padhi","email":"prasant.padhi@dell.com","username":"prasant.padhi@dell.com"},"change_message_id":"2433f4420142c4d5629f75dacf9d11fe23ba2922","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"887fccb1_8aca8091","updated":"2026-01-19 09:23:43.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"484cd16f02be90cc7c46c7111391d6be5f724e41","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"8db36576_2dab897b","updated":"2026-01-22 05:54:39.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":38284,"name":"Prasant Padhi","display_name":"Prasant Padhi","email":"prasant.padhi@dell.com","username":"prasant.padhi@dell.com"},"change_message_id":"42f2bd5349fbb0dd32632bfe4002916963e2c9ef","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9dd8ff8a_9cdfc8dd","updated":"2026-02-09 08:06:55.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"1b78c71ba6c99c0baecdaf4c3a240d18882cd1db","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"aec2c530_cb605416","updated":"2026-02-13 14:29:01.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":38284,"name":"Prasant Padhi","display_name":"Prasant Padhi","email":"prasant.padhi@dell.com","username":"prasant.padhi@dell.com"},"change_message_id":"fd3c017d644908f69fe347c946450a920455c17e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cebed1d3_7f4c8a03","updated":"2026-02-12 06:55:02.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":38284,"name":"Prasant Padhi","display_name":"Prasant Padhi","email":"prasant.padhi@dell.com","username":"prasant.padhi@dell.com"},"change_message_id":"c738625dc39135be72907ec45f394d462129150d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"cfcbb79d_bc95a621","updated":"2026-02-03 13:54:44.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":37784,"name":"Abhishek Gupta","display_name":"Abhishek Gupta","email":"Abhishek_Gupta14@Dell.com","username":"abhishek-gupta14"},"change_message_id":"0a959cfa05b9ab0f796df4764d2389c926d5290d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d25556a0_70aa86f2","updated":"2026-01-19 07:25:48.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":37784,"name":"Abhishek Gupta","display_name":"Abhishek Gupta","email":"Abhishek_Gupta14@Dell.com","username":"abhishek-gupta14"},"change_message_id":"e20eabb0c9698397a4511c90f4a87b78d08985eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d298c47e_a81d3c1e","updated":"2026-01-19 19:38:31.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":38284,"name":"Prasant Padhi","display_name":"Prasant Padhi","email":"prasant.padhi@dell.com","username":"prasant.padhi@dell.com"},"change_message_id":"4fd47d2a1fe795d4604a18da902c0fe23fc20a41","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d4fdb329_458dd889","updated":"2026-02-10 05:29:17.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":37792,"name":"Dell Openstack CI","display_name":"Dell Openstack CI","email":"dellopenstackci@dell.com","username":"dellopenstackci"},"change_message_id":"f9eda3e1e5f0a10d98be65970c109edfeeed024c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"e1f93e3d_e3664120","updated":"2026-02-09 09:37:46.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":36725,"name":"Nilesh Thathagar","display_name":"Nilesh Thathagar","email":"nilesh.thathagar@dell.com","username":"NileshT"},"change_message_id":"adb5ade2c03689a59db64f8658daca032257326e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"f27d073b_77e8c5aa","updated":"2026-02-12 10:05:59.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":38284,"name":"Prasant Padhi","display_name":"Prasant Padhi","email":"prasant.padhi@dell.com","username":"prasant.padhi@dell.com"},"change_message_id":"77cf0dc0a4142f054903cb35162bdcafab530cb8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"fba023c3_e0473d7b","updated":"2026-02-03 07:03:46.000000000","message":"run-DellEMC PowerStore CI","commit_id":"e55599db68a27369de0cf787a5aad4bc25f701c0"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"bffa085a0e4b6672bfffb16735b3377d1d361434","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"4b8113eb_2c3aa3da","updated":"2026-04-13 08:21:54.000000000","message":"Please resolve PEP issues and resubmit","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"6bba019c2d670bfe074baab0fa48276913a8bee3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"14a313e1_372f7cf9","updated":"2026-08-28 11:36:24.000000000","message":"Please resolve merge conflicts","commit_id":"8cb1e241e3f57166289bed9f2dc09a323e4e4bdd"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"6124f4222da9834809ef8c0f615a1c0993de9efb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"2f83dbbd_76a1fdd5","updated":"2026-09-08 06:09:43.000000000","message":"leaving a general note rather then repeating inline\n\nreading through this, the patch looks like generated code that hasn\u0027t had a careful human pass afterwards - patch this size needs a author to go back over it and check more carefully.\n\nthis shows up most in the tests, several of them repeat the same thing and some mock so much that the code they are meant to test never runs - best to have second look on tests.\n\nnot asking for a rewrite, but I think this needs a pass by hand before it\u0027s\nworth more review time from community. \nhappy to look again once that\u0027s done. preferably as small patches - split it into series if that helps.","commit_id":"8cb1e241e3f57166289bed9f2dc09a323e4e4bdd"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"b25186e82bf6f723a418c2107d29c134d8752f4a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"8b30e785_f5a73224","updated":"2026-09-04 01:32:18.000000000","message":"run-DellEMC PowerStore CI","commit_id":"8cb1e241e3f57166289bed9f2dc09a323e4e4bdd"}],"cinder/opts.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"6124f4222da9834809ef8c0f615a1c0993de9efb","unresolved":true,"context_lines":[{"line_number":331,"context_line":"                cinder_zonemanager_drivers_cisco_ciscofczonedriver.cisco_opts,"},{"line_number":332,"context_line":"                cinder_zonemanager_fczonemanager.zone_manager_opts,"},{"line_number":333,"context_line":"            )),"},{"line_number":334,"context_line":"        (\u0027group\u0027,"},{"line_number":335,"context_line":"            itertools.chain("},{"line_number":336,"context_line":"                cinder_volume_drivers_dell_emc_powerstore_utils.backend_opts,"},{"line_number":337,"context_line":"            )),"}],"source_content_type":"text/x-python","patch_set":10,"id":"8c8e82da_c72733ff","line":334,"range":{"start_line":334,"start_character":10,"end_line":334,"end_character":15},"updated":"2026-09-08 06:09:43.000000000","message":"^^ \nliterally named \u0027group\u0027, it will show up as a [group] section, and config refrences","commit_id":"8cb1e241e3f57166289bed9f2dc09a323e4e4bdd"}],"cinder/tests/unit/volume/drivers/dell_emc/powerstore/test_volume_migrate.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"6124f4222da9834809ef8c0f615a1c0993de9efb","unresolved":true,"context_lines":[{"line_number":20,"context_line":"# Third-party"},{"line_number":21,"context_line":"from oslo_log import log as logging"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# Project (OpenStack Cinder)"},{"line_number":24,"context_line":"from cinder import context"},{"line_number":25,"context_line":"from cinder import exception"},{"line_number":26,"context_line":"from cinder.tests.unit import fake_volume as fvol"}],"source_content_type":"text/x-python","patch_set":10,"id":"50f044f5_d767b885","line":23,"updated":"2026-09-08 06:09:43.000000000","message":"nit: remove these comments, check existing code for ref","commit_id":"8cb1e241e3f57166289bed9f2dc09a323e4e4bdd"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"6124f4222da9834809ef8c0f615a1c0993de9efb","unresolved":true,"context_lines":[{"line_number":51,"context_line":"        self._vals[key] \u003d val"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"class TestDriverMigrateVolume(cinder_test.TestCase):"},{"line_number":55,"context_line":"    \"\"\"Unit tests for driver.migrate_volume() with adapter calls mocked.\"\"\""},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":10,"id":"5dd39fe5_3ffee0e7","line":54,"updated":"2026-09-08 06:09:43.000000000","message":"Why not use `powerstore.TestPowerStoreDriver` here ? \nsee other tests -  all other volume-operation tests in powestore are build on it","commit_id":"8cb1e241e3f57166289bed9f2dc09a323e4e4bdd"},{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"6124f4222da9834809ef8c0f615a1c0993de9efb","unresolved":true,"context_lines":[{"line_number":455,"context_line":"    # since the driver does not expose \u0027automatic_cutover\u0027 kwarg."},{"line_number":456,"context_line":""},{"line_number":457,"context_line":""},{"line_number":458,"context_line":"class DummyException(Exception):"},{"line_number":459,"context_line":"    \"\"\"Test replacement for VolumeBackendAPIException.\"\"\""},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"    def __init__(self, *args, **kwargs):"}],"source_content_type":"text/x-python","patch_set":10,"id":"13410d8c_55ec587a","line":458,"range":{"start_line":458,"start_character":6,"end_line":458,"end_character":20},"updated":"2026-09-08 06:09:43.000000000","message":"^^ !!","commit_id":"8cb1e241e3f57166289bed9f2dc09a323e4e4bdd"}],"cinder/volume/drivers/dell_emc/powerstore/adapter.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"319fd058f888dff3d5f1443e632f5a9aca2d9f3b","unresolved":true,"context_lines":[{"line_number":1006,"context_line":""},{"line_number":1007,"context_line":"        if not source_appliance:"},{"line_number":1008,"context_line":"            # fallback to THIS driver\u0027s configured appliance_id"},{"line_number":1009,"context_line":"            source_appliance \u003d self.configuration.safe_get(\"appliance_id\")"},{"line_number":1010,"context_line":""},{"line_number":1011,"context_line":"        if not source_appliance:"},{"line_number":1012,"context_line":"            msg \u003d ("}],"source_content_type":"text/x-python","patch_set":9,"id":"d3cfb686_117273f8","line":1009,"updated":"2026-04-15 05:29:07.000000000","message":"I am not finding any configuration in CommonAdapter. you may have to use imported configuration. self not required.","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"c747cb71029d8fad87b26f44fa22186e55f57ac5","unresolved":false,"context_lines":[{"line_number":1006,"context_line":""},{"line_number":1007,"context_line":"        if not source_appliance:"},{"line_number":1008,"context_line":"            # fallback to THIS driver\u0027s configured appliance_id"},{"line_number":1009,"context_line":"            source_appliance \u003d self.configuration.safe_get(\"appliance_id\")"},{"line_number":1010,"context_line":""},{"line_number":1011,"context_line":"        if not source_appliance:"},{"line_number":1012,"context_line":"            msg \u003d ("}],"source_content_type":"text/x-python","patch_set":9,"id":"f426cf2a_143873bf","line":1009,"in_reply_to":"d3cfb686_117273f8","updated":"2026-05-26 07:58:52.000000000","message":"Done","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"319fd058f888dff3d5f1443e632f5a9aca2d9f3b","unresolved":true,"context_lines":[{"line_number":1123,"context_line":"                raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1124,"context_line":""},{"line_number":1125,"context_line":"            try:"},{"line_number":1126,"context_line":"                self.cutover_migration_session(session_id)"},{"line_number":1127,"context_line":"            except Exception as e:"},{"line_number":1128,"context_line":"                msg \u003d ("},{"line_number":1129,"context_line":"                    \"Failed to initiate cutover for migration session %s: %s\""}],"source_content_type":"text/x-python","patch_set":9,"id":"58ccea53_00a08040","line":1126,"updated":"2026-04-15 05:29:07.000000000","message":"i didn\u0027t find implementation of cutover_migration_session in adapter and super in Object. It is there in client","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"c747cb71029d8fad87b26f44fa22186e55f57ac5","unresolved":false,"context_lines":[{"line_number":1123,"context_line":"                raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":1124,"context_line":""},{"line_number":1125,"context_line":"            try:"},{"line_number":1126,"context_line":"                self.cutover_migration_session(session_id)"},{"line_number":1127,"context_line":"            except Exception as e:"},{"line_number":1128,"context_line":"                msg \u003d ("},{"line_number":1129,"context_line":"                    \"Failed to initiate cutover for migration session %s: %s\""}],"source_content_type":"text/x-python","patch_set":9,"id":"76aa6c5d_01d50705","line":1126,"in_reply_to":"58ccea53_00a08040","updated":"2026-05-26 07:58:52.000000000","message":"adapter now correctly delegates to self.client.cutover_migration_session() as the implementation resides in the client layer.","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"}],"cinder/volume/drivers/dell_emc/powerstore/client.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"319fd058f888dff3d5f1443e632f5a9aca2d9f3b","unresolved":true,"context_lines":[{"line_number":1169,"context_line":"            if state in terminal_states:"},{"line_number":1170,"context_line":"                return state"},{"line_number":1171,"context_line":""},{"line_number":1172,"context_line":"            time.sleep(poll_interval)"},{"line_number":1173,"context_line":""},{"line_number":1174,"context_line":"        msg \u003d _(\"Timeout waiting for migration session %s to complete.\") % ("},{"line_number":1175,"context_line":"            session_id"}],"source_content_type":"text/x-python","patch_set":9,"id":"4f21eec0_bf9fcc11","line":1172,"updated":"2026-04-15 05:29:07.000000000","message":"this is blocking can you check alternatives like cinder.utils.retry","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"319fd058f888dff3d5f1443e632f5a9aca2d9f3b","unresolved":true,"context_lines":[{"line_number":1169,"context_line":"            if state in terminal_states:"},{"line_number":1170,"context_line":"                return state"},{"line_number":1171,"context_line":""},{"line_number":1172,"context_line":"            time.sleep(poll_interval)"},{"line_number":1173,"context_line":""},{"line_number":1174,"context_line":"        msg \u003d _(\"Timeout waiting for migration session %s to complete.\") % ("},{"line_number":1175,"context_line":"            session_id"}],"source_content_type":"text/x-python","patch_set":9,"id":"b9e1c322_ff8aa725","line":1172,"updated":"2026-04-15 05:29:07.000000000","message":"this is blocking can you check alternatives like cinder.utils.retry","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"c747cb71029d8fad87b26f44fa22186e55f57ac5","unresolved":false,"context_lines":[{"line_number":1169,"context_line":"            if state in terminal_states:"},{"line_number":1170,"context_line":"                return state"},{"line_number":1171,"context_line":""},{"line_number":1172,"context_line":"            time.sleep(poll_interval)"},{"line_number":1173,"context_line":""},{"line_number":1174,"context_line":"        msg \u003d _(\"Timeout waiting for migration session %s to complete.\") % ("},{"line_number":1175,"context_line":"            session_id"}],"source_content_type":"text/x-python","patch_set":9,"id":"b66145c6_cc69e8b3","line":1172,"in_reply_to":"4f21eec0_bf9fcc11","updated":"2026-05-26 07:58:52.000000000","message":"Done","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"c747cb71029d8fad87b26f44fa22186e55f57ac5","unresolved":false,"context_lines":[{"line_number":1169,"context_line":"            if state in terminal_states:"},{"line_number":1170,"context_line":"                return state"},{"line_number":1171,"context_line":""},{"line_number":1172,"context_line":"            time.sleep(poll_interval)"},{"line_number":1173,"context_line":""},{"line_number":1174,"context_line":"        msg \u003d _(\"Timeout waiting for migration session %s to complete.\") % ("},{"line_number":1175,"context_line":"            session_id"}],"source_content_type":"text/x-python","patch_set":9,"id":"3969c41a_45e2a299","line":1172,"in_reply_to":"b9e1c322_ff8aa725","updated":"2026-05-26 07:58:52.000000000","message":"Done","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"}],"cinder/volume/drivers/dell_emc/powerstore/driver.py":[{"author":{"_account_id":34860,"name":"Amit Uniyal","email":"auniyal@redhat.com","username":"auniyal"},"change_message_id":"6124f4222da9834809ef8c0f615a1c0993de9efb","unresolved":true,"context_lines":[{"line_number":247,"context_line":"                \"Volume %s already on destination; not accepting.\","},{"line_number":248,"context_line":"                volume.id"},{"line_number":249,"context_line":"            )"},{"line_number":250,"context_line":"            return False, None"},{"line_number":251,"context_line":""},{"line_number":252,"context_line":"        # For \u0027Completed\u0027 or \u0027Idle\u0027 terminal states, accept."},{"line_number":253,"context_line":"        if state in (\"Completed\", \"Idle\"):"}],"source_content_type":"text/x-python","patch_set":10,"id":"c9828f2e_c93aefb9","line":250,"updated":"2026-09-08 06:09:43.000000000","message":"this doesn\u0027t look right! returning false here means asking cinder to run `self._migrate_volume_generic(ctxt, volume, host, new_type_id)` which creates a new vol and copies the data.\n\ncould you check with powerflex, on which scenarios its ok to return False like different-backends or when storage pool compatibility - those sounds more genuine.\nhttps://github.com/openstack/cinder/blob/a4767351ce74612281497cef7cd7a36f8d784f56/cinder/volume/drivers/dell_emc/powerflex/driver.py#L1401-L1412\n\n--\nsame for netapp, checking backend\nhttps://github.com/openstack/cinder/blob/a4767351ce74612281497cef7cd7a36f8d784f56/cinder/volume/drivers/netapp/dataontap/utils/data_motion.py#L1738-L1755","commit_id":"8cb1e241e3f57166289bed9f2dc09a323e4e4bdd"}],"cinder/volume/drivers/dell_emc/powerstore/options.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"319fd058f888dff3d5f1443e632f5a9aca2d9f3b","unresolved":true,"context_lines":[{"line_number":80,"context_line":"        ),"},{"line_number":81,"context_line":"    ),"},{"line_number":82,"context_line":"    cfg.IntOpt("},{"line_number":83,"context_line":"        store_utils.POWERSTORE_REST_READ_TIMEOUT,"},{"line_number":84,"context_line":"        default\u003d30,"},{"line_number":85,"context_line":"        min\u003d1,"},{"line_number":86,"context_line":"        help\u003d("}],"source_content_type":"text/x-python","patch_set":9,"id":"f17e46d4_5d9dac88","line":83,"updated":"2026-04-15 05:29:07.000000000","message":"i see utils in import not store_tils","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"},{"author":{"_account_id":38286,"name":"Siddharth Kumar","display_name":"Siddharth","email":"siddharth.kumar1@dell.com","username":"siddhvrth"},"change_message_id":"c747cb71029d8fad87b26f44fa22186e55f57ac5","unresolved":false,"context_lines":[{"line_number":80,"context_line":"        ),"},{"line_number":81,"context_line":"    ),"},{"line_number":82,"context_line":"    cfg.IntOpt("},{"line_number":83,"context_line":"        store_utils.POWERSTORE_REST_READ_TIMEOUT,"},{"line_number":84,"context_line":"        default\u003d30,"},{"line_number":85,"context_line":"        min\u003d1,"},{"line_number":86,"context_line":"        help\u003d("}],"source_content_type":"text/x-python","patch_set":9,"id":"12f4d249_2135a4a3","line":83,"in_reply_to":"f17e46d4_5d9dac88","updated":"2026-05-26 07:58:52.000000000","message":"Done","commit_id":"0202928f5f01b897cfb33a7feb8ca2fc7011f426"}]}
