Bug 1468025 - Heat ScaleIO/Cinder Puppet module deploys invalid Cinder driver by default
Summary: Heat ScaleIO/Cinder Puppet module deploys invalid Cinder driver by default
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: puppet-cinder
Version: 11.0 (Ocata)
Hardware: All
OS: Linux
medium
medium
Target Milestone: z4
: 11.0 (Ocata)
Assignee: Alan Bishop
QA Contact: Avi Avraham
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-07-05 19:36 UTC by Siggy Sigwald
Modified: 2020-09-10 10:51 UTC (History)
6 users (show)

Fixed In Version: puppet-cinder-10.3.1-2.el7ost
Doc Type: Bug Fix
Doc Text:
Previously, the ScaleIO cinder driver code had been relocated in the cinder code tree, however, the puppet code responsible for configuring the driver had not been updated to refer to the driver in its new location. Consequently, Puppet would misconfigure the ScaleIO driver by using an obsolete reference to the driver location. The driver still continued to function because cinder has a built-in mechanism for mapping references to old driver locations to their new location. With this update, the `puppet-cinder` code has been updated to reference the ScaleIO driver in its new location.
Clone Of:
Environment:
Last Closed: 2018-02-13 16:31:14 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Launchpad 1718986 0 None None None 2017-09-22 17:41:08 UTC
OpenStack gerrit 506741 0 None MERGED Fix path to ScaleIO backend driver 2020-04-20 10:46:10 UTC
OpenStack gerrit 507530 0 None MERGED Fix path to ScaleIO backend driver 2020-04-20 10:46:10 UTC
Red Hat Product Errata RHBA-2018:0310 0 normal SHIPPED_LIVE Red Hat OpenStack Platform 11.0 director Bug Fix Advisory 2018-02-14 00:13:02 UTC

Description Siggy Sigwald 2017-07-05 19:36:12 UTC
Description of problem:
In RHOSP11 the bundled Dell EMC ScaleIO module used for heat deployments uses an old driver name. This driver file does not exist anymore -- this makes the entire Cinder backend deployment using the provided template not work.

To fix this, please make the following changes. 
In this file: /usr/share/openstack-puppet/modules/cinder/manifests/backend/scaleio.pp
The volume_driver variable must be changed from:
cinder.volume.drivers.emc.scaleio.ScaleIODriver
to
cinder.volume.drivers.dell_emc.scaleio.driver.ScaleIODriver

The correctly named driver file comes with RHOSP11, the Puppet module just specifies the incorrect legacy file that has not existed for a while already.

How reproducible:
This happens evey time when using the ScaleIO Cinder backend.

Comment 1 Eric Harney 2017-07-19 13:38:24 UTC
We should fix this, but things should work with the old path, since the Cinder volume manager has a compat layer that maps the old path to the new path:

https://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/manager.py?h=10.0.3#n158

Comment 2 Alan Bishop 2017-09-25 16:19:58 UTC
Patch has been accepted upstream, and I'm backporting to stable/pike, and then stable/ocata.

Comment 3 Alan Bishop 2017-09-26 13:13:23 UTC
Patch has been accepted on stable/pike, and proposed to stable/ocata.

Comment 4 Alan Bishop 2017-10-13 16:38:02 UTC
It looks like the fix will be picked up during the import from RDO for 11z4.

Comment 7 Tzach Shefi 2018-01-29 11:58:22 UTC
Set OtherQA, 
Don't have ScaleIO system to actually run any Cinder functions over. 

I did however verify that on this file:
/usr/share/openstack-puppet/modules/cinder/manifests/backend/scaleio.pp

Volume_driver variable has been changed to correct one:
cinder.volume.drivers.dell_emc.scaleio.driver.ScaleIODriver

Comment 10 errata-xmlrpc 2018-02-13 16:31:14 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:0310


Note You need to log in before you can comment on or make changes to this bug.