<p> Attention is currently required from: N A, Joshua Colp. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/15962">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">File apps/app_reload.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.asterisk.org/c/asterisk/+/15962/comment/f3f7558e_6374787b">Patch Set #3, Line 88:</a> <code style="font-family:monospace,monospace">   if (!ast_strlen_zero(target)) {</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Wait, so I should document the common hardcoded things and then keep the aliases or remove them? Wit […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">A few ways maybe to handle this. I'm not sure if any are really that great, or a "clean" way of doing things:</p><p style="white-space: pre-wrap; word-wrap: break-word;">1) Add a public API call that executes a cli alias, and then call it from reload</p><p style="white-space: pre-wrap; word-wrap: break-word;">2) Move this functionality to res_clialiases.c unless that's what Josh meant by not moving to core.</p><p style="white-space: pre-wrap; word-wrap: break-word;">2) Similar to the above but instead assume a list of hard-coded alias names. Attempt to execute them via the ast_cli_command call. If it fails output a warning stating an alias of name "abx xyz" needs to be defined in clialiases.conf to do what you want. In this case this would be less a "reload" app at this point and more an "alias execute" app.</p><p style="white-space: pre-wrap; word-wrap: break-word;">3) Leave mostly as is, remove all "aliases", and force the dialplan creator to pass in all modules they want to reload. So now in the pjsip case they'd have to pass in Reload(chan_pjsip,res_pjsip,res_pjsip_ etc...).</p><p style="white-space: pre-wrap; word-wrap: break-word;">4) Similar to 3, but add wildcards to alleviate some typing. So now can do Reload(chan_*) -> reloads all channel drivers, Reload(chan_pjsip, res_pjsip_*) -> reload all pjsip reloadable items, etc...</p><p style="white-space: pre-wrap; word-wrap: break-word;">5) Something else?</p><p style="white-space: pre-wrap; word-wrap: break-word;">All these have good and bad to them, so not sure which I'd choose. If I had to put an order maybe 2,3,1,2,4? Thoughts?</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/15962">change 15962</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.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.asterisk.org/c/asterisk/+/15962"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: Ic8ab025d8b38dd525b872b41c465c999c5810774 </div>
<div style="display:none"> Gerrit-Change-Number: 15962 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Attention: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 08 Jun 2021 20:39:37 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Comment-In-Reply-To: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>