<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 18 October 2017 at 14:11, Antony Antony <span dir="ltr"><<a href="mailto:antony@phenome.org" target="_blank">antony@phenome.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Oct 18, 2017 at 11:35:55AM -0400, Andrew Cagney wrote:<br>
> Hi,<br>
><br>
> I'm wondering if debug output should be prefixed with the connection/state<br>
> information as in:<br>
><br>
> | "westnet-eastnet" #1: .....<br>
<br>
</span>If every line add such a prefix it would be hard to read, I would struggle<br>
to read the debug logs prepended with 10-30 character prefix, think of OE,<br>
the prefix will be long. I have tried it in th past and did not like it, except<br>
for grepping:)<br>
<br>
It could be setup option, like logtime.<br>
<br></blockquote><div><br></div><div>Yea.  I'm going to add:</div><div><br></div><div>  --debug add-prefix</div><div><br></div><div>for now (not part of all) so we can play with it<br></div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Would per peer logging be an option. I don't know what is the current state<br>
of it.<br>
<span class=""><br>
> for our our tests it doesn't matter, but in the field when there are lots<br>
> of connections, being able to grep for this string makes things easier<br>
<br>
</span>Greping sounds attractive. However, in simple cases it would be hard to<br>
read. I read them very often. How would the grep work if connections are<br>
instances. You have to know more than connection name.<br>
<br>
Greping for state number helped in the past with O(10) non instance<br>
connections.<br>
<br>
Note: some of the debug lines already prepend log prefix. In those cases you<br>
will get double prefix.<br>
</blockquote></div><br></div></div>