Richard W.M. Jones
2021-Jan-23 06:27 UTC
[Libguestfs] [PATCH 2/6] v2v: rhv-upload-plugin: Use API_VERSION 2
On Sat, Jan 23, 2021 at 12:45:20AM +0200, Nir Soffer wrote:> @failing > -def pwrite(h, buf, offset): > +def pwrite(h, buf, offset, flags): > http = h['http'] > > count = len(buf) > @@ -234,7 +253,7 @@ def pwrite(h, buf, offset): > > > @failing > -def zero(h, count, offset, may_trim): > +def zero(h, count, offset, flags):Assume there's no way to pass the possible FUA flag through to oVirt here? Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com libguestfs lets you edit virtual machines. Supports shell scripting, bindings from many languages. http://libguestfs.org
Nir Soffer
2021-Jan-23 12:02 UTC
[Libguestfs] [PATCH 2/6] v2v: rhv-upload-plugin: Use API_VERSION 2
On Sat, Jan 23, 2021, 08:28 Richard W.M. Jones <rjones at redhat.com> wrote:> On Sat, Jan 23, 2021 at 12:45:20AM +0200, Nir Soffer wrote: > > @failing > > -def pwrite(h, buf, offset): > > +def pwrite(h, buf, offset, flags): > > http = h['http'] > > > > count = len(buf) > > @@ -234,7 +253,7 @@ def pwrite(h, buf, offset): > > > > > > @failing > > -def zero(h, count, offset, may_trim): > > +def zero(h, count, offset, flags): > > Assume there's no way to pass the possible FUA flag through to oVirt here? >We have flush=true, which send nbd flush command after the write/zero. Maybe we could handle flush=true as fua when using nbd backend, I'm not sure how important is this. I think we miss converting fua to flush=true to have full api 2 support.> Rich. > > -- > Richard Jones, Virtualization Group, Red Hat > http://people.redhat.com/~rjones > Read my programming and virtualization blog: http://rwmj.wordpress.com > libguestfs lets you edit virtual machines. Supports shell scripting, > bindings from many languages. http://libguestfs.org > >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://listman.redhat.com/archives/libguestfs/attachments/20210123/b86601d5/attachment.htm>