Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[release-0.53] sriov: Fix network to PCI-Address mapping #8416

Merged

Commits on Sep 6, 2022

  1. vendor, Bump multus-client to v1.3.0

    In order to use the latest network-status annotation api, that among
    else exposes pci-device information on sriov secondary nic, bumping
    multus-client vendor to v1.3.0
    
    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 6, 2022
    Configuration menu
    Copy the full SHA
    8f21437 View commit details
    Browse the repository at this point in the history

Commits on Sep 7, 2022

  1. network, vmispec: Move multus network filtering helper

    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    d58e4c6 View commit details
    Browse the repository at this point in the history
  2. virt-launcher, sriov: Drop redundant function filterVMISRIOVInterfaces

    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    9175c61 View commit details
    Browse the repository at this point in the history
  3. network, vmispec: Refactor FilterMultusNonDefaultNetworks

    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    ae97fc4 View commit details
    Browse the repository at this point in the history
  4. network: Introduce the namescheme package

    The new package is used to give consistent names to networks and their
    pod interfaces.
    The primary network is using a pod interface name (eth0) that is defined
    in this package and should be used by other packages for consistency.
    
    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    1bd2e3c View commit details
    Browse the repository at this point in the history
  5. network, namescheme: Consolidate pod interface names

    There is a need to consolidate the pod-interface name scheme of networks
    given in the VMI spec, so that they can be approached in consistent way
    in different times in the VMI's lifecycle.
    Introducing NameScheme that maps network names to
    their appropriate network/pod interface name.
    
    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    d1638ca View commit details
    Browse the repository at this point in the history
  6. network, sriov: Map sriov devices to pci addresses

    Create a map between SRIOV networks and their appropriate
    PCI-Address, using the multus network-status annotation.
    
    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    b7d5e3a View commit details
    Browse the repository at this point in the history
  7. virt-controller, vmi: Patch network-pci map annotation

    Patch the network-pci map to the annotation in virt-controller context,
    if such map is created.
    In addtion patch rbac is added to virt-controller.
    
    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    e06852e View commit details
    Browse the repository at this point in the history
  8. virt-controller, Expose network-pci-map with DownwardAPI

    Expose network-pci-map annotation using DownwardAPI feature [0].
    
    [0] https://kubernetes.io/docs/tasks/inject-data-application/downward-api-volume-expose-pod-information/
    
    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    d7172d4 View commit details
    Browse the repository at this point in the history
  9. virt-launcher, sriov: Use new Network to PCI map with fallback

    Utilize the SRIOV-network PCI-address mapping that was
    digested from the multus network-status annotation and mounted to a
    volume, in order to create the SRIOV devices.
    If the map does not exist, then the mapping will fall back to the former
    approach.
    
    Solves issue [0] reported on flawed mapping of sriov on VMIs
    with multiple SRIOV networks.
    
    [0] kubevirt#7444
    
    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 7, 2022
    Configuration menu
    Copy the full SHA
    c17a115 View commit details
    Browse the repository at this point in the history

Commits on Sep 11, 2022

  1. sriov, tests: Introduce skip to multipe SR-IOV test

    The new test added in order to check network-status annotation based
    mapping of the networks to the appropriate PCI-Address - will
    consistently pass only if the annotation exists and holds the pci-data.
    
    In older versions of Multus the network-status annotation does not hold
    the PCI-Address data (on version >=3.7 [0])
    
    [0] https://github.com/k8snetworkplumbingwg/multus-cni/releases/tag/v3.7
    
    Signed-off-by: Ram Lavi <ralavi@redhat.com>
    RamLavi committed Sep 11, 2022
    Configuration menu
    Copy the full SHA
    96dd7a8 View commit details
    Browse the repository at this point in the history