[Xen-devel] domUloader.py failed when vm disks are declared using /dev/disk/by-path/... (quick and dirty bugfix)

["pithon, denis" <d.pithon@xxxxxxx>]

Hello all,

 

domUloader.py is developed by Novell and I have already posted this message on a SLES Novell dedicated forum. I guess it could be useful on this list anyway.

We use iSCSI to export vm disk from storage server (OpenFiler) to Xen Server. To ease migration we declare disks in config file with /dev/disk/by-path/… :

 

…

bootloader="/usr/lib/xen/boot/domUloader.py"

bootargs="--entry=xvda2:/boot/vmlinuz-xenpae,/boot/initrd-xenpae"

…

disk=[ 'phy:/dev/disk/by-path/ip-192.168.10.250:3260-iscsi-iqn.2006-01.com.openfiler:tsn.x905root-lun-0,xvda,w', ]

…

 

domUloader.py failed to start the vm. I have eventually figure out that  the ‘:’ (colon) character is used to split kpartx output in scanpartitions functions oof domUloader.py.

I fixed it by changing a single line in scanpartitions function of domUloader.py :

 

Replace :

 

(pname, params) = line.split(':')

 

and add spaces around the colon, ie:

 

(pname, params) = line.split(' : ')

 

There is probably much better way to fix it …

 

Denis Pithon

CG49

 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel