derive_npiv_map doesn't account for existing mappings that don't have backing ports

[Matt Rabe]

The trace bringing this into question is:

  File "/usr/lib/python2.7/dist-packages/nova_powervm/virt/powervm/volume/npiv.py", line 422, in wwpns
    port_maps = self._configure_wwpns_for_migration(fabric)

  File "/usr/lib/python2.7/dist-packages/nova_powervm/virt/powervm/volume/npiv.py", line 373, in _configure_wwpns_for_migration
    self.stg_ftsk.feed, self._fabric_ports(fabric), client_wwpns)

  File "/usr/lib/python2.7/dist-packages/pypowervm/tasks/vfc_mapper.py", line 210, in derive_npiv_map
    mapping = (vfc_map.backing_port.wwpn, fused_v_wwpn)

AttributeError: 'NoneType' object has no attribute 'wwpn'

derive_npiv_map checks for existing mappings for the WWPNs it's looking for, but if there is an existing mapping and it doesn't have a backing port, it'll raise an AttributeError.

[Eric Fried (efried)]

Anything happening here?

[Matt Rabe (mdrabe)]

Fixed under 6054, not sure if it's been released yet.

[Eric Fried (efried)]

Yup, https://github.com/powervm/pypowervm/commit/e6da979cd9f8f340ab9791d0197d5667ec2fc162

For purposes of paperwork, we consider it "released" as soon as it hits github, even in the develop branch.

Thanks Matt.