On Tue, Aug 21, 2018 at 2:02 PM Richard W.M. Jones <[email protected]> wrote:
> On Tue, Aug 21, 2018 at 01:15:02AM +0300, Nir Soffer wrote: > > The oVirt server may fail a PUT request before reading all request body. > > However before closing the connection, it writes a detailed error > > response, that will make debugging issues like expired tickets much > > easier to handle. If we don't get the response and log the error, the > > error may be lost when the daemon log is rotated. > > This series looks fine, I'll push it in a minute. > > Couple of questions: > > (1) Are we using the nbdkit enhanced Python error reporting yet? > What's stopping us if we are not? > In request_failed() we raise RuntimeError, so it should log a traceback, but I did not see a traceback, maybe because I did not test it in verbose mode. (2) Is there a BZ associated with this change? If not, we'll need one > if you want to get these changes into RHEL. > Not yet, the only thing we have is https://bugzilla.redhat.com/1619019, showing why this change is important. I will open a new bug for this. Nir
_______________________________________________ Libguestfs mailing list [email protected] https://www.redhat.com/mailman/listinfo/libguestfs
