)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":36800,"name":"Vrinda Dhakad","display_name":"vdhakad","email":"vrinda.dhakad@ibm.com","username":"vdhakad"},"change_message_id":"8ccacabad526ca07a1ef49e1eaf323d9825e86d8","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"89e50f4b_21931da9","updated":"2025-06-18 05:28:01.000000000","message":"The changes will break non-PBHA operations, please check.","commit_id":"8ae8a485e97d693d60afd61f6ca5068360686fae"},{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"15c95373c7c83fab8aa7340e71c688c6f17157e7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cc7d53be_7d8f3c4f","in_reply_to":"89e50f4b_21931da9","updated":"2025-07-14 09:32:14.000000000","message":"Resolved.","commit_id":"8ae8a485e97d693d60afd61f6ca5068360686fae"},{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"00a49a660a86f920dc407404504ae011e16679bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"c95f36dd_8465b241","updated":"2025-07-16 10:11:37.000000000","message":"recheck","commit_id":"3203a548031a524dfeb32bee41af59009ef0fc66"},{"author":{"_account_id":37328,"name":"Nimesh Desai","display_name":"Nimesh Desai","email":"nimesh.desai@ibm.com","username":"nimeshdesai"},"change_message_id":"66e4a9e0b234833a99856883c25d663a6c9bf10c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"19bdff25_a46aac6a","updated":"2025-11-13 02:15:56.000000000","message":"recheck cinder-plugin-ceph-tempest","commit_id":"61ae9663ca88dfcb287f2e6534fe743f63f1ceaf"},{"author":{"_account_id":37328,"name":"Nimesh Desai","display_name":"Nimesh Desai","email":"nimesh.desai@ibm.com","username":"nimeshdesai"},"change_message_id":"2672759937761e3a5812e9eb915960217d2ee918","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"361115ed_9d32b0fb","updated":"2025-11-15 07:30:55.000000000","message":"recheck devstack-plugin-nfs-tempest-full","commit_id":"f055b878131e7391a5b27927a32ed33b8957b9fd"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"d8df4da08367425e674c18c9da62ebd14b7b6f0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":32,"id":"a8bccfd1_698a25db","updated":"2026-04-02 05:08:38.000000000","message":"Can you please confirm snapshot_id validation can be removed?","commit_id":"aecfb1e91c0ba0d86171d12d659902b96d10077c"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_common.py":[{"author":{"_account_id":36800,"name":"Vrinda Dhakad","display_name":"vdhakad","email":"vrinda.dhakad@ibm.com","username":"vdhakad"},"change_message_id":"8ccacabad526ca07a1ef49e1eaf323d9825e86d8","unresolved":true,"context_lines":[{"line_number":672,"context_line":"            raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":673,"context_line":""},{"line_number":674,"context_line":"    def addsnapshot(self, params):"},{"line_number":675,"context_line":"        ssh_cmd \u003d [\u0027svctask\u0027, \u0027addsnapshot\u0027]"},{"line_number":676,"context_line":"        if \"volumegroup\" in params:"},{"line_number":677,"context_line":"            ssh_cmd.extend([\u0027-volumegroup\u0027, params[\"volumegroup\"]])"},{"line_number":678,"context_line":"        if \"name\" in params:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7cc3c7c9_4d74fb57","line":675,"range":{"start_line":675,"start_character":30,"end_line":675,"end_character":43},"updated":"2025-06-18 05:28:01.000000000","message":"Removing ignorelegacy will result in failure for non-pbha users with FlashCopy mappings and migrate_from_flashcopy set to True.","commit_id":"8ae8a485e97d693d60afd61f6ca5068360686fae"},{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"15c95373c7c83fab8aa7340e71c688c6f17157e7","unresolved":false,"context_lines":[{"line_number":672,"context_line":"            raise exception.VolumeBackendAPIException(data\u003dmsg)"},{"line_number":673,"context_line":""},{"line_number":674,"context_line":"    def addsnapshot(self, params):"},{"line_number":675,"context_line":"        ssh_cmd \u003d [\u0027svctask\u0027, \u0027addsnapshot\u0027]"},{"line_number":676,"context_line":"        if \"volumegroup\" in params:"},{"line_number":677,"context_line":"            ssh_cmd.extend([\u0027-volumegroup\u0027, params[\"volumegroup\"]])"},{"line_number":678,"context_line":"        if \"name\" in params:"}],"source_content_type":"text/x-python","patch_set":3,"id":"c012e9fb_ceed13a6","line":675,"range":{"start_line":675,"start_character":30,"end_line":675,"end_character":43},"in_reply_to":"7cc3c7c9_4d74fb57","updated":"2025-07-14 09:32:14.000000000","message":"Resolved.","commit_id":"8ae8a485e97d693d60afd61f6ca5068360686fae"},{"author":{"_account_id":36800,"name":"Vrinda Dhakad","display_name":"vdhakad","email":"vrinda.dhakad@ibm.com","username":"vdhakad"},"change_message_id":"8ccacabad526ca07a1ef49e1eaf323d9825e86d8","unresolved":true,"context_lines":[{"line_number":7512,"context_line":""},{"line_number":7513,"context_line":"            # Extract snapshot ID from metadata"},{"line_number":7514,"context_line":"            metadata_dict \u003d dict(snapshots[0].get(\u0027metadata\u0027, {}))"},{"line_number":7515,"context_line":"            parent_uid \u003d metadata_dict.get(\u0027svc_volumegroup_parent_uid\u0027)"},{"line_number":7516,"context_line":"            if not parent_uid:"},{"line_number":7517,"context_line":"                msg \u003d _(\"Snapshot metadata is missing \""},{"line_number":7518,"context_line":"                        \"\u0027svc_volumegroup_parent_uid\u0027.\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"77ef38c8_0ceb37da","line":7515,"range":{"start_line":7515,"start_character":12,"end_line":7515,"end_character":72},"updated":"2025-06-18 05:28:01.000000000","message":"For non-pbha, this will fail for restore from older snapshots that do not have parent_uid in metadata, there should be a condition to use snapshot-id if parent-uid is missing.","commit_id":"8ae8a485e97d693d60afd61f6ca5068360686fae"},{"author":{"_account_id":36538,"name":"Harsh Ailani","display_name":"Harsh Ailani","email":"harsh.ailani@ibm.com","username":"harshailani"},"change_message_id":"15c95373c7c83fab8aa7340e71c688c6f17157e7","unresolved":false,"context_lines":[{"line_number":7512,"context_line":""},{"line_number":7513,"context_line":"            # Extract snapshot ID from metadata"},{"line_number":7514,"context_line":"            metadata_dict \u003d dict(snapshots[0].get(\u0027metadata\u0027, {}))"},{"line_number":7515,"context_line":"            parent_uid \u003d metadata_dict.get(\u0027svc_volumegroup_parent_uid\u0027)"},{"line_number":7516,"context_line":"            if not parent_uid:"},{"line_number":7517,"context_line":"                msg \u003d _(\"Snapshot metadata is missing \""},{"line_number":7518,"context_line":"                        \"\u0027svc_volumegroup_parent_uid\u0027.\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"1fc9970d_3efb4363","line":7515,"range":{"start_line":7515,"start_character":12,"end_line":7515,"end_character":72},"in_reply_to":"77ef38c8_0ceb37da","updated":"2025-07-14 09:32:14.000000000","message":"Resolved.","commit_id":"8ae8a485e97d693d60afd61f6ca5068360686fae"},{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"d8df4da08367425e674c18c9da62ebd14b7b6f0e","unresolved":true,"context_lines":[{"line_number":7976,"context_line":"            # Extract snapshot metadata"},{"line_number":7977,"context_line":"            metadata_dict \u003d dict(snapshots[0].get(\u0027metadata\u0027, {}))"},{"line_number":7978,"context_line":""},{"line_number":7979,"context_line":"            if not self.partition_name:"},{"line_number":7980,"context_line":"                snapshot_id \u003d metadata_dict.get("},{"line_number":7981,"context_line":"                    \u0027svc_volumegroup_snapshot_id\u0027)"},{"line_number":7982,"context_line":"                vg_restore_params[\"id\"] \u003d snapshot_id"}],"source_content_type":"text/x-python","patch_set":32,"id":"12ef3b17_660bdf29","line":7979,"updated":"2026-04-02 05:08:38.000000000","message":"partition_name is optional. what about cases where partition_name is missing and snapshot_id is present? do we need old validation?","commit_id":"aecfb1e91c0ba0d86171d12d659902b96d10077c"}]}
