Summary:	ASTERISK-10058: Unable to join queues until app_queue.so is reloaded
Reporter:	Benjamin Howell (bhowell)	Labels:
Date Opened:	2007-08-09 15:09:10	Date Closed:	2007-08-10 10:14:52
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	Applications/app_queue
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) modules.conf
Description:	I'm using the modules.conf sample file included with asterisk 1.4.10. Upon loading asterisk, my queues show '(Invalid)' next to all static members of the queue and the queues appear to be treated like they're empty. Once I issue a 'module reload app_queue.so' (or 'reload'), queues work properly. Output before reload: ==================== asterisk2*CLI> show queue tier2-iglou tier2-iglou  has 0 calls (max unlimited) in 'ringall' strategy (0s holdtime), W:0, C:0, A:0, SL:0.0% within 0s   Members: I>      Local/1153@acd-iglou with penalty 1 (Invalid) has taken no calls yet      Local/1134@acd-iglou (Invalid) has taken no calls yet      Local/1128@acd-iglou (Invalid) has taken no calls yet   No Callers> Output after reload: =================== asterisk2*CLI> show queue tier2-iglou tier2-iglou  has 0 calls (max unlimited) in 'ringall' strategy (0s holdtime), W:0, C:0, A:0, SL:0.0% within 0s   Members: I>      Local/1153@acd-iglou with penalty 1 (Not in use) has taken no calls yet      Local/1134@acd-iglou (Not in use) has taken no calls yet      Local/1128@acd-iglou (Not in use) has taken no calls yet   No Callers> Presumably this occurs because autoload=yes in modules.conf is loading modules in the wrong order (app_queue.so may be loaded before chan_sip.so). I have not tested this. Actually this exact same problem was reported in bug ID 0010094, but the bug was closed and the error was assumed to be a user-supplied modules.conf. I'm using the default modules.conf file and the the bug is still present. This behavior is also present in Asterisk 1.4.4. ****** STEPS TO REPRODUCE ****** Start asterisk with default modules.conf included with distribution. If SIP is used and queues are defined, (Invalid) will be displayed until a 'reload' or 'module reload app_queue.so' is issued.
Comments:	By: Benjamin Howell (bhowell) 2007-08-09 15:09:30 Additionally I am not able to get the workaround described in that bug ID 0010094 to work correctly. I put the following at the end of modules.conf and the queues still won't function after asterisk starts up and is then subsequently reloaded: load => chan_zap.so load => chan_sip.so load => app_queue.so By: Mark Michelson (mmichelson) 2007-08-09 18:05:09 I think your problem is related to the module load order. The difference between this issue and ASTERISK-9787 is that you're using local channels. So try adding these lines to the end of modules.conf instead. load => chan_sip.so load => chan_local.so load => app_queue.so By: Digium Subversion (svnbot) 2007-08-10 08:31:40 Repository: asterisk Revision: 78951 ------------------------------------------------------------------------ r78951 | file | 2007-08-10 08:31:39 -0500 (Fri, 10 Aug 2007) | 4 lines (closes issue ASTERISK-10058) Reported by: bhowell Add note to sample configuration about module load order and how it can cause perfectly good queue members to be marked as invalid. ------------------------------------------------------------------------ By: Digium Subversion (svnbot) 2007-08-10 08:33:18 Repository: asterisk Revision: 78952 ------------------------------------------------------------------------ r78952 | file | 2007-08-10 08:33:18 -0500 (Fri, 10 Aug 2007) | 12 lines Merged revisions 78951 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78951 | file | 2007-08-10 10:49:19 -0300 (Fri, 10 Aug 2007) | 4 lines (closes issue ASTERISK-10058) Reported by: bhowell Add note to sample configuration about module load order and how it can cause perfectly good queue members to be marked as invalid. ........ ------------------------------------------------------------------------ By: Digium Subversion (svnbot) 2007-08-10 10:14:52 Repository: asterisk Revision: 79012 ------------------------------------------------------------------------ r79012 | murf | 2007-08-10 10:14:50 -0500 (Fri, 10 Aug 2007) | 156 lines Merged revisions 78750,78779,78807,78824,78828-78829,78861-78862,78906,78908,78952,78956 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ................ r78750 | russell | 2007-08-09 11:25:09 -0600 (Thu, 09 Aug 2007) | 17 lines Merged revisions 78749 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78749 | russell | 2007-08-09 12:24:40 -0500 (Thu, 09 Aug 2007) | 9 lines Fix subscriptions to multiple mailboxes for ODBC_STORAGE.  Also, leave a comment for this to be fixed for IMAP_STORAGE, as well.  I left IMAP alone since I know MarkM was working on this code right now for another reason. This is broken even worse in trunk, but for a different reason.  The fact that the mailbox option supported multiple mailboxes is completely not obvious from the code in the channel drivers.  Anyway, I will fix that in another commit ... ........ ................ r78779 | russell | 2007-08-09 11:59:06 -0600 (Thu, 09 Aug 2007) | 9 lines Merged revisions 78778 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78778 | russell | 2007-08-09 12:58:31 -0500 (Thu, 09 Aug 2007) | 1 line add a comment to indicate that inboxcount for ODBC_STORAGE needs to be fixed to support multiple mailboxes ........ ................ r78807 | russell | 2007-08-09 13:11:53 -0600 (Thu, 09 Aug 2007) | 4 lines Restore the ability to have multiple mailboxes listed for the mailbox option in sip.conf.  chan_sip now maintains separate internal MWI subscriptions for each one. ................ r78824 | russell | 2007-08-09 13:35:07 -0600 (Thu, 09 Aug 2007) | 2 lines When looking up a mailbox, use the default context if not specified as something else ................ r78828 | mmichelson | 2007-08-09 14:07:32 -0600 (Thu, 09 Aug 2007) | 10 lines Blocked revisions 78826 via svnmerge ........ r78826 | mmichelson | 2007-08-09 14:52:43 -0500 (Thu, 09 Aug 2007) | 3 lines I broke canreinvite...Now I'm fixing it. I put some new code in the wrong place and so I've reverted the canreinvite section to how it was and put my new code where it should be. ........ ................ r78829 | russell | 2007-08-09 14:07:59 -0600 (Thu, 09 Aug 2007) | 3 lines Don't use strncpy for moving a chunk of memory to another that is overlapping. This was found by running Asterisk under valgrind. ................ r78861 | mmichelson | 2007-08-09 16:18:23 -0600 (Thu, 09 Aug 2007) | 17 lines Merged revisions 78859 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78859 | mmichelson | 2007-08-09 16:51:17 -0500 (Thu, 09 Aug 2007) | 9 lines Quite a few changes regarding IMAP storage. 1. instead of using inboxcount as the core message counting function, we use messagecount instead. This makes it possible to count messages in folders besides just INBOX and Old. 2. inboxcount and hasvoicemail now use messagecount as their means of determining return values. 3. Added a copy_message function for IMAP storage. Unfortunately I don't have the means to test it, but it seems like a pretty straightforward function. 4. Removed a #ifndef IMAP_STORAGE and matching #endif from leave_voicemail for a couple of reasons. One, we want to support copying mail to multiple IMAP boxes, and two, IMAP was   broken because a STORE macro had been moved into this section of code. ........ ................ r78862 | mmichelson | 2007-08-09 16:19:45 -0600 (Thu, 09 Aug 2007) | 10 lines Blocked revisions 78860 via svnmerge ........ r78860 | mmichelson | 2007-08-09 17:03:48 -0500 (Thu, 09 Aug 2007) | 3 lines Removing some extra debug code I left in my last commit ........ ................ r78906 | murf | 2007-08-09 17:32:23 -0600 (Thu, 09 Aug 2007) | 9 lines Merged revisions 78891 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78891 | murf | 2007-08-09 17:10:46 -0600 (Thu, 09 Aug 2007) | 1 line This fixes bug 10416; thanks to mvanbaak for the pretty output ........ ................ r78908 | mmichelson | 2007-08-09 17:49:05 -0600 (Thu, 09 Aug 2007) | 12 lines Merged revisions 78907 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78907 | mmichelson | 2007-08-09 18:47:00 -0500 (Thu, 09 Aug 2007) | 4 lines Improved a bit of logic regarding comma-separated mailboxes in has_voicemail. Also added some braces to some compound if statements since unbraced if statements scare me in general. ........ ................ r78952 | file | 2007-08-10 07:50:58 -0600 (Fri, 10 Aug 2007) | 12 lines Merged revisions 78951 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78951 | file | 2007-08-10 10:49:19 -0300 (Fri, 10 Aug 2007) | 4 lines (closes issue ASTERISK-10058) Reported by: bhowell Add note to sample configuration about module load order and how it can cause perfectly good queue members to be marked as invalid. ........ ................ r78956 | file | 2007-08-10 08:17:42 -0600 (Fri, 10 Aug 2007) | 10 lines Merged revisions 78955 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r78955 | file | 2007-08-10 11:15:53 -0300 (Fri, 10 Aug 2007) | 2 lines Don't bother having the core pass through or emulate begin DTMF frames when in an ast_waitstream. It only cares about the end of DTMF. ........ ................ ------------------------------------------------------------------------