Re: [PATCH 6/8] qemu: monitor: Use qemuMonitorJSONBlockJobError in qemuMonitorJSONDrivePivot

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

 



On Thu, Aug 23, 2018 at 17:38:39 -0400, John Ferlan wrote:
> 
> 
> On 08/15/2018 07:52 AM, Peter Krempa wrote:
> > The API deals with a block job so use the common error reporting
> > function for it.
> > 
> > Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
> > ---
> >  src/qemu/qemu_monitor_json.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> 
> Very strange - this got applied to qemuMonitorJSONBlockStream in my "git
> am" of the patches... Gets stranger on the next patch too, but at least
> this explains what happened there...

Yeah. The blockdev series probably shifted some code arround (I was
adding the APIs for media manipulation) and the context was not big
enough to apply this correctly. I had the same problem now when I've
rebased it locally.

I've made sure now that it's applied at the correct point.

> 
> Reviewed-by: John Ferlan <jferlan@xxxxxxxxxx>
> 
> John
> 
> --
> libvir-list mailing list
> libvir-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/libvir-list

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux