<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Aug 21, 2018 at 2:02 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 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></blockquote><div><br></div><div>In request_failed() we raise RuntimeError, so it should log a traceback, but</div><div>I did not see a traceback, maybe because I did not test it in verbose mode.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
(2) Is there a BZ associated with this change?  If not, we'll need one<br>
if you want to get these changes into RHEL.<br></blockquote><div><br></div><div>Not yet, the only thing we have is <a href="https://bugzilla.redhat.com/1619019">https://bugzilla.redhat.com/1619019</a>,</div><div>showing why this change is important.</div><div><br></div><div>I will open a new bug for this.</div><div><br></div><div>Nir</div></div></div>