<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Aug 21, 2018 at 2:49 PM Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Aug 21, 2018 at 02:40:49PM +0300, Nir Soffer wrote:<br>
> On Tue, Aug 21, 2018 at 2:02 PM Richard W.M. Jones <<a href="mailto:rjones@redhat.com" target="_blank">rjones@redhat.com</a>><br>
> wrote:<br>
> <br>
> > On Tue, Aug 21, 2018 at 01:15:02AM +0300, Nir Soffer wrote:<br>
> > > The oVirt server may fail a PUT request before reading all request body.<br>
> > > However before closing the connection, it writes a detailed error<br>
> > > response, that will make debugging issues like expired tickets much<br>
> > > easier to handle. If we don't get the response and log the error, the<br>
> > > error may be lost when the daemon log is rotated.<br>
> ><br>
> > This series looks fine, I'll push it in a minute.<br>
> ><br>
> > Couple of questions:<br>
> ><br>
> > (1) Are we using the nbdkit enhanced Python error reporting yet?<br>
> > What's stopping us if we are not?<br>
> ><br>
> <br>
> In request_failed() we raise RuntimeError, so it should log a traceback, but<br>
> I did not see a traceback, maybe because I did not test it in verbose mode.<br>
<br>
If you see anything at all, then you'd see the traceback.<br>
<br>
It seems more likely that whatever version of nbdkit being used does<br>
not include:<br>
<br>
  <a href="https://github.com/libguestfs/nbdkit/commit/72c0d64a47db642cafa89884f2ee554bd0b8e822" rel="noreferrer" target="_blank">https://github.com/libguestfs/nbdkit/commit/72c0d64a47db642cafa89884f2ee554bd0b8e822</a><br>
<br>
This has now been backported to the 1.2 & 1.4 branches as well.  Do<br>
you know what version (of nbdkit) they are testing with?<br></blockquote><div><br></div><div>Tested with:</div><div><div><div>$ rpm -qa | grep nbdkit</div><div>nbdkit-plugin-python3-1.4.2-1.fc28.x86_64</div><div>nbdkit-1.4.2-1.fc28.x86_64</div><div>nbdkit-plugin-python-common-1.4.2-1.fc28.x86_64</div></div></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks for opening the bug, let me know the BZ# when you have it.<br>
<br>
Rich.<br>
<br>
-- <br>
Richard Jones, Virtualization Group, Red Hat <a href="http://people.redhat.com/~rjones" rel="noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
libguestfs lets you edit virtual machines.  Supports shell scripting,<br>
bindings from many languages.  <a href="http://libguestfs.org" rel="noreferrer" target="_blank">http://libguestfs.org</a><br>
</blockquote></div></div>