Bug 1687300 - [OSP13] Fix VF-rep lookup routine to use parent PF number
Summary: [OSP13] Fix VF-rep lookup routine to use parent PF number
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: openstack-nova
Version: 13.0 (Queens)
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: ---
Assignee: OSP DFG:Compute
QA Contact: OSP DFG:Compute
URL: https://github.com/openstack/os-vif/c...
Whiteboard:
Depends On: 1687298
Blocks: 1653846 1686758 1686772
TreeView+ depends on / blocked
 
Reported: 2019-03-11 08:50 UTC by Bertrand
Modified: 2023-03-21 19:12 UTC (History)
10 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1687298
Environment:
Last Closed: 2019-03-15 15:19:43 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Bertrand 2019-03-11 08:50:04 UTC
+++ This bug was initially created as a clone of Bug #1687298 +++

Fix VF-rep lookup routine to use parent PF number

The current logic in get_representor_port() matches the VF number with
the one in 'phys_port_name' file. If and when multiple PFs on the same
card are configured in switchdev mode, we could end up returning a wrong
representor device. Fix this by including the parent PF number also in
this logic, when 'pfXvfY' (or vfY@pfX) format is used in 'phys_port_name'
file.

github: https://github.com/openstack/os-vif/commit/84688a22db22039f5566c1439c3983db23672a45

This patch appear to be missing from nova-compute container in Queen / OSP 13

Comment 1 Bertrand 2019-03-11 08:55:54 UTC
Patch is tagged in os-vif 1.10 and above and nova requirements for queens is min os-vif 1.7 [1].


[1] https://github.com/openstack/nova/blob/stable/queens/requirements.txt#L57

Comment 3 Stephen Finucane 2019-03-15 15:17:48 UTC
This was resolved in python-os-vif-1.9.1-2, which is included in the latest container version, 13.0-81 [1], which was released yesterday. Apologies for the delay in releasing this.

[1] https://access.redhat.com/containers/?tab=package-list#/registry.access.redhat.com/rhosp13/openstack-nova-compute/images/13.0-81


Note You need to log in before you can comment on or make changes to this bug.