[asterisk-bugs] [Asterisk 0013186]: Queue timeout doesn't work when periodic-announce is set.

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13186 
====================================================================== 
Reported By:                miquel_cabrespina
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13186
Category:                   Applications/app_queue
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     ready for testing
Asterisk Version:           1.4.21.1 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-07-29 03:19 CDT
Last Modified:              2008-07-31 10:48 CDT
====================================================================== 
Summary:                    Queue timeout doesn't work when periodic-announce is
set.
Description: 
When periodic-announce is setted and the general timeout for the queue is
reached, the queue keeps trying to call its members.
When the timeout is reached the queue must finalize and go to the next
priority in context but when a periodic-announce is set the queue never
finalize. At queue timeout the periodic announce won't play anymore and the
option "retry" is also ignored. The call remains in the queue until a
member answers.

When the periodic timeout is reached we see this in the console until some
member answer:
  == Spawn extension (MYCONTEXT_102, s-CHANUNAVAIL, 3) exited non-zero on
'Local/s at MYCONTEXT_102-95c1,2'
====================================================================== 

---------------------------------------------------------------------- 
 (0090905) svnbot (reporter) - 2008-07-31 10:48
 http://bugs.digium.com/view.php?id=13186#c90905 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 134758

U   branches/1.4/apps/app_queue.c

------------------------------------------------------------------------
r134758 | mmichelson | 2008-07-31 10:48:08 -0500 (Thu, 31 Jul 2008) | 16
lines

Add more timeout checks into app_queue, specifically
targeting areas where an unknown and potentially
long time has just elapsed. Also added a check
to try_calling() to return early if the timeout
has elapsed instead of potentially setting a negative
timeout for the call (thus making it have *no* timeout
at all).


(closes issue http://bugs.digium.com/view.php?id=13186)
Reported by: miquel_cabrespina
Patches:
      13186.diff uploaded by putnopvut (license 60)
Tested by: miquel_cabrespina


------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=134758 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-07-31 10:48 svnbot         Checkin                                      
2008-07-31 10:48 svnbot         Note Added: 0090905                          
======================================================================