<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 5: Code-Review+1</p><p style="white-space: pre-wrap; word-wrap: break-word;">You changed the log line to avoid saying "fail" in the log line, but my wonder was more in the direction of why are we logging a NOTICE level log line for a scenario which is expected when a wildcarded DLCX is received.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I have put the log level to notice because the line appears only when the all trunk resources are in use, which is already a problem but until that point we can not say if we are able to execute the request handler properly. Wildcarded DLCX is at the moment the only situation that would turn out well.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Normally a wildcarded request will always get an endpoint, it will be always the next endpoint that is free. When all trunk resources are in use, we will get a trunk only.</p><p><a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24941">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.osmocom.org/c/osmo-mgw/+/24941/4/src/libosmo-mgcp/mgcp_protocol.c">File src/libosmo-mgcp/mgcp_protocol.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.osmocom.org/c/osmo-mgw/+/24941/4/src/libosmo-mgcp/mgcp_protocol.c@374">Patch Set #4, Line 374:</a> <code style="font-family:monospace,monospace">           LOGP(DLMGCP, LOGL_NOTICE,</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">I still don't get why printing a failure here, if it's actually accepted to receive no endpoint, for […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Maybe "cannot find endpoint" sounds better here.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-mgw/+/24941">change 24941</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-mgw/+/24941"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-mgw </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I9f519d8a0ee8a513fa1e74acf3ee7dbc0991cdde </div>
<div style="display:none"> Gerrit-Change-Number: 24941 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 19 Jul 2021 13:05:08 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>