[test-results] [Bamboo] Asterisk Testing > Asterisk Trunk > #1146 has FAILED (1 tests failed, no failures were new). Change made by dlee.

Bamboo bamboo at asterisk.org
Mon Apr 1 13:32:32 CDT 2013


-----------------------------------------------------------------------
Asterisk Testing > Asterisk Trunk > #1146 failed.
-----------------------------------------------------------------------
Code has been updated by dlee.
1/2 jobs failed, with 1 failing test, no failures were new.

http://bamboo.asterisk.org/browse/TESTING-ASTERISKTRUNK-1146/


--------------
Failing Jobs
--------------
  - Asterisk CentOS 6 32-Bit (CentOS 6): 1 of 467 tests failed.



--------------
Code Changes
--------------
dlee (384413):

>stasis: Fixed message ordering issues when forwarding
>
>This patch fixes an issue of message ordering that occurs when
>multiple topics are forwarded to an aggregator topic (such as
>ast_channel_topic_all()).
>
>It is (very reasonably) expected that the rules governing message
>dispatch order still apply, so long as the messages start from the
>same thread, and are received by the same subscription. Because the
>existing code had an additional layer of dispatching via the Stasis
>thread pool for forwards, those promises couldn't be kept.
>
>Forwarding subscriptions no longer have their own mailbox, and now
>dispatch directly from the forwarding topic's stasis_publish()
>call. This means that the topic's lock is held for the duration of not
>only a message's dispatch, but the dispatch of all the forwards. This
>shouldn't be a problem right now, but if an aggregator topic had many
>subscribers, it could become a problem. But I figure we can write more
>clever code when the time comes, if necessary.
>
>Review: https://reviewboard.asterisk.org/r/2419/
>



--------------
Tests
--------------
Existing Test Failures (1)
   - AsteriskTestSuite: S/apps/confbridge/confbridge marked
Fixed Tests (2)
   - AsteriskTestSuite: S/channels/ s i p/sip custom presence/multiple state change
   - AsteriskTestSuite: S/channels/ s i p/sip custom presence/nominal state change

--
This message is automatically generated by Atlassian Bamboo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/test-results/attachments/20130401/df64d1ac/attachment-0001.htm>


More information about the Test-results mailing list