openstack/nova-powervm
Code Issues Proposed changes

Remove volume_group_vios_name

Browse Source 
nova-powervm commit c64087adb0
removed the only code using CONF.localdisk.volume_group_vios_name
but neglected to remove the conf definition itself or the
references in devref. This change removes both since they no
longer apply.

Change-Id: I2bd2739c99d7fd12f9a111dec62286f04e345e6b
This commit is contained in:
Matthew Edmonds
2017-05-26 18:42:09 -04:00
parent af5932c9b3
commit 74f92e1397
2 changed files with 1 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
doc/source/devref/usage.rst
View File

@@ -103,7 +103,7 @@ Disk Options
| | Group exists on one of the Virtual I/O Servers, then no | | | Group exists on one of the Virtual I/O Servers, then no |
| | further config is needed. If multiple volume groups exist,| | | further config is needed. If multiple volume groups exist,|
| | then further specification can be done via the | | | then further specification can be done via the |
| | volume_group_* options. | | | volume_group_name option. |
| | | | | |
| | Live migration is not supported with a localdisk config. | | | Live migration is not supported with a localdisk config. |
| | | | | |
@@ -123,15 +123,6 @@ Disk Options
| | than one non-rootvg volume group exists across the | | | than one non-rootvg volume group exists across the |
| | Virtual I/O Servers, then this attribute must be specified.| | | Virtual I/O Servers, then this attribute must be specified.|
+--------------------------------------+------------------------------------------------------------+ +--------------------------------------+------------------------------------------------------------+
| volume_group_vios_name = None | (StrOpt) (Optional) The name of the Virtual I/O Server |
| | hosting the volume group. If this is not specified, the |
| | system will query through the Virtual I/O Servers looking |
| | for one that matches the volume_group_vios_name. This is |
| | only needed if the system has multiple Virtual I/O Servers |
| | with a non-rootvg volume group whose name is duplicated. |
| | |
| | Typically paired with the volume_group_name attribute. |
+--------------------------------------+------------------------------------------------------------+
Volume Options Volume Options

9
nova_powervm/conf/powervm.py
View File

@@ -102,15 +102,6 @@ localdisk_opts = [
'disk_driver is localdisk, then this attribute must be ' 'disk_driver is localdisk, then this attribute must be '
'specified. It is strongly recommended NOT to use ' 'specified. It is strongly recommended NOT to use '
'rootvg.'), 'rootvg.'),
cfg.StrOpt('volume_group_vios_name',
default='',
help='(Optional) The name of the Virtual I/O Server hosting '
'the volume group. If this is not specified, the system '
'will query through the Virtual I/O Servers looking for '
'one that matches the volume_group_vios_name. This is '
'only needed if the system has multiple Virtual I/O '
'Servers with a volume group whose name matches the '
'`volume_group_name` option.')
] ]
ssp_opts = [ ssp_opts = [