Convert OVS os-vif driver to use native Python bindings

Bug #1666917 reported by sean mooney on 2017-02-22
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
os-vif
Low
Rodolfo Alonso

Bug Description

Today os-vif uses ovs-vsctl via processutils. This spawns a new shell for every invocation and repeatedly dumps the ovsdb into memory which is both inefficient and slow. OVS has native Python binding for configuring the ovsdb which would allow us to avoid these issues.

The OVS plugin should be rewritten to leverage the native python bindings
to improve performance, reduce memory footprint and improve security by reducing use of processutils.

Tags: rfe Edit Tag help
Changed in os-vif:
importance: Undecided → Low
sean mooney (sean-k-mooney) wrote :

investigate using new ovsdbapp lib for this as neutron will be moving to it in the future
https://github.com/openstack/ovsdbapp/

summary: - convert ovs os-vif driver to use nativ python bindings
+ convert ovs os-vif driver to use native Python bindings
summary: - convert ovs os-vif driver to use native Python bindings
+ Convert ovs os-vif driver to use native Python bindings
description: updated
summary: - Convert ovs os-vif driver to use native Python bindings
+ Convert OVS os-vif driver to use native Python bindings

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

Changed in os-vif:
assignee: sean mooney (sean-k-mooney) → Rodolfo Alonso (rodolfo-alonso-hernandez)
status: New → In Progress

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

Pranab (pranabjb) on 2018-03-06
Changed in os-vif:
assignee: Rodolfo Alonso (rodolfo-alonso-hernandez) → Pranab (pranabjb)
Changed in os-vif:
assignee: Pranab (pranabjb) → Rodolfo Alonso (rodolfo-alonso-hernandez)
Changed in os-vif:
assignee: Rodolfo Alonso (rodolfo-alonso-hernandez) → Pranab (pranabjb)
Changed in os-vif:
assignee: Pranab (pranabjb) → Rodolfo Alonso (rodolfo-alonso-hernandez)
Changed in os-vif:
assignee: Rodolfo Alonso (rodolfo-alonso-hernandez) → sean mooney (sean-k-mooney)
Changed in os-vif:
assignee: sean mooney (sean-k-mooney) → Rodolfo Alonso (rodolfo-alonso-hernandez)

Reviewed: https://review.openstack.org/476612
Committed: https://git.openstack.org/cgit/openstack/os-vif/commit/?id=7bfb01bcb151087d70df115ba01f789a8a7f161f
Submitter: Zuul
Branch: master

commit 7bfb01bcb151087d70df115ba01f789a8a7f161f
Author: Rodolfo Alonso Hernandez <“<email address hidden>”>
Date: Thu Sep 20 16:22:00 2018 +0100

    Add abstract OVSDB API

    Abstract all existing 'ovs-vsctl' calls to an abstract OVSDB API.
    This will allow the future addition of a native OVSDB protocol
    implementation of the API without breaking backwards compatibility.

    Partial-Bug: #1666917
    Change-Id: I9ec36be390d713a928a673191008612f3fddad8f

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers