<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 21, 2016 at 2:54 AM, Michal Skrivanek <span dir="ltr"><<a href="mailto:michal.skrivanek@redhat.com" target="_blank">michal.skrivanek@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><br>
> On 20 Sep 2016, at 19:52, Nir Soffer <<a href="mailto:nsoffer@redhat.com">nsoffer@redhat.com</a>> wrote:<br>
><br>
> Hi all,<br>
><br>
> We are reviewing patches for improving vdsm log format.<br>
><br>
> The goal is to have more standard logging format, similar to engine<br>
> logs and other logs in the ovirt project, to make it easier to debug<br>
> and support the system.<br>
><br>
> We know that this change will break tools people wrote to parse vdsm<br>
> logs (I wrote couple), but we find vdsm log format too painful to work<br>
> with, and we hope this is be useful in the long term.<br>
><br>
> Please review.<br>
> <a href="https://gerrit.ovirt.org/#/q/topic:log-format" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/q/<wbr>topic:log-format</a><br>
<br>
</span>Hi Nir,<br>
typically for concise changes reviewing a single patch is way easier than whole topic. I know vdsm developers prefer it that way, but please do realize it is harder for “external” people to comprehend and you get less feedback.<br>
I see example of the new format mentioned in one commit message as a link to pastebin. I suppose including a single line example in the commit message itself is a bit more persistent<br>
<br>
Just two notes - big -1 on moving the thread id further down the line. Typically this is the only thing with some continuity, making it the most important thing to see. Putting it at a fixed aligned position somewhere at the beginning of the line would help a lot.<br>
Do we switch to UTC timestamp or still want to use local time? It does make it a bit hard to correlate with libvirt and qemu.<br></blockquote><div><br></div><div>Lurker chiming in here ...</div><div><br></div><div>Using UTC is really important when correlating across multiple systems. All of the boxes we run in production we have placed in UTC to help us correlate events across our environment. We had no end of frustrations when we could not find ANY logs to debug a problem due to a timezone shift.</div><div><br></div><div>And keep in mind, the one reviewing events in time may not be in the same timezone as well, so the original timezone information will often get converted to the local timezone of that reviewer.</div><div><br></div><div>Thanks for considering,</div><div><br></div><div>-peter</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks for the improvements!<br>
<span class="HOEnZb"><font color="#888888">michal<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> Thanks,<br>
> Nir<br>
> ______________________________<wbr>_________________<br>
> Devel mailing list<br>
> <a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
> <a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/devel</a><br>
<br>
______________________________<wbr>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/devel</a></div></div></blockquote></div><br></div></div>