vmbuilder XML is incomplete with libvirt 0.8.3 (No bootable devices)
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| vm-builder (Ubuntu) |
Fix Released
|
High
|
Jamie Strandboge | ||
| Lucid |
Fix Released
|
High
|
Jamie Strandboge | ||
| Maverick |
Fix Released
|
High
|
Jamie Strandboge | ||
| Natty |
Fix Released
|
High
|
Jamie Strandboge | ||
Bug Description
Binary package hint: vmbuilder
SRU
1. libvirt 0.8.3 and later (and the upcoming libvirt update for lucid-security) does not probe qemu disks for the image format and defaults to 'raw' when the format is not specified in the XML. This breaks vmbuilder because the version in maverick does not specify a format in the XML.
WORKAROUND:
After creating, use '/usr/sbin/
2. this is not yet fixed upstream (or Natty), but is ACKd by upstream
3. Patch adjusts VMbuilder/disk.py to have a new 'self.format_type' attribute. This attribute will normally be 'None', but will be set when a disk is convert()ed. VMbuilder/
4a. TEST CASE 1 (in maverick release):
$ sudo vmbuilder kvm ubuntu --suite lucid --flavour virtual --arch i386 -o --libvirt qemu:///system
$ virsh start ubuntu
$ virt-viewer ubuntu
At this point the VM will have started, but shows 'No bootable devices' if unpatched. If patched, it should boot up fine.
4b. TEST CASE 2 (don't regress --raw (first delete the previous 'ubuntu' domain)):
$ qemu-img create /tmp/foo.img 5G
$ sudo /sbin/losetup /dev/loop1 /tmp/foo.img
$ sudo vmbuilder kvm ubuntu --suite lucid --flavour virtual --arch i386 -o --libvirt qemu:///system --raw=/dev/loop1
$ virsh --connect=
$ virt-viewer --connect=
At this point the VM should be started and running.
5. Risk of regression is considered low. Patch is not intrusive and only will add the extra XML only if a disk is converted, so it should affect no other parts of libvirt. Since vm-builder with libvirt and qcow2 disks (the default) are currently broken, there is no risk of regression in the libvirt code. Tests show that --raw still works properly too.
| Changed in vm-builder (Ubuntu): | |
| importance: | Undecided → High |
| status: | New → Triaged |
| description: | updated |
| Changed in vm-builder (Ubuntu Maverick): | |
| status: | New → Triaged |
| Changed in vm-builder (Ubuntu Lucid): | |
| status: | New → Triaged |
| importance: | Undecided → High |
| Changed in vm-builder (Ubuntu Maverick): | |
| importance: | Undecided → High |
| description: | updated |
| Changed in vm-builder (Ubuntu Maverick): | |
| status: | Triaged → In Progress |
| description: | updated |
| Jamie Strandboge (jdstrand) wrote | #1 |
| Jamie Strandboge (jdstrand) wrote | #2 |
| Jamie Strandboge (jdstrand) wrote | #3 |
| description: | updated |
| Jamie Strandboge (jdstrand) wrote | #4 |
| description: | updated |
| Jamie Strandboge (jdstrand) wrote | #5 |
| Changed in vm-builder (Ubuntu Lucid): | |
| assignee: | nobody → Jamie Strandboge (jdstrand) |
| Changed in vm-builder (Ubuntu Maverick): | |
| assignee: | nobody → Jamie Strandboge (jdstrand) |
| Changed in vm-builder (Ubuntu Natty): | |
| assignee: | nobody → Jamie Strandboge (jdstrand) |
| Changed in vm-builder (Ubuntu Maverick): | |
| milestone: | none → maverick-updates |
| Martin Pitt (pitti) wrote Please test proposed package | #6 |
| Changed in vm-builder (Ubuntu Maverick): | |
| status: | In Progress → Fix Committed |
| tags: | added: verification-needed |
| Jamie Strandboge (jdstrand) wrote | #7 |
| Launchpad Janitor (janitor) wrote | #8 |
| Changed in vm-builder (Ubuntu Maverick): | |
| status: | Fix Committed → Fix Released |
| Jamie Strandboge (jdstrand) wrote | #9 |
| Changed in vm-builder (Ubuntu Maverick): | |
| status: | Fix Released → Fix Committed |
| Changed in vm-builder (Ubuntu Natty): | |
| status: | Triaged → Fix Released |
| Jamie Strandboge (jdstrand) wrote | #10 |
| Changed in vm-builder (Ubuntu Lucid): | |
| status: | Triaged → Fix Committed |
| tags: |
added: verification-done removed: verification-needed |
| tags: | added: verification-needed |
| Launchpad Janitor (janitor) wrote | #11 |
| Changed in vm-builder (Ubuntu Maverick): | |
| status: | Fix Committed → Fix Released |
| Launchpad Janitor (janitor) wrote | #12 |
| Changed in vm-builder (Ubuntu Lucid): | |
| status: | Fix Committed → Fix Released |
| tags: | added: testcase |
Attached is a preliminary patch for vm-builder on maverick which adjusts VMbuilder/disk.py to have a new 'self.format_type' attribute. This attribute will normally be 'None', but will be set when a disk is convert()ed. VMbuilder/
Tested using:
$ sudo vmbuilder kvm ubuntu --suite lucid --flavour virtual --arch i386 -o --libvirt qemu:///system
$ sudo vmbuilder kvm ubuntu --suite lucid --flavour virtual --arch i386 -o --libvirt qemu:///system --raw /dev/loop1