[Bug 1959712] Re: [SRU] Add iSCSI support to HPE 3PAR driver for Primera 4.2 and higher

[Nobuto Murata]

Tested and verified with focal-proposed.

apt-cache policy cinder-common
cinder-common:
  Installed: 2:16.4.2-0ubuntu2
  Candidate: 2:16.4.2-0ubuntu2
  Version table:
 *** 2:16.4.2-0ubuntu2 500
        500 http://archive.ubuntu.com/ubuntu focal-proposed/main amd64 Packages
        100 /var/lib/dpkg/status
     2:16.4.1-0ubuntu1 500
        500 http://archive.ubuntu.com/ubuntu focal-updates/main amd64 Packages
     2:16.1.0-0ubuntu1 500
        500 http://security.ubuntu.com/ubuntu focal-security/main amd64 Packages
     2:16.0.0~b3~git2020041012.eb915e2db-0ubuntu1 500
        500 http://archive.ubuntu.com/ubuntu focal/main amd64 Packages

[before applying -proposed]

2022-02-24 06:59:34.018 52526 ERROR cinder.volume.drivers.hpe.hpe_3par_iscsi [req-65769a6c-7b72-49b4-84e8-dac7bbacc5d1 - - - - -] For Primera, only FC is supported. iSCSI cannot be used
2022-02-24 06:59:34.018 52526 ERROR cinder.volume.manager [req-65769a6c-7b72-49b4-84e8-dac7bbacc5d1 - - - - -] Failed to initialize driver.: NotImplementedError
2022-02-24 06:59:34.018 52526 ERROR cinder.volume.manager Traceback (most recent call last):
2022-02-24 06:59:34.018 52526 ERROR cinder.volume.manager   File "/usr/lib/python3/dist-packages/cinder/volume/manager.py", line 466, in _init_host
2022-02-24 06:59:34.018 52526 ERROR cinder.volume.manager     self.driver.do_setup(ctxt)
2022-02-24 06:59:34.018 52526 ERROR cinder.volume.manager   File "/usr/lib/python3/dist-packages/cinder/volume/drivers/hpe/hpe_3par_base.py", line 438, in do_setup
2022-02-24 06:59:34.018 52526 ERROR cinder.volume.manager     self._do_setup(common)
2022-02-24 06:59:34.018 52526 ERROR cinder.volume.manager   File "/usr/lib/python3/dist-packages/cinder/volume/drivers/hpe/hpe_3par_iscsi.py", line 149, in _do_setup
2022-02-24 06:59:34.018 52526 ERROR cinder.volume.manager     raise NotImplementedError()
2022-02-24 06:59:34.018 52526 ERROR cinder.volume.manager NotImplementedError


[after applying -proposed]

2022-02-24 08:59:05.246 78217 INFO
cinder.volume.flows.manager.create_volume
[req-3599f424-29db-45b4-bf2d-4a665f3a30b0
91f76503c81d4398ad8f7ec426d2f149 9c05cd46a17c460d87ca375f20a20803 -
5c2cd4d3196643e2b6a4d8f7dc64eef0 5c2cd4d3196643e2b6a4d8f7dc64eef0]
Volume volume-2bfb4855-b8c9-4862-9d83-293ff43c6a32
(2bfb4855-b8c9-4862-9d83-293ff43c6a32): created successfully


** Tags removed: verification-needed verification-needed-focal
** Tags added: verification-done verification-done-focal

-- 
You received this bug notification because you are a member of Ubuntu
OpenStack, which is subscribed to cinder in Ubuntu.
https://bugs.launchpad.net/bugs/1959712

Title:
  [SRU] Add iSCSI support to HPE 3PAR driver for Primera 4.2 and higher

Status in Ubuntu Cloud Archive:
  Invalid
Status in Ubuntu Cloud Archive ussuri series:
  Fix Committed
Status in Ubuntu Cloud Archive victoria series:
  Fix Committed
Status in cinder package in Ubuntu:
  Invalid
Status in cinder source package in Focal:
  Fix Committed

Bug description:
  [Description]
  OpenStack cinder in Focal (OpenStack Ussuri) is lacking iSCSI support for HPE Primera 4.2 and higher. This is now supported in Cinder and we would like to enable it in Ubuntu Focal as well as OpenStack Ussuri.

  The rationale for this SRU falls under harware enablement for Long
  Term Support releases.

  [Test Case]
  1. Deploy OpenStack with:
   - HPE 3PAR *iSCSI* driver enabled for Cinder
   - with Primera >= 4.2
    as per: https://docs.openstack.org/cinder/latest/configuration/block-storage/drivers/hpe-3par-driver.html
  2. Create a volume
    openstack volume create (--type hpe-primera) --size 10 sru-test-before

  3. Confirm the volume is successfully created while it gave an error
  as "For Primera, only FC is supported. iSCSI cannot be used"
  previously.

  [Regression Potential]
  This is a pretty simple patch. Where the code used to always raise a not implemented error for Primera, it now raises the not implemented error only for < Primera 4.2. Where things could go wrong? It's possible the version checking could have an issue but that code seems standard and the upstream patch has been merged back to stable/victoria [1]. Regression testing will cover that code path as well.
  [1] https://review.opendev.org/c/openstack/cinder/+/820611

To manage notifications about this bug go to:
https://bugs.launchpad.net/cloud-archive/+bug/1959712/+subscriptions