openstack/puppet-cinder
Code Issues Proposed changes

Merge "PowerStore: Remove the deprecated powerstore_appliances parameter"

Browse Source
This commit is contained in:
Zuul 2021-08-09 16:56:05 +00:00 committed by Gerrit Code Review
commit d2232bfbd2
2 changed files with 5 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
manifests/backend/dellemc_powerstore.pp
View File

@ -42,12 +42,6 @@
# Example:
# { 'dellemc_powerstore_backend/param1' => { 'value' => value1 } }
#
# DEPRECATED PARAMETERS
#
# [*powerstore_appliances*]
# (optional) PowerStore appliances
# Defaults to undef
#
define cinder::backend::dellemc_powerstore (
$san_ip,
$san_login,
@ -58,15 +52,10 @@ define cinder::backend::dellemc_powerstore (
$backend_availability_zone = $::os_service_default,
$manage_volume_type = false,
$extra_options = {},
$powerstore_appliances = undef,
) {
include cinder::deps
if $powerstore_appliances != undef {
warning('The powerstore_appliances parameter has been deprecated and has no effect')
}
if $storage_protocol == 'iSCSI' {
$driver = 'dell_emc.powerstore.driver.PowerStoreDriver'
}

5
releasenotes/notes/remove-powerstore_appliances-36212e32db70ca44.yaml Normal file
View File

@ -0,0 +1,5 @@
---
upgrade:
- |
The ``cinder::backend::dellemc_powerstore::powerstore_appliances``
parameter has been removed.