diff --git a/nova/conf/libvirt.py b/nova/conf/libvirt.py index 8ee6708441bf..690e6234087f 100644 --- a/nova/conf/libvirt.py +++ b/nova/conf/libvirt.py @@ -736,7 +736,13 @@ Use multipath connection of the iSCSI or FC volume Volumes can be connected in the LibVirt as multipath devices. This will provide high availability and fault tolerance. -""") +"""), + cfg.IntOpt('num_volume_scan_tries', + deprecated_name='num_iscsi_scan_tries', + default=5, + help=""" +Number of times to scan given storage protocol to find volume. +"""), ] libvirt_volume_aoe_opts = [ @@ -760,15 +766,7 @@ compute node. """) ] -# TODO(sneti): This config option is also used for other protocols like -# fibrechannel, scaleio, disco. So this should be renamed to -# num_volume_scan_tries libvirt_volume_iscsi_opts = [ - cfg.IntOpt('num_iscsi_scan_tries', - default=5, - help=""" -Number of times to scan iSCSI target to find volume. -"""), cfg.StrOpt('iscsi_iface', deprecated_name='iscsi_transport', help=""" diff --git a/nova/virt/libvirt/volume/disco.py b/nova/virt/libvirt/volume/disco.py index 3af2750e8225..15cdfdadea17 100644 --- a/nova/virt/libvirt/volume/disco.py +++ b/nova/virt/libvirt/volume/disco.py @@ -37,7 +37,7 @@ class LibvirtDISCOVolumeDriver(libvirt_volume.LibvirtBaseVolumeDriver): is_block_dev=False) self.connector = connector.InitiatorConnector.factory( 'DISCO', utils.get_root_helper(), - device_scan_attempts=CONF.libvirt.num_iscsi_scan_tries) + device_scan_attempts=CONF.libvirt.num_volume_scan_tries) def get_config(self, connection_info, disk_info): """Get DISCO volume attachment configuration.""" diff --git a/nova/virt/libvirt/volume/fibrechannel.py b/nova/virt/libvirt/volume/fibrechannel.py index a84e7d47575f..47b0744a50b7 100644 --- a/nova/virt/libvirt/volume/fibrechannel.py +++ b/nova/virt/libvirt/volume/fibrechannel.py @@ -34,7 +34,7 @@ class LibvirtFibreChannelVolumeDriver(libvirt_volume.LibvirtBaseVolumeDriver): self.connector = connector.InitiatorConnector.factory( 'FIBRE_CHANNEL', utils.get_root_helper(), use_multipath=CONF.libvirt.volume_use_multipath, - device_scan_attempts=CONF.libvirt.num_iscsi_scan_tries) + device_scan_attempts=CONF.libvirt.num_volume_scan_tries) def get_config(self, connection_info, disk_info): """Returns xml for libvirt.""" diff --git a/nova/virt/libvirt/volume/hgst.py b/nova/virt/libvirt/volume/hgst.py index e5df5bea01d4..05c65afb04e0 100644 --- a/nova/virt/libvirt/volume/hgst.py +++ b/nova/virt/libvirt/volume/hgst.py @@ -30,7 +30,7 @@ class LibvirtHGSTVolumeDriver(libvirt_volume.LibvirtBaseVolumeDriver): self).__init__(host, is_block_dev=True) self.connector = connector.InitiatorConnector.factory( 'HGST', utils.get_root_helper(), - device_scan_attempts=CONF.libvirt.num_iscsi_scan_tries) + device_scan_attempts=CONF.libvirt.num_volume_scan_tries) def get_config(self, connection_info, disk_info): """Returns xml for libvirt.""" diff --git a/nova/virt/libvirt/volume/iscsi.py b/nova/virt/libvirt/volume/iscsi.py index 1bd9a3c82513..278a5e80d6e3 100644 --- a/nova/virt/libvirt/volume/iscsi.py +++ b/nova/virt/libvirt/volume/iscsi.py @@ -37,7 +37,7 @@ class LibvirtISCSIVolumeDriver(libvirt_volume.LibvirtBaseVolumeDriver): self.connector = connector.InitiatorConnector.factory( 'ISCSI', utils.get_root_helper(), use_multipath=CONF.libvirt.volume_use_multipath, - device_scan_attempts=CONF.libvirt.num_iscsi_scan_tries, + device_scan_attempts=CONF.libvirt.num_volume_scan_tries, transport=self._get_transport()) def _get_transport(self): diff --git a/nova/virt/libvirt/volume/scaleio.py b/nova/virt/libvirt/volume/scaleio.py index b97152eb0507..a5fc0c308c56 100644 --- a/nova/virt/libvirt/volume/scaleio.py +++ b/nova/virt/libvirt/volume/scaleio.py @@ -38,7 +38,7 @@ class LibvirtScaleIOVolumeDriver(libvirt_volume.LibvirtBaseVolumeDriver): is_block_dev=False) self.connector = connector.InitiatorConnector.factory( 'SCALEIO', utils.get_root_helper(), - device_scan_attempts=CONF.libvirt.num_iscsi_scan_tries) + device_scan_attempts=CONF.libvirt.num_volume_scan_tries) def get_config(self, connection_info, disk_info): conf = super(LibvirtScaleIOVolumeDriver, self).get_config( diff --git a/releasenotes/notes/rename-libvirt-num-iscsi-scan-tries-opt-8329385f84d2518e.yaml b/releasenotes/notes/rename-libvirt-num-iscsi-scan-tries-opt-8329385f84d2518e.yaml new file mode 100644 index 000000000000..cd896b365c3f --- /dev/null +++ b/releasenotes/notes/rename-libvirt-num-iscsi-scan-tries-opt-8329385f84d2518e.yaml @@ -0,0 +1,7 @@ +--- +upgrade: + - | + ``libvirt.num_iscsi_scan_tries`` option has been renamed to + ``libvirt.num_volume_scan_tries``, as the previous name was suggesting + that this option only concerns devices connected using iSCSI interface. + It also concerns devices connected using fibrechannel, scaleio and disco. \ No newline at end of file