BrcdFCSanLookupService doesn't handle more than 1 switch

[Xing Yang]

In BrcdFCSanLookupService::get_device_mapping_from_network, there's a loop that loops through fabrics and finds visible targets and initiators. The SSH client connection was closed prematurally during the 1st loop.

finally:
    self.close_connection()

As a result, we can't communicate with the name server for the 2nd switch.

[Angela Smith (aallen-m)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/102599
Committed: https://git.openstack.org/cgit/openstack/cinder/commit/?id=0f7dba831dbab107b486f00d5b57f91d28994da0
Submitter: Jenkins
Branch: master

commit 0f7dba831dbab107b486f00d5b57f91d28994da0
Author: Angela Smith <email address hidden>
Date: Wed Jun 25 11:22:27 2014 -0700

    Fix BrcdFCSANlookupService to iterate multiple switches.

    Changed close_connection to client.close_connection.
    Fixed return map to key on fabric name instead of principal switch wwn.

    Closes-Bug: 1303460
    Change-Id: I3725c16cd416a028b85ac8ac6c0d895fd6900763