xenapi: nova_instance_uuid not set for root image
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
Undecided
|
Rick Harris |
Bug Description
In the xenapi driver, VDIs should be annotated with the disk-type and instance-uuid in the `other_config` attribute.
This is used to associate a VDI with an instance so that we can safely remove it in the event that it becomes orphaned (see cleanup_
Currently this is only being set in `create_vdi`. Unfortunately, doesn't account for root VDIs which are created implicitly by glance-plugin dropping a new VHD into the SR.
The solution is to modify the glance-plugin (or the compute-calling code) to add the other config.
For a cleanup script, we can populate VDI's that are missing this data by finding the sibling 'swap' VDI, and copying that info into the root VDI (changing disk type from swap to root).
Things to account for:
* Thin-provisioned images, does the `use_cow` path handle this?
* Bit-torrented images
* Snapshotted images
Changed in nova: | |
assignee: | nobody → Rick Harris (rconradharris) |
status: | New → In Progress |
Changed in nova: | |
milestone: | none → havana-1 |
status: | Fix Committed → Fix Released |
Changed in nova: | |
milestone: | havana-1 → 2013.2 |
I'm a fan of having nova-compute do it in a single code-path instead of spread out across plugins :)