SR-IOV: PCI devices aren't reconfigured when deallocated

[Brent Eagles]

Note that this only affects SR-IOV configuration support and the physical function use case!

When an SR-IOV physical function is allocated to a VM, the PF and it's associated resources (e.g. virtual functions) "disappear" from the hypervisor host. When the VM is destroyed, the physical function may become available again but the scripts created by the current SR-IOV support are not run again. The result is that the virtual functions will no longer be available.

The implementation for configuring the virtual functions should be extended to execute when the physical functions are released back to the system.

NOTE: the SR-IOV hosts should have the NICs for the physical functions in the network configuration templates with HOTPLUG=yes. AFAICT, os-net-config doesn't support this at the moment.

[Brent Eagles (beagles)]

I've assigned an importance of "medium" because doesn't affect tripleo itself or the majority of the users, but it is a pretty big deal for for NFV customers using SR-IOV.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/394604
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=9a79bdab86224e678414fd98ba2e888836083e79
Submitter: Jenkins
Branch: master

commit 9a79bdab86224e678414fd98ba2e888836083e79
Author: Brent Eagles <email address hidden>
Date: Mon Nov 7 16:22:31 2016 -0330

    Call VF configuration from udev rules

    When a physical function that was allocated to a guest is released back
    the system, it is not automatically brought "up" and the VF
    configuration is not restored. This patch creates a file containing some
    udev rules to force the VF configuration.

    Note: we may find that the ifup-local script is no longer required but
    this will require further testing.

    Change-Id: Ie6e78730aa0a748b3b5100ab7c7bc007d8ab176d
    Partial-Bug: #1639901

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/423312

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/423312
Committed: https://git.openstack.org/cgit/openstack/puppet-tripleo/commit/?id=a3b0c4f536bf50fc7ff72a8f20aa333938d9dd3d
Submitter: Jenkins
Branch: stable/newton

commit a3b0c4f536bf50fc7ff72a8f20aa333938d9dd3d
Author: Brent Eagles <email address hidden>
Date: Mon Nov 7 16:22:31 2016 -0330

    Call VF configuration from udev rules

    When a physical function that was allocated to a guest is released back
    the system, it is not automatically brought "up" and the VF
    configuration is not restored. This patch creates a file containing some
    udev rules to force the VF configuration.

    Note: we may find that the ifup-local script is no longer required but
    this will require further testing.

    Change-Id: Ie6e78730aa0a748b3b5100ab7c7bc007d8ab176d
    Partial-Bug: #1639901
    (cherry picked from commit 9a79bdab86224e678414fd98ba2e888836083e79)

[Brent Eagles (beagles)]

Relevant patches were merged but didn't automatically update launchpad.