openstack/cinder
Code Issues Proposed changes

Remove WindowsDriver mapping

Browse Source 
The WindowsDriver was renamed to WindowsISCSIDriver in change
I3877491463dce3d46f7ac0e194ffdf46a0e7c84c to avoid confusion with the
Windows SMB driver. It has now been two release cycles and the backward
compatibility mapping can now be removed.

Change-Id: I602cccac73b6dc8d0b0824a5dffb0373c151452e
Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com>
This commit is contained in:
Sean McGinnis 2018-08-22 14:18:01 -05:00
parent f68a0e80ed
commit b428e25faf
No known key found for this signature in database
GPG Key ID: CE7EE4BFAF8D70C8
2 changed files with 13 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
cinder/volume/manager.py
View File

@ -155,9 +155,11 @@ CONF = cfg.CONF
CONF.register_opts(volume_manager_opts)
CONF.register_opts(volume_backend_opts, group=config.SHARED_CONF_GROUP)
# MAPPING is used for driver renames to keep backwards compatibilty. When a
# driver is renamed, add a mapping here from the old name (the dict key) to the
# new name (the dict value) for at least a cycle to allow time for deployments
# to transition.
MAPPING = {
'cinder.volume.drivers.windows.windows.WindowsDriver':
'cinder.volume.drivers.windows.iscsi.WindowsISCSIDriver',
}

9
releasenotes/notes/remove-windows-mapping-51a004f466470a2b.yaml Normal file
View File

@ -0,0 +1,9 @@
---
upgrade:
- |
The ``WindowsDriver`` was renamed in the Queens release to
``WindowsISCSIDriver`` to avoid confusion with the SMB driver. The
backwards compatibility for this has now been removed, so any cinder.conf
settings still using
``cinder.volume.drivers.windows.windows.WindowsDriver`` must now be updated
to use ``cinder.volume.drivers.windows.iscsi.WindowsISCSIDriver``.