[asterisk-dev] [Code Review] Add event-triggered origination to the pluggable origination module

Matt Jordan reviewboard at asterisk.org
Thu Jan 17 09:35:24 CST 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/2261/#review7676
-----------------------------------------------------------



asterisk/trunk/lib/python/asterisk/PluggableModules.py
<https://reviewboard.asterisk.org/r/2261/#comment14629>

    This shouldn't have to be in your default config, as the default trigger value is 'scenario_start'.



asterisk/trunk/lib/python/asterisk/PluggableModules.py
<https://reviewboard.asterisk.org/r/2261/#comment14630>

    I'd structure this differently.
    
    If someone specified a trigger value of 'event', check to see if 'event' is in their config. If not, raise an exception - you need to kill the test as they have created an invalid configuration.
    
    Past the exception, just index into self.config['event'] like you are already.
    
    As it is, if you specify a trigger of event, the test will start up but never originate anything - meaning they'll have to dig into the test code to figure out why their test just timed out.


- Matt


On Jan. 4, 2013, 9:58 a.m., opticron wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2261/
> -----------------------------------------------------------
> 
> (Updated Jan. 4, 2013, 9:58 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> This patch adds the capability to have an origination triggered from an arbitrary event received via AMI from any available Asterisk instance.  The AMI listener defaults to the Asterisk instance where the origination will take place.
> 
> 
> Diffs
> -----
> 
>   asterisk/trunk/lib/python/asterisk/PluggableModules.py 3598 
>   asterisk/trunk/sample-yaml/originator-config.yaml.sample 3598 
> 
> Diff: https://reviewboard.asterisk.org/r/2261/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> opticron
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130117/611b1a59/attachment-0001.htm>


More information about the asterisk-dev mailing list