Dell PowerFlex: Additionnal params for enabling self signed certificates

Initially before the change
https://review.opendev.org/c/openstack/os-brick/+/810419 was merged
to close the bug https://bugs.launchpad.net/os-brick/+bug/1929223,
verify_cert was always set to False which can lead to security issues.
It has been decided through this change that this option can be set
to True or False based upon security requirements. This change
introduced a regression failure as the value set to the option
is not part of connection_properties.

This patch adds additional params during initialization so that it
can be carried over os-brick and get adequate REST API response.

Closes-Bug: 1990136
Change-Id: I0d266a57f68221a3b1740a7376e152bb64cac729
(cherry picked from commit 82823ace4d)
(cherry picked from commit 8794f84e77)
(cherry picked from commit be3c66ea47)
This commit is contained in:
happystacker
2022-09-19 08:55:24 +02:00
committed by Nilesh Thathagar
parent d1761af96a
commit 9878169b31
2 changed files with 16 additions and 0 deletions

View File

@ -867,6 +867,12 @@ class PowerFlexDriver(driver.VolumeDriver):
connection_properties["scaleIO_volume_id"] = vol_or_snap.provider_id
connection_properties["config_group"] = self.configuration.config_group
connection_properties["failed_over"] = self._is_failed_over
connection_properties["verify_certificate"] = (
self._get_client().verify_certificate
)
connection_properties["certificate_path"] = (
self._get_client().certificate_path
)
if vol_size is not None:
extra_specs = self._get_volumetype_extraspecs(vol_or_snap)

View File

@ -0,0 +1,10 @@
---
fixes:
- |
Dell PowerFlex driver `bug #1998136
<https://bugs.launchpad.net/cinder/+bug/1998136>`_:
When using self signed certificates, the option
sent to os-brick via the connection_properties was
not correctly handled. It has now been fixed by
adding the 'verify_certificate' and 'certificate_path'
to the driver when initializing the connection.