[libvirt] [ruby-libvirt] Don't make volume upload zero bytes

Guido Günther agx at sigxcpu.org
Sun Jun 12 11:34:02 UTC 2016


On Fri, Jun 10, 2016 at 03:58:09PM -0400, Chris Lalancette wrote:
> On Fri, Jun 10, 2016 at 11:10 AM, Guido Günther <agx at sigxcpu.org> wrote:
> 
> > The virStreamSourceFunc internal_sendall currently returns the first
> > value passed in from the block, which according the ruby-libvirt docs
> > should be zero:
> >
> >   The send block should return an array of 2 elements; the first element
> >   should be the return code from the block (-1 for error, 0 otherwise)
> >
> > But according to the libvirt docs of virStreamSourceFunc this indicates
> > EOF:
> >
> >   Returns:  the number of bytes filled, 0 upon end of file, or -1 upon
> > error
> >
> > So return the size of the buffer to unbreak volume uploads.
> 
> 
> This looks OK to me, so I applied it, thanks.  I really should add an

Thanks!

> example of virStream to the examples on the website; do you happen to have
> a relatively simple example that I could put up there?

Send via different mail. I've added this as examples/ but feel free to
not apply it and use it on the website, whatever fits best.

BTW the mirror at

    https://github.com/libvirt/ruby-libvirt/commits/master

does not seem to be updating.

Cheers,
 -- Guido




More information about the libvir-list mailing list