Comment 3 for bug 1927758

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to stx-puppet (master)

Reviewed: https://review.opendev.org/c/starlingx/stx-puppet/+/790300
Committed: https://opendev.org/starlingx/stx-puppet/commit/97371409b9b2ae3f0db6a6a0acaeabd74927160e
Submitter: "Zuul (22348)"
Branch: master

commit 97371409b9b2ae3f0db6a6a0acaeabd74927160e
Author: Steven Webster <email address hidden>
Date: Fri May 7 15:33:43 2021 -0400

    Add SR-IOV rate-limit dependency

    Currently, the binding of an SR-IOV virtual function (VF) to a
    driver has a dependency on platform::networking. This is needed
    to ensure that SR-IOV is enabled (VFs created) before actually
    doing the bind.

    This dependency does not exist for configuring the VF rate-limits
    however. There is a chance that the VF rate-limiting configuration
    happens before the VFs are actually created.

    This commit fixes the issue by creating a dependency on
    platform::networking from the sriov::config class, which ensures
    the VFs are created before both driver binding and rate
    limiting configuration occurs.

    Closes-Bug: #1927758
    Signed-off-by: Steven Webster <email address hidden>
    Change-Id: Ic452247eb8c980e1b18bdc54832eb635d7a9fc54