<div dir="ltr">Oops, sorry. Dropping the list was not intentional.  I didnt realise I had sent a "reply" in place of "reply-all" .<div>Adding back :)<br></div><div><br></div><div>I still would argue that having the VM's name narrows down the problem space. If a client knows what operations have been fired for a VM over the last time window, it is not too difficult to deduce which of them could have caused this message to occur.</div><div><br></div><div>As I said before, use of libvirt Debug logs adds way too much logspew. If we have more pointed error/warning messages, it can largely help debugging on its own.</div><div><br></div><div>Even if we turn debugging on, libvirt honestly has way too many messages that look like an en-masse CTRL-C/V operation. Sample:</div><div> All qemuMonitorEmit* except one have this message:</div><div>







<p class="gmail-p1"><span class="gmail-s1">VIR_DEBUG(</span><span class="gmail-s2">"mon=</span><span class="gmail-s3">%p</span><span class="gmail-s2">"</span><span class="gmail-s1">, mon);</span></p><p class="gmail-p1"><span class="gmail-s1">Even if this message shows up in logs, it is near impossible to trace which event was seen by the VM. And it is quite another task to see which VM this monitor belonged to.</span></p><p class="gmail-p1"><br></p><p class="gmail-p1">Not that more vebosity wont help here. </p><p class="gmail-p1">However, I am only trying to make logs easier to work with. Every message should ideally contribute to something meaningful, else it has little value to add while debugging.</p></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 22, 2017 at 1:59 PM, Peter Krempa <span dir="ltr"><<a href="mailto:pkrempa@redhat.com" target="_blank">pkrempa@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">[Please don't drop the list on the responses.]<br>
<span class=""><br>
On Wed, Mar 22, 2017 at 13:52:14 +0530, Prerna wrote:<br>
> Always enabling debug logs adds a *lot* of logspew.<br>
> It would be good to have self-sufficient error messages, dont you think ?<br>
> What is wrong with adding a VM name field here ?<br>
<br>
</span>It's mostly useless. You get the VM name, but you don't have the<br>
operation that failed. So it's useless for debugging.<br>
</blockquote></div><br></div>