[RFE] Provide separation between VNIC and VIF in core Nova and Neutron

Bug #1616388 reported by Jan Gutter
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Opinion
Wishlist
Unassigned
neutron
Won't Fix
Wishlist
Unassigned

Bug Description

It might be easier to view the VM plugging process in OpenStack as composed of three partners, attributes of a Neutron port:
* The VIF edge (plugging in a TAP device into a bridge, configuring a NIC's VEB, some other form of virtual port manipulation…)
* The VNIC edge (hypervisor config, emulated hardware model, etc.)
* The transport mechanism (vhost-net, vhost-user, vfio for SR-IOV, etc.)

Currently, there are a few places in core OpenStack that conflate the three concepts. For example, nova/network/model.py has VNIC_TYPE_* that denote the different methods for connecting to the hardware, but VIF_MODEL_* denotes different hardware emulation settings for the hypervisors.

Compounding this problem is the current plugging method with libvirt. The plugging logic for many mechanism drivers moved into libvirt, meaning that nova passes both VNIC and VIF information to libvirt in some cases and not in others. The OS-VIF library is a step away from that direction: https://blueprints.launchpad.net/nova/+spec/os-vif-library

This RFE requests a mechanism to allow for more granularity in the plugging logic semantics: a mechanism driver should not need to re-invent common plugging code on the VIF or VNIC side. For example, just as primitives for plugging a netdev into a bridge should be part of the OS-VIF library, so should the VNIC configuration by the hypervisor be as cleanly abstracted and separated as possible.

Initially, the hypervisor driver should receive both VNIC and VIF objects and have logic to decide to generate the required VM configuration. For example if an OVS VIF and vhost-net VNIC is passed to the libvirt driver, it generates xml to handle the bridge and VM plugging. In the case a driver requires another method of plugging, but can re-use the OVS VIF or the VNIC code it should be able to do so.

Netronome is willing to devote resources to this project in order to improve the OpenStack infrastructure and reduce code duplication.

Jan Gutter (jangutter)
affects: devstack → nova
description: updated
Changed in neutron:
status: New → Confirmed
importance: Undecided → Wishlist
Matt Riedemann (mriedem)
Changed in nova:
status: New → Opinion
importance: Undecided → Wishlist
Revision history for this message
Kevin Benton (kevinbenton) wrote :

This sounds good, but I don't think there is anything we can do on the neutron side to help with this. Or am I missing something?

Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

Author never provided more details as requested.

Changed in neutron:
status: Confirmed → Incomplete
Revision history for this message
Slawek Kaplonski (slaweq) wrote :

As was discussed on the PTG, we decided to close this RFE for now. Feel free to reopen it if there will be valid use case for that and if You will want to work on that. Then we can discuss it again in the drivers team meeting.

Changed in neutron:
status: Incomplete → Opinion
tags: added: rfe-postponed
removed: rfe
Revision history for this message
Brian Haley (brian-haley) wrote :

Closing based on last comment and since no specs or patches were proposed.

Changed in neutron:
status: Opinion → Won't Fix
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.