openstack/puppet-cinder
Code Issues Proposed changes

Add support for NetApp direct drivers

Browse Source 
netapp_wsdl_url is only valid for the regular drivers.
netapp_storage_service is optional for all NetApp drivers.
See here for example:
http://docs.openstack.org/grizzly/openstack-block-storage/admin/content/netapp-iscsi-driver-direct-7mode.html

Change-Id: I8c6cd9d63b3e6da683bf210f1b4db7bd7fd3d315
This commit is contained in:
Logan McNaughton 2013-07-17 14:53:42 -06:00 committed by Gerrit Code Review
parent 86cc398cc2
commit b4819ab7ad
2 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/volume/netapp.pp
View File

@ -1,10 +1,10 @@
#
class cinder::volume::netapp (
$netapp_wsdl_url,
$netapp_login,
$netapp_password,
$netapp_server_hostname,
$netapp_storage_service,
$netapp_wsdl_url = undef,
$netapp_storage_service = undef,
$netapp_server_port = '8088',
$netapp_storage_service_prefix = 'openstack',
$netapp_vfiler = undef

2
spec/classes/cinder_volume_netapp_spec.rb
View File

@ -4,11 +4,9 @@ describe 'cinder::volume::netapp' do
let :params do
{
:netapp_wsdl_url => 'http://127.0.0.1/dfm.wsdl',
:netapp_login => 'netapp',
:netapp_password => 'password',
:netapp_server_hostname => '127.0.0.2',
:netapp_storage_service => 'OpenStack',
}
end