Bug 1930255

Summary:	Add support for port filtering for XIO cinder driver
Product:	Red Hat OpenStack	Reporter:	arkady kanevsky <arkady_kanevsky>
Component:	openstack-cinder	Assignee:	Pablo Caruana <pcaruana>
Status:	CLOSED ERRATA	QA Contact:	Tzach Shefi <tshefi>
Severity:	medium	Docs Contact:	RHOS Documentation Team <rhos-docs>
Priority:	medium
Version:	16.1 (Train)	CC:	abishop, a.stripeikis, gcharot, gregraka, jvisser, kholtz, kurt_hey, lmarsh, morazi, pcaruana, pgrist, sam.wan, vladislav.belogrudov
Target Milestone:	z7	Keywords:	OtherQA, Triaged
Target Release:	16.1 (Train on RHEL 8.2)
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	openstack-cinder-15.4.0-1.20210709072758.58f0e73.el8ost	Doc Type:	Bug Fix
Doc Text:	In previous releases, in Red Hat OpenStack Platform (RHOSP) deployments that use the Dell EMC XtremIO driver, attach volume operations waited for a timeout if iSCSI or FC targets were not connected to a RHOSP host. This caused attach volume operations to fail. + This release adds port filtering support for the Dell EMC XtremIO driver to allow iSCSI or FC ports that are not in use to be ignored.	Story Points:	---
Clone Of:
Clones:	1956370 (view as bug list)		Environment:
Last Closed:	2021-12-09 20:18:00 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:	Wallaby
Embargoed:
Bug Depends On:
Bug Blocks:	1595325, 1715964, 1930267, 1941598, 1941616, 1956370

Description arkady kanevsky 2021-02-18 15:26:53 UTC

Description of problem:
Add ports filtering support to Dell EMC XtremIO driver.
If there are some iSCSI or FC targets that are not connected to the OpenStack host, attach volume operation waits until timeout when trying to use these ports.
The XtremIO Cinder driver needs a new option to support ports filtering.
This functionality already exists for many other cinder drivers 

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1.
2.
3.

Actual results:


Expected results:


Additional info:

Comment 1 arkady kanevsky 2021-03-30 14:07:03 UTC

It has been packported to Victoria - https://review.opendev.org/c/openstack/puppet-cinder/+/779000
Do we need to backport it all the way to Train?

Comment 2 arkady kanevsky 2021-03-30 14:14:30 UTC

We need to land it in 16.1 & 16.2.
Customer is waiting for it.
If we need to provide HF for it we can ask Customer.

Please, assign target release.
Expect it is z6 for RHOSP-16.1.

Comment 3 Alan Bishop 2021-03-30 14:16:37 UTC

Arkady, 

That's a puppet-cinder patch. Whereas we will be backporting the cinder driver change, as discussed in an email thread we plan to deliver full TripleO integration in OSP-17. In OSP-16.1 and 16.2, the driver feature can be configured using ExtraConfig hiera data.

Comment 4 arkady kanevsky 2021-03-30 14:23:25 UTC

Alan,
thanks for quick response.
I am a bit confused now.
I am under assumption that this BZ is for cinder driver.
While 1930267, 1941598, and 1941616  are all handling 3 parts of TripleO for it.

As you suggested customer accepted that they will use ControllerExtraConfig and not use for TripleO patches for XIO port filtering.
But we do need to land cinder driver patch and maybe even deliver it to Customer as HotFix if z6 is too late.

Comment 5 Alan Bishop 2021-03-30 14:32:56 UTC

Arkady,

Yes, this BZ is for the cinder driver, but the patch you referenced in comment #1 is puppet-cinder. I'm guessing you meant to reference [1]

[1] https://review.opendev.org/c/openstack/cinder/+/778982.

Because [1] was accepted as a bug fix on a stable branch, we would like your team to continue the backports to stable/train. However, we will do a downstream backport for OSP-16.1,2 in parallel to meet the customer commitments.

Comment 6 arkady kanevsky 2021-03-30 14:37:33 UTC

Good point Alan.
You are correct I was referring to https://review.opendev.org/c/openstack/cinder/+/783695
It has 2 +2 but not merged yet.
I will ask Dell team to backport it to Train.

But I will not ask them to backport 3 tripleO patches.

How quickly can we create HF for cinder?
And assume that it will also land the first z-stream for 16.1 and 16.2.

Comment 7 arkady kanevsky 2021-03-30 14:42:02 UTC

I stand corrected.
https://review.opendev.org/c/openstack/puppet-tripleo/+/779005 only has only 1 +2.
https://review.opendev.org/c/openstack/cinder/+/775798 is landed in master.

Comment 8 Vladislav Belogrudov 2021-03-30 15:51:55 UTC

https://review.opendev.org/c/openstack/cinder/+/783866 - Cinder Train patch.

Comment 30 errata-xmlrpc 2021-12-09 20:18:00 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 (Red Hat OpenStack Platform 16.1.7 (Train) bug fix and enhancement 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-2021:3762