<p><a href="https://gerrit.osmocom.org/10022">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/10022/1/src/libosmo-mgcp/mgcp_vty.c">File src/libosmo-mgcp/mgcp_vty.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/10022/1/src/libosmo-mgcp/mgcp_vty.c@193">Patch Set #1, Line 193:</a> <code style="font-family:monospace,monospace"> vty_out(vty, "Endpoint %s0x%.2x:%s",</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">not sure if we should include a 0x here (I remember there has been confusion around decimal vs hex o […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">The idea to add rtpbridge/ prefix came from pmaier.</p><p style="white-space: pre-wrap; word-wrap: break-word;">I wrote this entire change sitting next to him and we were discussing it throughout the entire process.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/10022/1/src/libosmo-mgcp/mgcp_vty.c@256">Patch Set #1, Line 256:</a> <code style="font-family:monospace,monospace">      "Display information about an MGCP Media Gateway endpoint\n"</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Common prefix with other commands, here 'show mgcp', should share similar doc strings. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Conceptually, a trunk contains endpoints. A trunk corresponds to a physical wire bundle which contains several cables, and each end of a cable inside this conceptual trunk corresponds to an endpoint. See RFC 3435 for more details.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Issue OS#2660 is asking for a way to list endpoints., but a command which lists all trunks and their endpoints already exists ('show mgcp'). So I'm working under the assumption that this issue is asking for something which has already been implemented since the issue was filed. However, since the current implementation doesn't allow for filtering, I decided to implement a command which provides filtering of the existing output such that the state of specific endpoints can be viewed in isolation. Because any endpoint is part of a  trunk, the commands currently require a trunk number in addition to an endpoint name.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The tweaks to existing output which sneaked into this change are based on discussion between me and Philip. Philip helped me a lot in making sense of the media gateway in general and also the evolution of this code and its inconsistencies. We also discussed the UI of the new command together.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If you are merely concerned that some design discussion involving Philip should happen, rest assured that it has already occured. If instead your intention is to take part in this design discussion yourself, I'm happy to start the discussion over from the top with you and Philip. But I also think we should not drag this out forever because this topic doesn't seem to be worth a huge deal.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/10022">change 10022</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/10022"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I5330e697ec34bf215de91d44209048a8dc226d51 </div>
<div style="display:none"> Gerrit-Change-Number: 10022 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Sperling <ssperling@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 20 Jul 2018 11:20:26 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>