Cinder

get_connector_properties in brick requires both iscsi and fc

Bug #1203486 reported by Avishay Traeger on 2013-07-21

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	Cinder	Fix Released	High	Avishay Traeger	Cinder 2013.2 "havana"

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.

Avishay Traeger (avishay-il) on 2013-07-21

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

Avishay Traeger (avishay-il) on 2013-07-21

Changed in cinder:
importance:	Undecided → High

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-07-21: Fix proposed to cinder (master)

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

Changed in cinder:
status:	New → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2013-07-30: Fix merged to cinder (master)

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