Python virStorageVolUpload binding fails on i386
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
uvtool |
Fix Released
|
High
|
Unassigned | ||
libvirt (Ubuntu) |
Fix Released
|
High
|
Unassigned | ||
Precise |
Fix Released
|
High
|
Unassigned |
Bug Description
[Impact]
uvtool fails to work on Precise i386, which means that juju's upcoming KVM container feature won't work when a guest is on the current LTS, which is our current recommendation.
[Test Case]
0. Start with Precise on i386
1. sudo add-apt-repository cloud-archive:tools
2. sudo apt-get update
3. sudo apt-get install uvtool-libvirt
4. sg libvirtd
5. uvt-simplestrea
Expected results: zero exit status (some spurious errors printed)
Actual results: non-zero exit status, "libvirt.
[Upstream Fix]
http://
[Development Fix]
Already on an upstream version that has the fix.
[Stable Fix]
Trivial backport of the upstream fix.
[Regression Potential]
The only change is in the Python bindings, so API calls involving 64-bit integer parameters are affected. The fix is obviously correct; upstream and Saucy both have it and so this fix has been extensively tested.
[Details]
Root cause: in the libvirt source, python/generator.py maps "unsigned long long" to PyArg_ParseTuple parameter "l", instead of "L" or "K".
Upstream fix: http://
Development fix: packaged from upstream
Stable fix: a trivial backport of this fix.
python-libvirt 0.9.8-2ubuntu17.13
uvtool cannot sync images using libvirt on Precise. This is limited to i386. The error looks something like:
[...]
File "/usr/lib/
vol.
File "/usr/lib/
if ret == -1: raise libvirtError ('virStorageVol
libvirt.
Note that this ignores the signed/unsigned distinction. I think the correct fix should be "K", but upstream are using "L", which will suffice for the actual problem. To avoid diverging I think "L" is better for an SRU if we have to do it today. I have raised this separately with upstream.
Changed in libvirt (Ubuntu): | |
importance: | Undecided → High |
Changed in uvtool: | |
importance: | Undecided → High |
description: | updated |
description: | updated |
Thanks, I'll push the SRU as soon as you attach the debdiff.