[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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


  • To: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
  • From: "pithon, denis" <d.pithon@xxxxxxx>
  • Date: Fri, 30 Apr 2010 15:25:05 +0200
  • Accept-language: fr-FR
  • Acceptlanguage: fr-FR
  • Delivery-date: Fri, 30 Apr 2010 06:26:14 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>
  • Thread-index: AcroaIvCjfz9+WsLSxajBiP+NGA/lg==
  • Thread-topic: domUloader.py failed when vm disks are declared using /dev/disk/by-path/... (quick and dirty bugfix)

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

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.