<p><a href="https://gerrit.asterisk.org/c/asterisk/+/14405">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14405/2/res/res_pjsip_exten_state.c">File res/res_pjsip_exten_state.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/14405/2/res/res_pjsip_exten_state.c@419">Patch Set #2, Line 419:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">                   ast_log(LOG_NOTICE, "Endpoint '%s' state subscription failed: "<br>                             "Extension '%s' does not exist in context '%s' or has no associated hint\n",<br>                                ast_sorcery_object_get_id(endpoint), resource, context);<br></pre></blockquote></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Instead of adding a whole new option just for this one log statement I think I'd be more inclined fo […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think it's better to actually start to invent a new log paradigm altogether that's more flag oriented.  But for now, since that doesn't exist, individual per-item logging would be incredibly helpful to match a wide variety of use cases without having to force a workflow.</p><p style="white-space: pre-wrap; word-wrap: break-word;">My use case:<br>I pretty much run everything at core verbose 4 without question.  We have some pretty complex systems and need the ability to go to logging at any point and see what happened, so we can resolve the issue without having to ask people to 'reproduce the problem live'.  Something like subscription failures from a historical point of view, I could care less about.  Failing subscribes is more of a "what is it doing right now?" sort of thing, for us.  Turning off this one log, while keeping the rest of the logging, probably saves me about 2,000-5,000 lines of logging per second, per system.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Dropping down to verbose 0, I would consider it a no-go to just avoid logging one particular, but very common type of issue.  A type of issue that I think most I would agree to be not a major issue.  And I would think it would be safe to assume that other users would want to have verb 1,2,3,4,etc and also the ability to trim down certain types of extra logging.</p><p style="white-space: pre-wrap; word-wrap: break-word;"><br>General use case:<br>People should be able to control what they want across the entire ecosystem at a fairly atomic level, if it is reasonable to do so.  And that's a huge part of Asterisk history as a whole... one persons feature is another one's annoyance, so it begets that there's options to turn off and on.</p><p style="white-space: pre-wrap; word-wrap: break-word;">And with the complexity of the logging system as it is right now, adding 'one more if' is not much of a performance hit by any means.  If someone wants full logging, but avoid what is noise to them, they should be able to do that without having to turn off logging completely.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14405">change 14405</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.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.asterisk.org/c/asterisk/+/14405"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I5dff21c3c8ac3a3e3aefbd89053581fb90bc7018 </div>
<div style="display:none"> Gerrit-Change-Number: 14405 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Mark Murawski <markm@intellasoft.net> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-CC: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 12 May 2020 01:00:29 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>