Nicira Plugin need to work with different versions of the NVP API

Bug #1126352 reported by Salvatore Orlando
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Fix Released
Medium
Salvatore Orlando

Bug Description

The Nicira plugin is currently targeting a specific version of the NVP API.
We need to ensure the same plugin can work even with different versions of the NVP controllers.

This is important in order to allow NVP to be upgraded independently of Quantum.

Changes should be limited to nvplib, the module for interfacing with the nvp api.
This could be achieved, for instance, using some for of monkey patching to methods whose implementation changes from one version to another.

Note: This cannot possibly be a general solution, but will work perfectly when there are minor changes in the NVP API that do not require any change in the Quantum logic

Tags: nicira
Revision history for this message
Salvatore Orlando (salvatore-orlando) wrote :

Setting importance to 'medium'.
The plugin maintainer (arosen) will set definitive important and milestone target.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to quantum (master)

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

Changed in quantum:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to quantum (master)

Reviewed: https://review.openstack.org/22037
Committed: http://github.com/openstack/quantum/commit/b17bd081a16513307e5b8dd8a2d738173050ff93
Submitter: Jenkins
Branch: master

commit b17bd081a16513307e5b8dd8a2d738173050ff93
Author: Salvatore Orlando <email address hidden>
Date: Mon Jan 28 16:41:53 2013 -0800

    Allow nicira plugin to handle multiple NVP API versions

    Bug #1126352

    This patch introduces a simple framework for enabling
    nvlib to call the appropriate routine according to the
    current version. To this aim, we leverage the 'server' header
    which is returned by every NVP API calls (except login/logout).
    The patch also accounts for the changes introduced in NVP 3.0

    Change-Id: Ief3a6f2b5b99ea33548353ce3bee3fa23e42752f

Changed in quantum:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in quantum:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in quantum:
milestone: grizzly-3 → 2013.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.