openstack/puppet-cinder
Code Issues Proposed changes

Dell EMC SC: Fix the excluded_domain_ip value

Browse Source 
Exclude the Dell SC cinder driver 'excluded_domain_ip'
if it is not specified

Closes Bug: #1758145

Change-Id: Iabeb5553f2505dff97901ac73d10030f6b215453
This commit is contained in:
rajinir 2018-03-27 10:42:55 -05:00
parent 75c62afe10
commit a897fe3b71
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
manifests/backend/dellsc_iscsi.pp
View File

@ -121,13 +121,18 @@ default of \"vol\" and will be changed to the upstream OpenStack default in N-re
"${name}/dell_sc_verify_cert": value => $dell_sc_verify_cert;
"${name}/dell_sc_volume_folder": value => $dell_sc_volume_folder;
"${name}/iscsi_port": value => $iscsi_port;
"${name}/excluded_domain_ip": value => $excluded_domain_ip;
"${name}/secondary_san_ip": value => $secondary_san_ip;
"${name}/secondary_san_login": value => $secondary_san_login;
"${name}/secondary_san_password": value => $secondary_san_password;
"${name}/secondary_sc_api_port": value => $secondary_sc_api_port;
}
if $excluded_domain_ip {
cinder_config {
"${name}/excluded_domain_ip": value => $excluded_domain_ip;
}
}
if $manage_volume_type {
cinder_type { $volume_backend_name:
ensure => present,

5
releasenotes/notes/sc-driver-d428e372280c44e8.yaml Normal file
View File

@ -0,0 +1,5 @@
---
fixes:
- Fixes `bug 1758145
<https://bugs.launchpad.net/puppet-cinder/+bug/1758145>`__
to ignore the excluded_domain_ip Dell EMC SC Driver when blank.