Cinder

Dell SC: Cinder driver fails on Dell API 2.2 or later.

Bug #1476794 reported by Tom Swanson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Tom Swanson
Cinder 7.0.0 "liberty"
Kilo
Fix Released
Undecided
Unassigned
Cinder 2015.1.2

Bug Description

The Dell REST API version 2.2 and later require a different payload format.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.openstack.org/204217

Changed in cinder:
assignee: nobody → Tom Swanson (tom-swanson)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (master)

Reviewed: https://review.openstack.org/204217
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=6806ea491f76be8e6717a7ec4238ef672df9b85a
Submitter: Jenkins
Branch: master

commit 6806ea491f76be8e6717a7ec4238ef672df9b85a
Author: Tom Swanson <email address hidden>
Date: Tue Jul 21 14:05:14 2015 -0500

    Dell SC: Fix legacy bug, init_conn bug and REST API bug.

    If Dell iSCSI ports are in legacy mode rather than virtual port
    mode initialize_connection would fail to return connection information.

    If initialize_connection was called more than once for the same
    connection it would attempt to map the volumes again and fail. It should
    instead just return the connection information and not try to map the
    volumes again.

    The Dell REST API 2.2 and later changed the format of the payload
    filter. We need to support either format.

    Finally consistency group support is only supported in Dell REST API
    2.1 or later. If the version is 2.0 we raise NotImplemented.

    Change-Id: Ie798c49a591abb510bd947fc89c26de18f0f3c87
    Closes-Bug: #1476786
    Closes-Bug: #1476794
    Closes-Bug: #1476797

Changed in cinder:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (stable/kilo)

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/206615

Doug Hellmann (doug-hellmann)
Changed in cinder:
milestone: none → liberty-2
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/210564

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on cinder (stable/kilo)

Change abandoned by Tom Swanson (<email address hidden>) on branch: stable/kilo
Review: https://review.openstack.org/206615
Reason: Change id was wrong. When fixed it created a new patch. That patch is here ...

https://review.openstack.org/#/c/210564/

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to cinder (stable/kilo)

Reviewed: https://review.openstack.org/210564
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=db372231d4491508046b0fbed1973259c0d11716
Submitter: Jenkins
Branch: stable/kilo

commit db372231d4491508046b0fbed1973259c0d11716
Author: Tom Swanson <email address hidden>
Date: Tue Jul 21 14:05:14 2015 -0500

    Dell SC: Fix legacy, init_conn and REST API bug

    If Dell iSCSI ports are in legacy mode rather than virtual port
    mode initialize_connection would fail to return connection information.

    If initialize_connection was called more than once for the same
    connection it would attempt to map the volumes again and fail. It should
    instead just return the connection information and not try to map the
    volumes again.

    The Dell REST API 2.2 and later changed the format of the payload
    filter. We need to support either format.

    Closes-Bug: #1476786
    Closes-Bug: #1476794
    Closes-Bug: #1476797
    (cherry picked from commit 6806ea491f76be8e6717a7ec4238ef672df9b85a)

    Conflicts:
     cinder/tests/test_dellscapi.py
     cinder/volume/drivers/dell/dell_storagecenter_api.py

    Change-Id: Ie798c49a591abb510bd947fc89c26de18f0f3c87

tags: added: in-stable-kilo
Thierry Carrez (ttx)
Changed in cinder:
milestone: liberty-2 → 7.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.