<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1: Code-Review-1</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Patch Set 1:</p><p style="white-space: pre-wrap; word-wrap: break-word;">I usually use osmo-trx with info level set and it looks good for me. I like seeing the clock indications to see that everything's smoothly. I'm not against the patch though, just sharing my opinion.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Same here, I am fine with INFO. Clock indications can be suspended by setting DTRXCLK to NOTICE. And I don't see what a regular user would benefit from this change... among with sending us even less informative bug reports.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The general "best practice" in Osmcoom so far is that all applications typically should have NOTICE  as a compile-time default for their log levels. This is not set in stone.  However, we should definitely not log lots of messages all the time, unless thy are e.g. triggered by user interaction (like a LU request) or something aobut a connection to another network element failing to come up (or that it was established).</p><p style="white-space: pre-wrap; word-wrap: break-word;">The fact that apparently there is no other way (or none that you guys know of) to check if everything is allright with OsmoTRX means that you are working around the underlying cause: That this status is not easily available otherwise, either by some 'show' command, or by some event based message that's printed when the status becomes OK or becomes NOT-OK.<br></p><p><a href="https://gerrit.osmocom.org/c/osmo-trx/+/17594">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-trx/+/17594">change 17594</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/osmo-trx/+/17594"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-trx </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ief82b85d9ff0e0e94eaabd255ebea961396fff32 </div>
<div style="display:none"> Gerrit-Change-Number: 17594 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: fixeria <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-CC: laforge <laforge@osmocom.org> </div>
<div style="display:none"> Gerrit-CC: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 27 Mar 2020 09:30:11 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>