Renaming vf representros is failing if we have vfs over different Nics with switchdev

Bug #1823526 reported by waleed mousa
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
os-net-config
Fix Released
Undecided
waleed mousa

Bug Description

It seems that if have more different NICS with switchdev mode in sriov, the names of the represntors will be overlapped for those NICS, like pf0vf0 or the first port of the first NIC and pfvf0 for the first port of the second NIC, so we need to make the name related the name of the port itself

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-net-config (master)

Fix proposed to branch: master
Review: https://review.openstack.org/651751

Changed in os-net-config:
assignee: nobody → waleed mousa (waleedm)
status: New → In Progress
Changed in os-net-config:
assignee: waleed mousa (waleedm) → Alex Schultz (alex-schultz)
Changed in os-net-config:
assignee: Alex Schultz (alex-schultz) → waleed mousa (waleedm)
Changed in os-net-config:
assignee: waleed mousa (waleedm) → Alex Schultz (alex-schultz)
Changed in os-net-config:
assignee: Alex Schultz (alex-schultz) → waleed mousa (waleedm)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-net-config (master)

Reviewed: https://review.opendev.org/651751
Committed: https://git.openstack.org/cgit/openstack/os-net-config/commit/?id=22aeee1b87731618356f2792d49795a01f40ec19
Submitter: Zuul
Branch: master

commit 22aeee1b87731618356f2792d49795a01f40ec19
Author: waleed mousa <email address hidden>
Date: Thu Apr 11 06:44:36 2019 -0400

    Modify udev rule for VF representor rename

    It seems that if we have more than one NIC in SR_IOV switchdev mode,
    the names of the represntors will overlap for those NICS,
    like pf0vf0 for the first port of the first NIC and pf0vf0 for the
    first port of the second NIC, so we need to rename the VF
    representor as "<sriov_pf_name>_<vf_num>" "enp3s0f0_2" in order to
    be unique and consistent

    Change-Id: I25263926e736ed0e4fafaa9aa9891e5961226462
    Closes-Bug: #1823526

Changed in os-net-config:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-net-config (stable/stein)

Fix proposed to branch: stable/stein
Review: https://review.opendev.org/658637

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-net-config (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.opendev.org/658638

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to os-net-config (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.opendev.org/658639

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-net-config (stable/stein)

Reviewed: https://review.opendev.org/658637
Committed: https://git.openstack.org/cgit/openstack/os-net-config/commit/?id=bb5d8cb046c1683d437d08db8132c5a40dbb30c4
Submitter: Zuul
Branch: stable/stein

commit bb5d8cb046c1683d437d08db8132c5a40dbb30c4
Author: waleed mousa <email address hidden>
Date: Thu Apr 11 06:44:36 2019 -0400

    Modify udev rule for VF representor rename

    It seems that if we have more than one NIC in SR_IOV switchdev mode,
    the names of the represntors will overlap for those NICS,
    like pf0vf0 for the first port of the first NIC and pf0vf0 for the
    first port of the second NIC, so we need to rename the VF
    representor as "<sriov_pf_name>_<vf_num>" "enp3s0f0_2" in order to
    be unique and consistent

    Change-Id: I25263926e736ed0e4fafaa9aa9891e5961226462
    Closes-Bug: #1823526
    (cherry picked from commit 22aeee1b87731618356f2792d49795a01f40ec19)

tags: added: in-stable-stein
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-net-config (stable/rocky)

Reviewed: https://review.opendev.org/658638
Committed: https://git.openstack.org/cgit/openstack/os-net-config/commit/?id=9ed8760c0f40b8149d293aabf110392265daf60d
Submitter: Zuul
Branch: stable/rocky

commit 9ed8760c0f40b8149d293aabf110392265daf60d
Author: waleed mousa <email address hidden>
Date: Thu Apr 11 06:44:36 2019 -0400

    Modify udev rule for VF representor rename

    It seems that if we have more than one NIC in SR_IOV switchdev mode,
    the names of the represntors will overlap for those NICS,
    like pf0vf0 for the first port of the first NIC and pf0vf0 for the
    first port of the second NIC, so we need to rename the VF
    representor as "<sriov_pf_name>_<vf_num>" "enp3s0f0_2" in order to
    be unique and consistent

    Change-Id: I25263926e736ed0e4fafaa9aa9891e5961226462
    Closes-Bug: #1823526
    (cherry picked from commit 22aeee1b87731618356f2792d49795a01f40ec19)

tags: added: in-stable-rocky
tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to os-net-config (stable/queens)

Reviewed: https://review.opendev.org/658639
Committed: https://git.openstack.org/cgit/openstack/os-net-config/commit/?id=cd0aa7db4f16a22e770971b848234260e5c24898
Submitter: Zuul
Branch: stable/queens

commit cd0aa7db4f16a22e770971b848234260e5c24898
Author: waleed mousa <email address hidden>
Date: Thu Apr 11 06:44:36 2019 -0400

    Modify udev rule for VF representor rename

    It seems that if we have more than one NIC in SR_IOV switchdev mode,
    the names of the represntors will overlap for those NICS,
    like pf0vf0 for the first port of the first NIC and pf0vf0 for the
    first port of the second NIC, so we need to rename the VF
    representor as "<sriov_pf_name>_<vf_num>" "enp3s0f0_2" in order to
    be unique and consistent

    Change-Id: I25263926e736ed0e4fafaa9aa9891e5961226462
    Closes-Bug: #1823526
    (cherry picked from commit 22aeee1b87731618356f2792d49795a01f40ec19)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/os-net-config 11.0.0

This issue was fixed in the openstack/os-net-config 11.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/os-net-config 9.4.0

This issue was fixed in the openstack/os-net-config 9.4.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/os-net-config 8.5.0

This issue was fixed in the openstack/os-net-config 8.5.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/os-net-config 10.4.1

This issue was fixed in the openstack/os-net-config 10.4.1 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.