)]}'
{"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_common.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"816574b0cff5a802830e79c857e7f97566a41f9a","unresolved":true,"context_lines":[{"line_number":7577,"context_line":""},{"line_number":7578,"context_line":"        # Check if the disk is in use, if we need to."},{"line_number":7579,"context_line":"        manage_if_in_use \u003d ref.get(\u0027manage_if_in_use\u0027, False)"},{"line_number":7580,"context_line":"        if (not manage_if_in_use and"},{"line_number":7581,"context_line":"                self._helpers.is_vdisk_in_use(vdisk[\u0027name\u0027])):"},{"line_number":7582,"context_line":"            reason \u003d _(\u0027The specified vdisk is mapped to a host.\u0027)"},{"line_number":7583,"context_line":"            raise exception.ManageExistingInvalidReference(existing_ref\u003dref,"}],"source_content_type":"text/x-python","patch_set":25,"id":"ab952041_9164ae63","side":"PARENT","line":7580,"updated":"2026-04-15 16:35:07.000000000","message":"This check was a safety guard preventing accidental management of in-use volumes for all Storwize backends (not just PBHA). Removing it means:\n\nAn operator can now manage an in-use non-PBHA volume without manage_if_in_use: True in the ref.","commit_id":"13693f5c512564374b14964121b1eecc83e6bd76"}],"cinder/volume/drivers/ibm/storwize_svc/storwize_svc_fc.py":[{"author":{"_account_id":36171,"name":"jayaanand borra","display_name":"jayaanand borra","email":"jayaanand.borra@netapp.com","username":"jayaanan","status":"netapp"},"change_message_id":"816574b0cff5a802830e79c857e7f97566a41f9a","unresolved":true,"context_lines":[{"line_number":375,"context_line":"        # to PBHA or partition volume using unmanage/manage we"},{"line_number":376,"context_line":"        # want to retain host-volume mappings on storage. Hence,"},{"line_number":377,"context_line":"        # do nothing"},{"line_number":378,"context_line":"        if self._get_config_param_value(\u0027migrate_to_partition\u0027,"},{"line_number":379,"context_line":"                                        None) is True:"},{"line_number":380,"context_line":"            LOG.info(\"Leaving without removing attachment on storage.\")"},{"line_number":381,"context_line":"            return"}],"source_content_type":"text/x-python","patch_set":25,"id":"2d3b5487_b2d7b4c4","line":378,"updated":"2026-04-15 16:35:07.000000000","message":"_get_config_param_value calls self.configuration.safe_get(config_param). For a StrOpt, oslo.config returns None when unset or returns the string value \"True\" / \"False\" when set via cinder.conf. The string \"True\" is not the boolean True, so \"True\" is True evaluates to False. The condition can never be triggered by an operator setting migrate_to_partition \u003d True in cinder.conf. The feature is effectively broken from the day it ships.","commit_id":"5c4f47d7385d4c454ef9b29220bbd2b6ea5fb0bb"}]}
