openstack
/
nova
Code Issues Proposed changes

libvirt : remove broken olso_config choices option 

Changeset ed9f1b8b restricted the allowed options for iscsi_iface
to the allowed transports mentioned in the help text. iface &
transport are distinct entities for open-iscsi, and an iface is
generally a mix of transport plus an underlying interface, hence
this change was incorrect. This breaks transport support and is
generally incorrect, hence reverting

Closes-Bug:1466730
Change-Id: I687fb9c95133dbbb193004df44794404b58ca692
This commit is contained in:
Anish Bhatt 2015-06-18 22:52:04 -07:00
parent 275c9ace5c
commit 66b9c2ceae
1 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
nova/virt/libvirt/volume.py
View File

@ -106,12 +106,15 @@ volume_opts = [
help='Path to a Quobyte Client configuration file.'),
cfg.StrOpt('iscsi_iface',
deprecated_name='iscsi_transport',
choices=('be2iscsi', 'bnx2i', 'cxgb3i', 'cxgb4i', 'qla4xxx',
'ocs'),
help='The iSCSI transport iface to use to connect to target in '
'case offload support is desired. Default format is '
'transport_name.hwaddress and can be generated manually '
'or via iscsiadm -m iface'),
'case offload support is desired. Default format is of '
'the form <transport_name>.<hwaddress> where '
'<transport_name> is one of (be2iscsi, bnx2i, cxgb3i, '
'cxgb4i, qla4xxx, ocs) and <hwadress> is the MAC address '
'of the interface and can be generated via the '
'iscsiadm -m iface command. Do not confuse the '
'iscsi_iface parameter to be provided here with the '
'actual transport name.'),
# iser is also supported, but use LibvirtISERVolumeDriver
# instead
]