<p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4;">Code-Review +1</span></p><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/17502">View Change</a></p><p>6 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/17502/1/common/chapters/cs7-config.adoc">File common/chapters/cs7-config.adoc:</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-gsm-manuals/+/17502/1/common/chapters/cs7-config.adoc@63">Patch Set #1, Line 63:</a> <code style="font-family:monospace,monospace">  # IP address of the remote STP:</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">We usually do use some kind of markers for this. It shows up in the pdf as (1) and then below the explanation.<br>It'd be great also adding a line with "local-ip" (or however it's called) and stating it's optional, kernel deciding by default.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/17502/1/common/chapters/cs7-config.adoc@103">Patch Set #1, Line 103:</a> <code style="font-family:monospace,monospace">Each CNI program on an SCCP/M3UA link typically has a local point-code,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">That's a bit strange from reader point of view. You first explain the different formats supported and then you explained what you already showed through the format. I'd make a lot more sense to first have the "Local Point-Code" section and perhaps also de "Remote Point-Code" one, and later the "Point-Code Format" one, which is basically a detailed explanation on how to set them.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/17502/1/common/chapters/cs7-config.adoc@138">Patch Set #1, Line 138:</a> <code style="font-family:monospace,monospace"> sccp-address my-entry</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">my-entry-name? or my-entry-label</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/17502/1/common/chapters/cs7-config.adoc@185">Patch Set #1, Line 185:</a> <code style="font-family:monospace,monospace">An `asp` is part of an `as`. For details, see the OsmoSTP manual, chapters</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">Not necessarily, an ASP can be used by several AS afair. So let's say the relation is not "AS 1 --- * ASP".</p><p style="white-space: pre-wrap; word-wrap: break-word;">That's why in VTY config the asp node is not inside the AS one, but separated, and only a reference to the ASP is used inside the AS node.</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/17502/1/common/chapters/cs7-config.adoc@189">Patch Set #1, Line 189:</a> <code style="font-family:monospace,monospace">an `as` references the `asp` belonging to it by their names.  In typical usage,</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">In typical simple/minimal setup</p><p style="white-space: pre-wrap; word-wrap: break-word;">We support now running with several ASPs per AS, for instance using traffic-node loadshare</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/17502/1/common/chapters/cs7-config.adoc@204">Patch Set #1, Line 204:</a> <code style="font-family:monospace,monospace">It is possible to configure only an `as` or only an `asp` explicitly -- a</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm not sure to understand this part here. I think you should simplymake clear before hand that AS*---*ASP and they must be linked, and you can then drop completely these paragraphs below.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/17502">change 17502</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-gsm-manuals/+/17502"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-manuals </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Ia2508d4c7b0fef9cdc57e7e122799a480e340bf7 </div>
<div style="display:none"> Gerrit-Change-Number: 17502 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: neels <nhofmeyr@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: Wed, 18 Mar 2020 11:56:44 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Gerrit-MessageType: comment </div>