<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;">OK, so say the user changes the ipa-name. And now we print a warning that they did something bad. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">my question: do things break when changing during run-time?<br>If they break, then rejecting is justified and explain why in a comment or the vty out message.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Do things not break, and simply have no effect?<br>Then that is how many other vty options already work, and matches the feeling that we're editing cached config that doesn't necessarily have an effect on the running program. It matches the current scheme, and that can be allowed.</p><p style="white-space: pre-wrap; word-wrap: break-word;">For example in osmo-bsc you can edit the pchan settings of time slots, and you can write them back to config file, but they will only have an effect when you re-connect the OML. Re-connect OML can be done by 'do bts 0 drop oml' or something, or a program restart.</p><p style="white-space: pre-wrap; word-wrap: break-word;">(...still valid: just use vty->type)</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: Mon, 17 Dec 2018 19:39:54 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>