Re: [libvirt] libvirt RBD attach regression?

2012-11-21 Thread Eric Blake
On 11/21/2012 11:06 AM, Scott Sullivan wrote: > Hello, > > We were able to come up with this patch, which does fix the problem in > our use case: > > http://pastebin.com/izx40mRd > Pastebins are transient; so reposting here: > index d01e366..3b37ece 100644 > --- a/src/qemu/qemu_domain.c > +++

Re: [libvirt] libvirt RBD attach regression?

2012-11-21 Thread Scott Sullivan
Hello, We were able to come up with this patch, which does fix the problem in our use case: http://pastebin.com/izx40mRd Interested to hear any feedback on the patch. On 11/21/2012 12:13 PM, Eric Blake wrote: On 11/21/2012 09:45 AM, Scott Sullivan wrote: Hello, I have been testing lib

Re: [libvirt] libvirt RBD attach regression?

2012-11-21 Thread Eric Blake
On 11/21/2012 09:45 AM, Scott Sullivan wrote: > Hello, > > I have been testing libvirt v1.0.0 for deployment within my > organization, and in the process discovered what appears to be a bug > that breaks virsh attach-device, when attaching an RBD volume to an > instance. First, here is the error p

[libvirt] libvirt RBD attach regression?

2012-11-21 Thread Scott Sullivan
Hello, I have been testing libvirt v1.0.0 for deployment within my organization, and in the process discovered what appears to be a bug that breaks virsh attach-device, when attaching an RBD volume to an instance. First, here is the error presented, with v1.0.0 (this worked in v0.10.2): [ro