<p style="white-space: pre-wrap; word-wrap: break-word;">I'm genrally worried when we introduce patches like this. They are very likely to confuse the regular operator/user and have possibly dangerous effects in production setups.  At the very least, they should be added as HIDDEN vty commands, so they don't end up being documented and don't show up in vty autocompletion.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Alternatively, one could have e.g. an environment variable that's only set during testing, which would be the precondition to registering related VTY commands.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Specifically regarding this patch:  Why introduce a new "disabled" member?  Why not handle the disabled state as a state in the lchan FSM?</p><p><a href="https://gerrit.osmocom.org/12624">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/12624/2/src/osmo-bsc/bsc_vty.c">File src/osmo-bsc/bsc_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/12624/2/src/osmo-bsc/bsc_vty.c@4796">Patch Set #2, Line 4796:</a> <code style="font-family:monospace,monospace">DEFUN</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">if we merge this at all, this should be a hidden command that is now displayed to the normal user.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/12624">change 12624</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/12624"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-bsc </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I397e68e26d6a1727890353fa34f4897b54795866 </div>
<div style="display:none"> Gerrit-Change-Number: 12624 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-CC: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-CC: Vadim Yanitskiy <axilirator@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 18 Jan 2019 11:01:12 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>