[asterisk-dev] [Code Review] 2703: Strip down the old event system

[opticron]

> On July 29, 2013, 11:24 p.m., Russell Bryant wrote:
> > How does distributed device state work after this change?
> 
> Russell Bryant wrote:
>     Ah, I see how res_xmpp works now, but not res_corosync.  Presumably this change breaks that module?

res_corosync has been broken for a while now because nothing publishes to the event system anymore. res_corosync refactoring was put on hold until all the event types it depended on were refactored to Stasis because mixing Stasis subscriptions and event system subscriptions wasn't going to be easy given the architecture of res_corosync. The patch in this code review presents an additional problem to getting res_corosync up and running again. Unless it internalizes a significant chunk of the old event system code, it will not be able to communicate with older versions of Asterisk. One suggestion has been to break compatibility and use JSON blobs to transmit state.


- opticron


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


On July 26, 2013, 9:48 a.m., opticron wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviewboard.asterisk.org/r/2703/
> -----------------------------------------------------------
> 
> (Updated July 26, 2013, 9:48 a.m.)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Bugs: ASTERISK-22139
>     https://issues.asterisk.org/jira/browse/ASTERISK-22139
> 
> 
> Repository: Asterisk
> 
> 
> Description
> -------
> 
> This removes unused code, event types, IE pltypes, and event IE types where possible and makes several functions private that were once public. This includes a renumbering of the remaining event and IE types which breaks binary compatibility with previous versions. The last remaining consumers of the old event system (or parts thereof) are main/security_events.c, res/res_security_log.c, tests/test_cel.c, tests/test_event.c, main/cel.c, and the CEL backends.
> 
> 
> Diffs
> -----
> 
>   trunk/apps/app_queue.c 395498 
>   trunk/apps/app_voicemail.c 395498 
>   trunk/channels/chan_dahdi.c 395498 
>   trunk/channels/chan_iax2.c 395498 
>   trunk/channels/chan_mgcp.c 395498 
>   trunk/channels/chan_sip.c 395498 
>   trunk/channels/chan_skinny.c 395498 
>   trunk/channels/chan_unistim.c 395498 
>   trunk/channels/sig_pri.h 395498 
>   trunk/channels/sig_pri.c 395498 
>   trunk/funcs/func_presencestate.c 395498 
>   trunk/include/asterisk/devicestate.h 395498 
>   trunk/include/asterisk/event.h 395498 
>   trunk/include/asterisk/event_defs.h 395498 
>   trunk/include/asterisk/security_events_defs.h 395498 
>   trunk/main/ccss.c 395498 
>   trunk/main/devicestate.c 395498 
>   trunk/main/event.c 395498 
>   trunk/main/manager.c 395498 
>   trunk/main/pbx.c 395498 
>   trunk/main/presencestate.c 395498 
>   trunk/main/security_events.c 395498 
>   trunk/res/parking/parking_applications.c 395498 
>   trunk/res/parking/parking_manager.c 395498 
>   trunk/res/parking/parking_ui.c 395498 
>   trunk/res/res_jabber.c 395498 
>   trunk/res/res_parking.c 395498 
>   trunk/res/res_security_log.c 395498 
>   trunk/res/res_xmpp.c 395498 
>   trunk/tests/test_cel.c 395498 
>   trunk/tests/test_event.c 395498 
> 
> Diff: https://reviewboard.asterisk.org/r/2703/diff/
> 
> 
> Testing
> -------
> 
> Made sure that CEL and event system unit tests still pass.
> 
> 
> Thanks,
> 
> opticron
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130730/399036dd/attachment.htm>