OpenStack Compute (nova)

Integrate the vmware driver with the oslo.vmware library (partial?)

Registered by Vui Lam on 2014-04-03

Now that the oslo.vmware library has been released, the vmware driver should be updated to use it. This means mostly adding new import lines, mechanical conversion of call sites and deleting existing code obsoleted by the library. Most of the work has already be done and proposed in the icehouse cycle.

Read the full specification

Blueprint information

Status:: Complete

Approver:: John Garbutt

Priority:: Medium

Drafter:: Vui Lam

Direction:: Approved

Assignee:: Vui Lam

Definition:: Approved

Series goal:: Accepted for juno

Implementation:: Implemented

Milestone target:: 2014.2

Started by: John Garbutt on 2014-04-28

Completed by: John Garbutt on 2014-09-03

Related branches

Related bugs

Sprints

Whiteboard

Most of the changes was already proposed in https://review.openstack.org/#/c/70175/ during the icehouse cycle.
The hope is that this can be fast-tracked along the vmware-spawn-refactor effort as these changes makes widespread changes to the driver code and are probably best done before anything else.

Gerrit topic: https://review.openstack.org/#q,topic:bp/use-oslo-vmware,n,z

Addressed by: https://review.openstack.org/85469
vmware: use oslo.vmware library

updated the url to match the spec, can now add to juno series --johnthetubaguy (28th April 2014)

Need this to merge early, and due to mixing with the refactoring, and other things, moving this to medium. Approving this as we have some code that needs restoring, that covers most of this --johnthetubaguy

Pending changes
==============

https://review.openstack.org/#/c/70175/ (manually added for now)

No code really up, so pushing to Juno-2 --johnthetubaguy (30th May 2014)

Addressed by: https://review.openstack.org/70175
vmwareapi oslo.vmware library integration

Gerrit topic: https://review.openstack.org/#q,topic:vim-prop,n,z

Reviews do not have +2s and are not close to approval, so moving to juno-3. But please move this back to juno-2, should you get your patches approved in time. --johnthetubaguy 21st July 2014

Gerrit topic: https://review.openstack.org/#q,topic:use-oslo-vmware,n,z

Addressed by: https://review.openstack.org/100927
VMware: remove _get_vim() from VMwareAPISession

Addressed by: https://review.openstack.org/96140
Add VIF_VHOSTUSER

Gerrit topic: https://review.openstack.org/#q,topic:bp/vmware-spbm-support,n,z

Gerrit topic: https://review.openstack.org/#q,topic:bug/1363349,n,z

Gerrit topic: https://review.openstack.org/#q,topic:oslo.vmware,n,z

Basically this is one patch, and its in the gate:
https://review.openstack.org/#/c/70175/
--johnthetubaugy 2nd September 2014

Addressed by: https://review.openstack.org/125073
VMware: use oslo.vmware methods for handling tokens

(?)