Bug 1823416 - [OSP13] Backport Datera v2.2 cinder driver
Summary: [OSP13] Backport Datera v2.2 cinder driver
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: openstack-cinder
Version: 13.0 (Queens)
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: z12
: 13.0 (Queens)
Assignee: Alan Bishop
QA Contact: Tzach Shefi
Chuck Copello
URL:
Whiteboard:
Depends On: 1820794 1822304
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-04-13 15:58 UTC by Alan Bishop
Modified: 2020-06-24 11:52 UTC (History)
8 users (show)

Fixed In Version: openstack-cinder-12.0.10-7.el7ost
Doc Type: Enhancement
Doc Text:
This enhancement updates the cinder Datera driver to version 2.2.
Clone Of: 1820794
Environment:
Last Closed: 2020-06-24 11:51:47 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
OpenStack gerrit 704153 0 None MERGED Update in-tree Datera Cinder driver 2020-08-31 23:35:30 UTC
Red Hat Product Errata RHBA-2020:2722 0 None None None 2020-06-24 11:52:14 UTC

Description Alan Bishop 2020-04-13 15:58:57 UTC
+++ This bug was initially created as a clone of Bug #1820794 +++

Datera updated their cinder driver to v2.2 on Ussuri, and the updated driver is required on OSP releases based on Train and earlier. The nature of the patch makes it unsuitable for upstream backports to stable releases, and so the backport must be handled downstream.

The Ussuri patch is https://review.opendev.org/704153.

--- Additional comment from Virgil Mihailovici on 2020-04-05 20:52:19 UTC ---

I backported the driver to Train here:
https://github.com/Datera/cinder/tree/datera_train_backport

In particular, this commit contains all the changes required:
https://github.com/Datera/cinder/commit/9d71729fb8571384cf7e69b06b5ce53812651d6f

The patch passes out automation, it passes tempest tests and tox tests, the same as the original patch in Ussuri:
https://review.opendev.org/#/c/704153/

I followed the steps outlined by Allen here:
1. Datera forks the entire cinder project on github to another repo in their own namespace (e.g. https://github.com/Datera/cinder)

2. Create a branch off stable/train, and cherry pick the driver patch from master

% git clone https://github.com/Datera/cinder
% git checkout -b train/datera-v2.2 origin/stable/train
% git cherry-pick -x 6aabf4669204cb28d0ededfd3a6efba1fdd06293

Resolve the conflicts, confirm all unit tests and pep8 pass, perform any additional tests deemed appropriate.

Commit the final working patch, with the commit message including the list of files in the "Conflicts:" section of the commit message.

3. Push the patch to https://github.com/Datera/cinder, and add a comment in the BZ so that Red Hat can find the patch.

From there, Red Hat will pull the patch into our downstream source repository. Once that is complete, the openstack-cinder RPM containing the new Datera driver will make its way into a subsequent OSP-16.0 zStream release. After the RPM is released, Datera will be able to rebuild their cinder-volume plugin, in order to supply the dfs_sdk python module (an external out-of-tree dependency).

--- Additional comment from Virgil Mihailovici on 2020-04-05 20:54:06 UTC ---

Sorry, I misspelled Alan's name and I don't see a way to edit my comment.

--- Additional comment from Virgil Mihailovici on 2020-04-06 15:35:51 UTC ---

Updated the commit with proper formatting.
The branch is the same: https://github.com/Datera/cinder/tree/datera_train_backport
The specific commit is: https://github.com/Datera/cinder/commit/5a7e93bbe580393c7eedfa1bb751cec4948f5bec

----------------------

For OSP-13, see the corresponding patch from the https://github.com/Datera/cinder/tree/datera_queens_backport branch.

Comment 15 errata-xmlrpc 2020-06-24 11:51:47 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-2020:2722


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