[Pkg-libvirt-maintainers] Bug#630478: Problem with size of virtual floppies

[John Summerfield]

Package: virtinst
Version: 0.500.3-2
Severity: important
Justification: Some functionality is unavailable


virt-install appears to convert existing  disk image files sizes to
gigabytes at around line 650 of VirtualDisk.py which has this code:
	newsize = float(newsize) / 1024.0 / 1024.0 / 1024.0
Even with floppy disk images. Whether the above assumption is actually what happens, the
consequence is that I get this error:
[Sat, 11 Jun 2011 22:02:46 virt-install 32253] DEBUG (virt-install:330) parse_disk: returning {'format': None, 'bus': None, 'readOnly': True, 'volInstall': None, 'path': '/var/lib/libvirt/images/fd.img', 'device': 'floppy', 'volName': None, 'conn': <libvirt.virConnect instance at 0x2402998>, 'size': None, 'driverCache': None, 'shareable': False, 'sparse': True}
[Sat, 11 Jun 2011 22:02:47 virt-install 32253] DEBUG (VirtualDisk:860) Path '/var/lib/libvirt/images' is target for pool 'default'. Creating volume 'fd.img'.
[Sat, 11 Jun 2011 22:02:47 virt-install 32253] ERROR (cli:196) Error with storage parameters: Size must be specified for non existent volume path '/var/lib/libvirt/images/fd.img'
[Sat, 11 Jun 2011 22:02:47 virt-install 32253] DEBUG (_util:221) Traceback (most recent call last):
  File "/usr/lib/pymodules/python2.6/virtinst/cli.py", line 480, in disk_prompt
    dev = VirtualDisk(**arg_dict)
  File "/usr/lib/pymodules/python2.6/virtinst/VirtualDisk.py", line 437, in __init__
    self.__validate_params()
  File "/usr/lib/pymodules/python2.6/virtinst/VirtualDisk.py", line 946, in __validate_params
    self.__check_if_path_managed()
  File "/usr/lib/pymodules/python2.6/virtinst/VirtualDisk.py", line 864, in __check_if_path_managed
    "existent volume path '%s'" % self.path))
ValueError: Size must be specified for non existent volume path '/var/lib/libvirt/images/fd.img'

This is the actual file:
-rw-r--r-- 1 root root 1474560 Jun 11 21:46 /var/lib/libvirt/images/fd.img

Note that this floppy image was okay in the RHEL6 beta where I created it ans used it for several Debian Squeeze installs using python-virtinst-0.500.3-7.el6.noarch or (slightly) earlier.

Note, if the disk image is missing, this message arises:
ERROR    A size must be specified for non-existent disks.

It does not identify the problem file, and the text is slightly different although the meaning,
it seems, is the same.



-- System Information:
Debian Release: 6.0.1
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.32-5-amd64 (SMP w/8 CPU cores)
Locale: LANG=en_AU.utf8, LC_CTYPE=en_AU.utf8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages virtinst depends on:
ii  python             2.6.6-3+squeeze6      interactive high-level object-orie
ii  python-libvirt     0.8.3-5+squeeze1      libvirt Python bindings
ii  python-libxml2     2.7.8.dfsg-2+squeeze1 Python bindings for the GNOME XML 
ii  python-support     1.0.10                automated rebuilding support for P
ii  python-urlgrabber  3.9.1-4               A high-level cross-protocol url-gr

Versions of packages virtinst recommends:
ii  qemu                       0.12.5+dfsg-3 fast processor emulator
ii  virt-viewer                0.2.1-1       Displaying the graphical console o

virtinst suggests no packages.

-- no debconf information