Dell SC driver only honoring vol folder name on create

Bug #1449290 reported by Tom Swanson
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
Fix Released
Undecided
Tom Swanson

Bug Description

The dell_sc_volume_folder cinder.conf option for the Dell Storage Center iSCSI and FC drivers is only honored on volume creation. If the volume was duplicated with the same name the driver would use the first volume it found regardless of its location.

This should be changed to use the volume found in dell_sc_volume_folder if there is one before searching the rest of the array.

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/177994

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/177994
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=5a5b9ca3ed013dd6cfcda393709c4e130ee3a9ab
Submitter: Jenkins
Branch: master

commit 5a5b9ca3ed013dd6cfcda393709c4e130ee3a9ab
Author: Tom Swanson <email address hidden>
Date: Mon Apr 27 17:33:40 2015 -0500

    Dell SC driver honoring folder name after volume creation.

    The Dell SC driver was not honoring the volume folder name
    after creation. This has been changed so that find_volume
    now checks the dell_sc_volume_folder for the requested volume
    before doing a general search of the storage center. In
    either case if more than one volume is returned (two in
    the dell_sc_volume_folder or two outside of that folder) then
    an exception is thrown.

    Also the dell_sc_volume_folder, dell_sc_server_folder and
    dell_sc_ssn variables are no longer being passed arond. They
    are saved as part of the backend API object.

    Tests have been updated.

    Change-Id: I962ea3ed262d21deff65cdc8476939aab6ba07e2
    Closes-Bug: #1449290

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in cinder:
milestone: none → liberty-1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in cinder:
milestone: liberty-1 → 7.0.0
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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