From 95a02d2419fa279838a5bc3b600b235f41615cac Mon Sep 17 00:00:00 2001 From: katarimanoj Date: Tue, 28 Jun 2022 19:59:09 +0530 Subject: [PATCH] Fix Cinder Dell EMC SC backend template The 'CinderSCStorageProtocol' parameter defined in cinder-backend-dellemc-sc-puppet.yaml is not used and has no effect. Modified it as 'CinderScStorageProtocol' similar to other parameters. The parameter 'CinderScExcludedDomainIp' is introduced when the backend support was added in the patch https://review.opendev.org/c/openstack/tripleo-heat-templates/+/727283 but it is never used. Removed it as the 'CinderScExcludedDomainIps' parameter is already used. Closes-Bug: #1978749 Closes-Bug: #1978750 Change-Id: I6a82a542855e32bd82c67f34138271b060df5de2 --- deployment/cinder/cinder-backend-dellemc-sc-puppet.yaml | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/deployment/cinder/cinder-backend-dellemc-sc-puppet.yaml b/deployment/cinder/cinder-backend-dellemc-sc-puppet.yaml index 418b8717f5..e53ac938c9 100644 --- a/deployment/cinder/cinder-backend-dellemc-sc-puppet.yaml +++ b/deployment/cinder/cinder-backend-dellemc-sc-puppet.yaml @@ -75,16 +75,13 @@ parameters: CinderScSecondaryScApiPort: type: number default: 3033 - CinderScExcludedDomainIp: - type: string - default: '' CinderScExcludedDomainIps: type: comma_delimited_list default: [] CinderScMultipathXfer: type: boolean default: true - CinderSCStorageProtocol: + CinderScStorageProtocol: type: string default: 'iSCSI' constraints: