Uploaded image for project: 'OpenShift Virtualization'
  1. OpenShift Virtualization
  2. CNV-30120

GA: Bridged network interface hot-plug for VMs

XMLWordPrintable

    • ga-bridge-nic-hotplug
    • Hide
      • All acceptance criteria of its Tech Preview predecessor
      • Hot-plug API is simplified and we are confident with the implementation
      • QE perform full regression testing + post-upgrade to validate API changes.
      • Tech preview note removed from docs
      • Tech preview note removed from UI
      • Docs adjusted to the new API, tracked in CNV-30166
      • UI adjusted to the new API
      • Enabled by default in CNV
      • (Must have) Test that the API enables hot plugging of bridged network interfaces for VMs without errors.
      Show
      All acceptance criteria of its Tech Preview predecessor Hot-plug API is simplified and we are confident with the implementation QE perform full regression testing + post-upgrade to validate API changes. Tech preview note removed from docs Tech preview note removed from UI Docs adjusted to the new API, tracked in CNV-30166 UI adjusted to the new API Enabled by default in CNV (Must have) Test that the API enables hot plugging of bridged network interfaces for VMs without errors.
    • Green
    • In Progress
    • CNV-25929 - GA: vNIC hot-plug for bridged networks
    • CNV-25929GA: vNIC hot-plug for bridged networks
    • 0% To Do, 0% In Progress, 100% Done
    • dev-ready, doc-ready, po-ready, qe-ready, ux-ready
    • Hide

      2024-02-19: QE finished, pending documentation....

      Show
      2024-02-19: QE finished, pending documentation....

      Goal

      Allow hot plug of bridged network interfaces for VMs.

      User Stories

      • As a developer,
        I expect hotplug being available in CNV platform,
        since it is an industry standard I was using in the previous platform.

      Non-Requirements

      • <List of things not included in this epic, to alleviate any doubt raised during the grooming process.>
      • This epic is not concerned with SR-IOV NIC hotplug, SR-IOV is tracked here: https://issues.redhat.com/browse/CNV-25461
      • Hot-unplug is not needed
      • In-place hot-plug is not a must, live-migration based hot-plug is also acceptable

      Notes

      • VM interface hot-plug should also be supported on cluster with multus version 3.
      • Storage already supports hot-plug, we should learn from them.
      • Support of hot-plug is being considered for CNI 2.0.
      • Multus RFE: https://github.com/k8snetworkplumbingwg/multus-cni/issues/585
      • One option to handle scheduling would be to a) If the needed resource is available, just plug it in b) if it is not, fail or migrate
      • We could use RHV test-plan as a reference
      • Epic https://issues.redhat.com/browse/CNV-21833 and specifically PR https://github.com/kubevirt/kubevirt/pull/6852 adding support to in-place hotplug, but only in case the cluster has multus version 4 or above.
      • The interfaces will first appear on the VMI spec and the VM spec if the request contains `--persist`. Only after migration the VM, the interfaces will appear in the virt-launcher pod and the guest.

      Design outline

      https://docs.google.com/document/d/1oEuw0UIQFYHTVPRcS6a-QtDdWW2yeyOAWmlg5C1eWyU/edit

      Done Checklist

      Who What Reference
      DEV Upstream roadmap issue https://github.com/kubevirt/kubevirt/issues/6733
      DEV Upstream code and tests merged Just testing
      DEV Upstream documentation merged https://kubevirt.io/user-guide/operations/hotplug_interfaces/#migration-based-hotplug
      DEV gap doc updated Yes
      DEV Upgrade consideration Hot-plug on VMs started before this feature was introduced is not allowed
      DEV CEE/PX summary presentation N/A
      QE Test plans in Polarion Polarion test plan
      QE Automated tests merged Automated bridge hot-plug tests
      DOC Downstream documentation merged https://github.com/openshift/openshift-docs/pull/71247

            alkaplan@redhat.com Alona Kaplan
            fdeutsch@redhat.com Fabian Deutsch
            Yossi Segev Yossi Segev
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: