SR-IOV: deprecate supported_pci_vendor_devs

Bug #1611302 reported by Moshe Levi
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fix Released
Moshe Levi

Bug Description

To reduce complexity in configuring SR-IOV I want to deprecate
the supported_pci_vendor_devs option. This option is doing extra
validation that pci vendor id and product id provided by
nova in the neutron port binding profile is matching
to the vendor id and product id in supported_pci_vendor_devs.
This is redundant, because nova-scheduler is the point to do
validation and select a suitable hypervisor and The compute node is
already validating this through the pci_passthrough_whitelist
option in nova.conf


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

Fix proposed to branch: master

Changed in neutron:
assignee: nobody → Moshe Levi (moshele)
status: New → In Progress
tags: added: deprecation
Changed in neutron:
importance: Undecided → Low
Changed in neutron:
assignee: Moshe Levi (moshele) → edan david (edand)
Changed in neutron:
assignee: edan david (edand) → Moshe Levi (moshele)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Submitter: Jenkins
Branch: master

commit de31df4211ca2b05c331ad910578893fe92052f4
Author: Moshe Levi <email address hidden>
Date: Mon Aug 8 20:44:30 2016 +0300

    SR-IOV: deprecate supported_pci_vendor_devs

    Deprecate the supported_pci_vendor_devs option in order to reduce
    complexity for configuring SR-IOV. Currently, neutron validates the pci
    vendor and product id. However this check is already done by the
    nova-scheduler when it selects a suitable hypervisor. More precisely,
    the compute node validates this through the pci_passthrough_whitelist
    option in nova.conf. Therefore this check in neutron is redundant.

    This patch deprecates the supported_pci_vendor_devs in Newton release
    and updates the supported_pci_vendor_devs default to None.
    In case of None value the SR-IOV mechanism driver won't do
    any pci vendor validation. In case this option is set
    the SR-IOV mechanism driver will do the validaiton as it was

    Closes-bug: #1611302

    Change-Id: Id5e2cef44da871965583abbae3e1140fd4f5786c

Changed in neutron:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/neutron

This issue was fixed in the openstack/neutron development milestone.

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.