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.

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:
milestone icon 2014.2
Started by
John Garbutt on 2014-04-28
Completed by
John Garbutt on 2014-09-03

Related branches

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

(?)

Work Items

Dependency tree

* Blueprints in grey have been implemented.

This blueprint contains Public information 
Everyone can see this information.