Comment 4 for bug 1691584

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to networking-powervm (stable/ocata)

Reviewed: https://review.openstack.org/466724
Committed: https://git.openstack.org/cgit/openstack/networking-powervm/commit/?id=fc311de47a1f5cca6a158f6d909845d662bc15ef
Submitter: Jenkins
Branch: stable/ocata

commit fc311de47a1f5cca6a158f6d909845d662bc15ef
Author: Sridhar Venkat <email address hidden>
Date: Mon May 22 09:00:23 2017 -0400

    Heal and optimize fails to retrieve vifs.

    heal_and_optimize flow retrieves a list of vifs
    using a method in util module. For SR-IOV agent,
    _find_vifs methid is invoked, it is common to CNA
    and VNIC objects. The vswitch_id is retrieved and
    validated, since vswitch_id is not present in VNIC
    object, getattr call fails. Since this method is
    common to both CNA and VNIC, a default return value
    of None should be included in the list of parameters.

    networking_powervm/plugins/ibm/agent/powervm/util.py
    _find_vifs method :
     Instead of getattr(vif, 'vswitch_id')
     getattr(vif, 'vswitch_id', None) should be used.

    Change-Id: Ifcbf0a9b9d5eb9a326c93cac675b19a2fefca75c
    Closes-Bug: #1691584