<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 2:</p><p style="white-space: pre-wrap; word-wrap: break-word;"><br>They can be left as they are I guess. You can try re-establishing them in the patch if you see it's easy to do so with current code base.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">No, I am not going to work on that because I don't think your suggested approach makes any sense.</p><p style="white-space: pre-wrap; word-wrap: break-word;">A program restart will look like one GSUP peer and route going away and a new GSUP peer and route appearing. And everything that needs to happen for this is already implemented by our current code. Compared to simply requiring that the problem be restarted. adding additional code paths to handle dynamic peer name and route changes at run-time carries a huge potential for bugs and side-effects.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Perhaps this is a weak analogy but: A routing protocol daemon (such as OSPF) won't change its router ID at run-time either.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I think it's fine requiring the user to re-start the process for the change to take effect, I was just pointing out that independent of that topic, there should be a "no foobar" for the reason I introduced.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I don't see a beneficial use case in clearing the IPA name before saving the current config. Why would anyone want to omit that name? If we require the name to be stored in the config file, why should it not always be saved when the file gets written out?</p><p><a href="https://gerrit.osmocom.org/12243">View Change</a></p><ul style="list-style: none; padding: 0;"></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: 2 </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: Thu, 13 Dec 2018 08:59:18 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>