openstack
/
puppet-tripleo
Code Issues Proposed changes

[Ocata,Pike,Queens-Only] Fix Cinder's Netapp backend 

The cinder::backend::netapp resource has logic to determine whether
deprecated parameters are in use. THT supplies empty strings for default
values, which worked fine with Puppet 3.x, where empty strings are treated
as boolean False. When TripleO upgraded to Puppet 4.x in Ocata, the empty
strings are now treated as boolean True. This results in an invalid
configuration, which causes the Netapp backend to fail.

This is not an issue in Rocky, where the deprecated parameters are
being removed.

Closes-Bug: #1782376
Change-Id: I38d175361e7ad3fe38c27bbdc6d29dc6587635dc
This commit is contained in:
Alan Bishop 2018-07-18 15:41:52 -04:00
parent 9fdb07ec66
commit 3eea8289f0
2 changed files with 36 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

33
manifests/profile/base/cinder/volume/netapp.pp
View File

@ -38,6 +38,33 @@ class tripleo::profile::base::cinder::volume::netapp (
$cinder_netapp_nfs_shares = split(hiera('cinder::backend::netapp::nfs_shares', undef), ',')
}
# These cinder::backend::netapp parameters are deprecated, so empty
# strings supplied by THT need to be treated as undef.
# netapp_volume_list
# netapp_storage_pools
# netapp_eseries_host_type
$netapp_volume_list = hiera('cinder::backend::netapp::netapp_volume_list', undef)
if $netapp_volume_list == '' {
$netapp_volume_list_real = undef
} else {
$netapp_volume_list_real = $netapp_volume_list
}
$netapp_storage_pools = hiera('cinder::backend::netapp::netapp_storage_pools', undef)
if $netapp_storage_pools == '' {
$netapp_storage_pools_real = undef
} else {
$netapp_storage_pools_real = $netapp_storage_pools
}
$netapp_eseries_host_type = hiera('cinder::backend::netapp::netapp_eseries_host_type', undef)
if $netapp_eseries_host_type == '' {
$netapp_eseries_host_type_real = undef
} else {
$netapp_eseries_host_type_real = $netapp_eseries_host_type
}
cinder::backend::netapp { $backend_name :
netapp_login => hiera('cinder::backend::netapp::netapp_login', undef),
netapp_password => hiera('cinder::backend::netapp::netapp_password', undef),
@ -48,7 +75,7 @@ class tripleo::profile::base::cinder::volume::netapp (
netapp_storage_protocol => hiera('cinder::backend::netapp::netapp_storage_protocol', undef),
netapp_transport_type => hiera('cinder::backend::netapp::netapp_transport_type', undef),
netapp_vfiler => hiera('cinder::backend::netapp::netapp_vfiler', undef),
netapp_volume_list => hiera('cinder::backend::netapp::netapp_volume_list', undef),
netapp_volume_list => $netapp_volume_list_real,
netapp_vserver => hiera('cinder::backend::netapp::netapp_vserver', undef),
netapp_partner_backend_name => hiera('cinder::backend::netapp::netapp_partner_backend_name', undef),
nfs_shares => $cinder_netapp_nfs_shares,
@ -57,8 +84,8 @@ class tripleo::profile::base::cinder::volume::netapp (
netapp_copyoffload_tool_path => hiera('cinder::backend::netapp::netapp_copyoffload_tool_path', undef),
netapp_controller_ips => hiera('cinder::backend::netapp::netapp_controller_ips', undef),
netapp_sa_password => hiera('cinder::backend::netapp::netapp_sa_password', undef),
netapp_storage_pools => hiera('cinder::backend::netapp::netapp_storage_pools', undef),
netapp_eseries_host_type => hiera('cinder::backend::netapp::netapp_eseries_host_type', undef),
netapp_storage_pools => $netapp_storage_pools_real,
netapp_eseries_host_type => $netapp_eseries_host_type_real,
netapp_webservice_path => hiera('cinder::backend::netapp::netapp_webservice_path', undef),
nas_secure_file_operations => hiera('cinder::backend::netapp::nas_secure_file_operations', undef),
nas_secure_file_permissions => hiera('cinder::backend::netapp::nas_secure_file_permissions', undef),

6
releasenotes/notes/fix-cinder-netapp-deprecated-params-80bcfac1b1d2f88e.yaml Normal file
View File

@ -0,0 +1,6 @@
---
fixes:
- |
Fixed how deprecated parameters for Cinder's Netapp backend are handled
so that empty strings are not misinterpreted.
Fixes `bug 1782376 <https://bugs.launchpad.net/tripleo/+bug/1782376>`__.