[test-results] [Bamboo] Asterisk > Asterisk Unit Tests > #821 has FAILED (5 tests failed, 1 failures were new). Change made by Matt Jordan.

[Bamboo]

-----------------------------------------------------------------------
Asterisk > Asterisk Unit Tests > #821 failed.
-----------------------------------------------------------------------
This build occurred because it is a dependant of AST-ATRUNKFULLBUILD-849.
2/2 jobs failed, with 5 failing tests, 1 failure was new.

https://bamboo.asterisk.org/bamboo/browse/AST-ATRUNKUNIT-821/

---------------------
Currently Responsible
---------------------

Corey Farrell (Automatically assigned)
John Bigelow (Automatically assigned)
Richard Mudgett (Automatically assigned)



--------------
Failing Jobs
--------------
  - CentOS 6 32-Bit Unit Testing (Unit Testing): 2 of 457 tests failed.
  - CentOS 6 64-Bit Unit Testing (Unit Testing): 3 of 457 tests failed.



--------------
Code Changes
--------------
Matt Jordan (428080):

>app_confbridge: Don't play leader leaving prompt if no one will hear it
>
>Consider the following:
>- A marked user in a conference
>- One or more end_marked only users in the conference
>
>When the marked users leaves, we will be in the conf_state_multi_marked state.
>This currently will traverse the users, kicking out any who have the end_marked
>flags. When they are kicked, a full ast_bridge_remove is immediately called on
>the channels. At this time, we also unilaterally set the need_prompt flag.
>
>When the need_prompt flag is set, we then playback a sound to the bridge
>informing everyone that the leader has left; however, no one is left in the
>bridge. This causes some odd behaviour for the end_marked users - they are
>stuck waiting for the bridge to be unlocked. This results in them waiting for
>5 or 6 seconds of dead air before hearing that they've been kicked.
>
>Unfortunately, we do have to keep the bridge locked while we're playing back
>the 'leader-has-left' prompt. If there are any wait_marked users in the
>conference, this behaviour can't be easily changed - but we do make the case
>of the end_marked users better with this patch.
>
>Review: https://reviewboard.asterisk.org/r/4184/
>
>ASTERISK-24522 #close
>Reported by: Matt Jordan
>........
>
>Merged revisions 428077 from http://svn.asterisk.org/svn/asterisk/branches/11
>........
>
>Merged revisions 428078 from http://svn.asterisk.org/svn/asterisk/branches/12
>........
>
>Merged revisions 428079 from http://svn.asterisk.org/svn/asterisk/branches/13
>



--------------
Tests
--------------
New Test Failures (1)
   - AsteriskUnitTests: /main/cel/test cel attended transfer bridges link
Existing Test Failures (4)
   - AsteriskUnitTests: /res/parking/dynamic parking variables
   - AsteriskUnitTests: /channels/features/test features channel dtmf
   - AsteriskUnitTests: /res/parking/dynamic parking variables
   - AsteriskUnitTests: /channels/features/test features channel dtmf

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