On Tue, Nov 06, 2007 at 09:10:54AM +0900, Masayuki Sunou wrote:
Hi
Virsh attach-disk fails when the disk is attached to domain.
And error message displays that device_configure() failed.
# virsh attach-disk PV_RH51RC_12 /dev/hda8 xvdb --driver phy
libvir: Xen Daemon error : POST operation failed:
(xend.err'device_configure() takes exactly 2 arguments (3 given)')
--> BZ346281 :
https://bugzilla.redhat.com/show_bug.cgi?id=346281
This problem occurs because search of device ID of disk doesn't
operate correctly.
This patch fixes it.
And this fixes a little comment too.
Okay, as usual, I really prefer if you could explain how the patch
actually work instead of just describing the symptoms you are trying
to fix. That said I think I understand how this work , asking the
xen store is probably the only reliable way to get the device ID
and we already do this for networking devices, so I guess the patch
is okay. But in the future please explain how the patches you send
are supposed to work, this makes review way faster and the patches
are less likely to sit unverified in the list archives,
patch applied and commited,
thanks a lot !
Daniel
--
Red Hat Virtualization group
http://redhat.com/virtualization/
Daniel Veillard | virtualization library
http://libvirt.org/
veillard(a)redhat.com | libxml GNOME XML XSLT toolkit
http://xmlsoft.org/
http://veillard.com/ | Rpmfind RPM search engine
http://rpmfind.net/