<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;"><p style="white-space: pre-wrap; word-wrap: break-word;">I don't see much use for these Load/Unload events.  The majority<br>of<br>the events would happen when nobody can be connected to AMI so<br>nobody would see the events.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Richard,</p><p style="white-space: pre-wrap; word-wrap: break-word;">This ami event is not only for the manager module, but also all<br>other modules.</p><p style="white-space: pre-wrap; word-wrap: break-word;">It designed for send load/unload notify of the all the modules. 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.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">That's my point.  You cannot be connected to AMI before the modules are loaded when asterisk starts so you could not see the events.  On asterisk shutdown, you would be disconnected from AMI and would not see the unload events.  You would only be able to see these events if you manually loaded/unloaded modules.</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 20:52:00 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>