)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"6cf9f1445a632e86b6d9bcbc196a9492e50e09e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b335d2a1_e8f8a14e","updated":"2026-09-15 16:56:15.000000000","message":"Add releasenotes for Behavior/error-path change for PowerMax retype.","commit_id":"6abed2175821167c61e9cea07523cd9d61fc5f20"}],"cinder/volume/drivers/dell_emc/powermax/utils.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"6cf9f1445a632e86b6d9bcbc196a9492e50e09e2","unresolved":true,"context_lines":[{"line_number":2114,"context_line":"        if pool_name is None:"},{"line_number":2115,"context_line":"            msg \u003d (_(\u0027No pool_name specified in volume-type.\u0027))"},{"line_number":2116,"context_line":"            raise exception.VolumeBackendAPIException(msg)"},{"line_number":2117,"context_line":""},{"line_number":2118,"context_line":"        pool_details \u003d pool_name.split(\u0027+\u0027)"},{"line_number":2119,"context_line":"        if len(pool_details) \u003d\u003d 4:"},{"line_number":2120,"context_line":"            array_id \u003d pool_details[3]"}],"source_content_type":"text/x-python","patch_set":1,"id":"3873d23a_cdabbf49","line":2117,"updated":"2026-09-15 16:56:15.000000000","message":"Raising here is swallowed by VolumeManager.retype() except Exception and converted to generic migration. Attached volumes and migration_policy\u003dnever still fail. Storage-assisted SG retype is never attempted.","commit_id":"6abed2175821167c61e9cea07523cd9d61fc5f20"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"a82a03d4fc92d9ac814f3f1057fb212273bf4dc5","unresolved":false,"context_lines":[{"line_number":2114,"context_line":"        if pool_name is None:"},{"line_number":2115,"context_line":"            msg \u003d (_(\u0027No pool_name specified in volume-type.\u0027))"},{"line_number":2116,"context_line":"            raise exception.VolumeBackendAPIException(msg)"},{"line_number":2117,"context_line":""},{"line_number":2118,"context_line":"        pool_details \u003d pool_name.split(\u0027+\u0027)"},{"line_number":2119,"context_line":"        if len(pool_details) \u003d\u003d 4:"},{"line_number":2120,"context_line":"            array_id \u003d pool_details[3]"}],"source_content_type":"text/x-python","patch_set":1,"id":"dcf7d051_5929012b","line":2117,"in_reply_to":"01fd268b_6d3b015b","updated":"2026-09-16 04:35:26.000000000","message":"upto you! you are the maintainers of driver. I am fine with it. Please add release notes","commit_id":"6abed2175821167c61e9cea07523cd9d61fc5f20"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"74927c1f5962615eb4b76b93b409a35aa9d23444","unresolved":true,"context_lines":[{"line_number":2114,"context_line":"        if pool_name is None:"},{"line_number":2115,"context_line":"            msg \u003d (_(\u0027No pool_name specified in volume-type.\u0027))"},{"line_number":2116,"context_line":"            raise exception.VolumeBackendAPIException(msg)"},{"line_number":2117,"context_line":""},{"line_number":2118,"context_line":"        pool_details \u003d pool_name.split(\u0027+\u0027)"},{"line_number":2119,"context_line":"        if len(pool_details) \u003d\u003d 4:"},{"line_number":2120,"context_line":"            array_id \u003d pool_details[3]"}],"source_content_type":"text/x-python","patch_set":1,"id":"01fd268b_6d3b015b","line":2117,"in_reply_to":"3873d23a_cdabbf49","updated":"2026-09-15 17:01:32.000000000","message":"The problem we are facing here is because the next instruction tries to split a None value when pool_name isn\u0027t specified in the extra_specs of the volume type.\nDo we need to refactor the whole function?","commit_id":"6abed2175821167c61e9cea07523cd9d61fc5f20"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"ee686876d79173b1122c31f4dec9f63bb49e280b","unresolved":false,"context_lines":[{"line_number":2114,"context_line":"        if pool_name is None:"},{"line_number":2115,"context_line":"            msg \u003d (_(\u0027No pool_name specified in volume-type.\u0027))"},{"line_number":2116,"context_line":"            raise exception.VolumeBackendAPIException(msg)"},{"line_number":2117,"context_line":""},{"line_number":2118,"context_line":"        pool_details \u003d pool_name.split(\u0027+\u0027)"},{"line_number":2119,"context_line":"        if len(pool_details) \u003d\u003d 4:"},{"line_number":2120,"context_line":"            array_id \u003d pool_details[3]"}],"source_content_type":"text/x-python","patch_set":1,"id":"91887524_be1c395e","line":2117,"in_reply_to":"76514003_7be9e460","updated":"2026-09-16 06:37:16.000000000","message":"I\u0027ll propose another patch to improve the logic behind so that _slo_workload_migration() constructs target extra specs and supplement them with the selected pool from host before checking compression","commit_id":"6abed2175821167c61e9cea07523cd9d61fc5f20"},{"author":{"_account_id":31779,"name":"Jean Pierre Roquesalane","display_name":"happystacker","email":"jeanpierre.roquesalane@dell.com","username":"happystacker"},"change_message_id":"a660054b732b4fc3164c30af12426464039cfdca","unresolved":false,"context_lines":[{"line_number":2114,"context_line":"        if pool_name is None:"},{"line_number":2115,"context_line":"            msg \u003d (_(\u0027No pool_name specified in volume-type.\u0027))"},{"line_number":2116,"context_line":"            raise exception.VolumeBackendAPIException(msg)"},{"line_number":2117,"context_line":""},{"line_number":2118,"context_line":"        pool_details \u003d pool_name.split(\u0027+\u0027)"},{"line_number":2119,"context_line":"        if len(pool_details) \u003d\u003d 4:"},{"line_number":2120,"context_line":"            array_id \u003d pool_details[3]"}],"source_content_type":"text/x-python","patch_set":1,"id":"76514003_7be9e460","line":2117,"in_reply_to":"dcf7d051_5929012b","updated":"2026-09-16 06:35:49.000000000","message":"Thanks, understood.\nThis change only converts the accidental AttributeError into the driver’s intended configuration exception; it does not make the retype succeed. VolumeManager catches the exception and falls back to generic migration, which still fails for attached volumes or migration_policy\u003dnever. The functional issue requires supplying the target pool_name or allowing the PowerMax driver to derive the target SG specifications without it. \nI’ll clarify the commit message and add a release note.","commit_id":"6abed2175821167c61e9cea07523cd9d61fc5f20"}]}
