<p>Richard Mudgett <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8166">View Change</a></p><p>Patch set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I don't see much use for these Load/Unload events.  The<br>majority<br>of<br>the events would happen when nobody can be connected to AMI<br>so<br>nobody would see the events.<br>Hi Richard,<br>This ami event is not only for the manager module, but also all<br>other modules.<br>It designed for send load/unload notify of the all the modules.<br>For<br>example, when the app_queue.so module was loaded/unloaded, the<br>Asterisk sends this ami event. Just like an Reload ami event.<br>That's my point.  You cannot be connected to AMI before the<br>modules<br>are loaded when asterisk starts so you could not see the events.<br>On asterisk shutdown, you would be disconnected from AMI and<br>would<br>not see the unload events.  You would only be able to see these<br>events if you manually loaded/unloaded modules.</p></blockquote></blockquote></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">A-ha, now I've got it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, you're right. It those of cases, not many user can see the<br>messages. And for the manual load/unload commands will fire this<br>events.</p><p style="white-space: pre-wrap; word-wrap: break-word;">But, that's why I designed this event.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If the some program which connected to the Asterisk via AMI<br>connection, there's no way to recognizing when the module has been<br>loaded/unloaded if the administrator typed "module load/unload<br><modulename>" command.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Do you have a need for this?</p><p style="white-space: pre-wrap; word-wrap: break-word;">It seems so unnecessary for a program to track manually loading/unloading modules.  Having to manually load/unload modules is an exception condition.  If a program were going to load/unload modules it would use the AMI ModuleLoad action which already reports the status of the action in the response.</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8166">change 8166</a>. To unsubscribe, 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/8166"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ib916c41eddd63651952998f2f49c57c42ef87a64 </div>
<div style="display:none"> Gerrit-Change-Number: 8166 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Richard Mudgett <rmudgett@digium.com> </div>
<div style="display:none"> Gerrit-Reviewer: sungtae kim <pchero21@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 06 Feb 2018 21:24:19 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>