<p><a href="https://gerrit.osmocom.org/12243">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/12243/3/src/gprs/sgsn_vty.c">File src/gprs/sgsn_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/12243/3/src/gprs/sgsn_vty.c@1195">Patch Set #3, Line 1195:</a> <code style="font-family:monospace,monospace">        if (!parsing_config_file) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">my question: do things break when changing during run-time? […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I can't enumerate the consequences of changing this name on the fly.<br>What seems obvious is that changing this name will result in some IPA connections using name A and others using name B.<br>Likewise for routes which get installed in the HLR which refers to these names, both of which point towards the same SGSN.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Will this cause real harm? I cannot tell. Will connections using the old name be negatively impacted? Maybe.<br>What I am pretty certain about is that it risks putting the system into an inconsistent state.<br>What I also know is that this is an unprecedented use case because Osmocom has never run in multi-MSC/multi-SGSN setups<br>which means that we cannot really judge the consequences based on past experience. We'd have to do a lot of testing and<br>code audit to be sure of the consequences. And all that for what looks like an edge-case to me which should never happen<br>in real multi-MSC/multi-SGSN deployments as far as I can tell.</p><p style="white-space: pre-wrap; word-wrap: break-word;">So I'll turn the question around on you again:<br>Why would you want to change this name without also restarting the process?<br>What is the use case for this?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12243">change 12243</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/12243"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-sgsn </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ib2f65fed9f56b9718e8a9647e3f01dce69870c1f </div>
<div style="display:none"> Gerrit-Change-Number: 12243 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Stefan Sperling <stsp@stsp.name> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Sperling <stsp@stsp.name> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 18 Dec 2018 10:51:33 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>