get_connector_properties in brick requires both iscsi and fc

Bug #1203486 reported by Avishay Traeger on 2013-07-21
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Cinder
High
Avishay Traeger

Bug Description

get_connector_properties() calls get_initiator(), which raises an exception if iSCSI is not configured. This is reasonable for an FC-only environment, and should therefore shouldn't raise an exception. Same with get_fc_hbas(), which raises RuntimeError if no FC HBAs were found.

Changed in cinder:
assignee: nobody → Avishay Traeger (avishay-il)
Changed in cinder:
importance: Undecided → High

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

Changed in cinder:
status: New → In Progress

Reviewed: https://review.openstack.org/38067
Committed: http://github.com/openstack/cinder/commit/55df41c2c6d12f6bf6586da3a66efec6bdaa75f3
Submitter: Jenkins
Branch: master

commit 55df41c2c6d12f6bf6586da3a66efec6bdaa75f3
Author: Avishay Traeger <email address hidden>
Date: Sun Jul 21 16:18:15 2013 +0300

    Allow connect by FC-only or iSCSI-only systems.

    Currently the brick code raises exceptions if it cannot determine the
    initiator name or FC HBA information. This patch makes this information
    optional, as Nova does.

    Fixes: bug 1203486

    Change-Id: I8efc101d7ac980261a8bc5d74209cafe4b06cc15

Changed in cinder:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2013-09-05
Changed in cinder:
milestone: none → havana-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2013-10-17
Changed in cinder:
milestone: havana-3 → 2013.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers