[BFE] Merge ~citrix-openstack/nova/xenapi-glance-2 for bexar-xenapi-support-for-glance blueprint
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| OpenStack Compute (nova) |
Opinion
|
Undecided
|
Unassigned | ||
Bug Description
This is a branch freeze exception request, asking to take the work that enables Glance support when using the XenAPI virtualization backend.
Blueprint: bexar-xenapi-
Benefit of the branch: At the moment, XenAPI only supports nova-objectstore for VM images. Adding Glance is obviously a big step towards the architecture that we want, as nova-objectstore is already deprecated. This will also help work on other blueprints, such as xs-snapshots, as those blueprints also depend upon glance support.
Risk of regression: This branch only affects nova.virt.xenapi and its unit tests, apart from minor bugfixes to nova.images.glance and the addition of glance to the pip-requires file. This makes the chance of regression for non-XenAPI users negligible. For XenAPI users, the risk is obviously slightly higher, but since most of the changes are adding new code branches to support Glance, this risk is still reasonably low.
Related branches
- Sandy Walsh (community): Approve
- Ed Leafe (community): Approve
- Thierry Carrez (community): Approve (ffe)
- Jay Pipes (community): Needs Information
- Rick Harris: Pending requested
-
Diff: 869 lines (+504/-70)10 files modifiednova/tests/glance/__init__.py (+20/-0)
nova/tests/glance/stubs.py (+37/-0)
nova/tests/test_xenapi.py (+72/-38)
nova/tests/xenapi/stubs.py (+19/-5)
nova/virt/xenapi/fake.py (+77/-10)
nova/virt/xenapi/vm_utils.py (+242/-14)
nova/virt/xenapi/vmops.py (+2/-1)
nova/virt/xenapi_conn.py (+3/-0)
plugins/xenserver/xenapi/etc/xapi.d/plugins/glance (+31/-2)
tools/pip-requires (+1/-0)
| Thierry Carrez (ttx) wrote | #1 |
| Changed in nova: | |
| status: | New → Opinion |
From a product perspective, this branch makes a lot of sense, so I support this... Though the branches proposed before get priority, so this could need a feature freeze exception soon :)