CiscoFCSanLookupService.get_device_mapping_from_network method should not use zoning_vsan as the key of the device mapping dictionary
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
New
|
Undecided
|
Unassigned |
Bug Description
In Cisco fc san lookup service implementation, source code from master branch below:
In CiscoFCSanLooku
Because this method looks up through all the san fabrics defined in Cinder configuration, and users may define multiple san fabrics
in the configuration, e.g., multiple principle switch IP addresses, so Cinder can manage multiple san fabrics, and for each principle switch, the zoning vsan can be the same. For example, users can define a vsan id 15 in switch 1 as well as switch 2. Let's assume WWN1 is in vsan 15 of switch 1, and WWN2 in vsan 15 of switch 2. Then the CiscoFCSanLooku
{
'15': {
'initiator_
'target_
}
}
This is wrong. Instead, we should use fabric_name as the key, just as Brocade lookup service does.
tags: | added: cisco |
Changed in cinder: | |
assignee: | nobody → Al Lau (alau2) |
tags: | added: zone-manager |
As described in the submitter's comment, I will make the change.