[Home]

Summary:ASTERISK-12035: Dynamic queue member does not survive reboot/restart when using aliases
Reporter:Benoit P (b_plessis)Labels:
Date Opened:2008-05-16 17:14:55Date Closed:2008-05-27 10:41:41
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:
Since i started to using the masquerading function for dynamic queue member like that:
queue add member Local/888@queues/n to support penalty 10 as Agent/888

everytime i do a shutdown or even a 'restart now' the queue member became all invalid.

Before reboot:

database show give:
...
/Queue/PersistentMembers/support                  : Local/136@queues/n;10;0;Agent/136|Local/131@queues/n;10;0;Agent/131|Local/888@queues/n;20;0;Agent/888
...

queue show give:
support      has 0 calls (max unlimited) in 'rrmemory' strategy (20s holdtime), W:0, C:67, A:79, SL:0.0% within 0s
  Members:  
     Agent/136 with penalty 10 (dynamic) (Not in use) has taken 27 calls (last was 25203 secs
     Agent/131 with penalty 10 (dynamic) (Not in use) has taken 8 calls (last was 26886 secs
     Agent/888 with penalty 20 (dynamic) (Not in use) has taken no calls yet
  No Callers


after reboot:
/Queue/PersistentMembers/support                  : Local/136@queues/n;10;0;Agent/136|Local/131@queues/n;10;0;Agent/131|Local/888@queues/n;20;0;Agent/888

support      has 0 calls (max unlimited) in 'rrmemory' strategy (0s holdtime), W:0, C:0, A:0, SL:0.0% within 0s
  Members:
     Agent/136 with penalty 10 (dynamic) (Invalid) has taken no calls yet
     Agent/131 with penalty 10 (dynamic) (Invalid) has taken no calls yet
     Agent/888 with penalty 20 (dynamic) (Invalid) has taken no calls yet
  No Callers
Comments:By: dimitripietro (dimitripietro) 2008-05-16 21:27:22

I fixed that problem by having the channel driver used by the agent (in this case chan_local) in the preload section of the modules.conf
By: Benoit P (b_plessis) 2008-05-17 06:48:20

this doesn't work for me, i've preloaded evan sip, iax2 and local like that:
preload => chan_sip.so
preload => chan_iax2.so
preload => chan_local.so

but this doesn't change anything. Even more so disabled masquerading/aliasing doesn't change anyhing, so this must be some kind of regression, since in 1.4.18 it worked
By: Mark Michelson (mmichelson) 2008-05-19 09:57:45

I believe that dimitripietro is correct about load order being the issue, but I think the problem lies with the fact that app_queue is being loaded before chan_local.

app_queue queries the device state of all its members when it is loaded. If app_queue loads prior to chan_local, then it will be querying device state from a channel driver which has not been loaded yet.

One test that would confirm this would be to issue a "module reload app_queue.so" from the CLI when the members appear to have an "Invalid" state. If the members suddenly become "Not in use" then this probably is the issue.
By: Benoit P (b_plessis) 2008-05-22 09:40:51

Putnopvut you're right, issuing a module reload app_queue.so does the trick.

i tried to force the load order using preload in modules.conf but this doesn't work
By: Mark Michelson (mmichelson) 2008-05-23 11:32:19

What load order did you enforce? Did you also try using "load" instead of "preload" to see if that works?
By: Benoit P (b_plessis) 2008-05-23 11:36:12

yes, i used the following order:
[pre]load => chan_sip.so
[pre]load => chan_iax2.so
[pre]load => chan_local.so
[pre]load => app_queue.so
By: Tilghman Lesher (tilghman) 2008-05-23 16:29:38

Please try adding:

preload => pbx_config.so

before

load => app_queue.so

I think the reason this should work is because the devicestate of a Local channel is whether the extension exists in the dialplan.  If you haven't loaded the dialplan yet, then the Local channel denoted is Invalid.


By: Benoit P (b_plessis) 2008-05-27 10:22:20

Yes, this is working:

preload => pbx_config.so

load => chan_sip.so
load => chan_iax2.so
load => chan_local.so
load => app_queue.so

i suppose i could strip the chan_sip/chan_iax2 part, but what don't hurt ...
By: Digium Subversion (svnbot) 2008-05-27 10:39:21

Repository: asterisk
Revision: 118358

U   branches/1.4/configs/queues.conf.sample

------------------------------------------------------------------------
r118358 | tilghman | 2008-05-27 10:39:18 -0500 (Tue, 27 May 2008) | 3 lines

Add a note that pbx_config.so is needed for Local channels.
(Closes issue ASTERISK-12035)

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

http://svn.digium.com/view/asterisk?view=rev&revision=118358
By: Digium Subversion (svnbot) 2008-05-27 10:40:54

Repository: asterisk
Revision: 118359

_U  trunk/
U   trunk/configs/queues.conf.sample

------------------------------------------------------------------------
r118359 | tilghman | 2008-05-27 10:40:53 -0500 (Tue, 27 May 2008) | 11 lines

Merged revisions 118358 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r118358 | tilghman | 2008-05-27 10:45:37 -0500 (Tue, 27 May 2008) | 3 lines

Add a note that pbx_config.so is needed for Local channels.
(Closes issue ASTERISK-12035)

........

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

http://svn.digium.com/view/asterisk?view=rev&revision=118359
By: Digium Subversion (svnbot) 2008-05-27 10:41:41

Repository: asterisk
Revision: 118360

_U  branches/1.6.0/
U   branches/1.6.0/configs/queues.conf.sample

------------------------------------------------------------------------
r118360 | tilghman | 2008-05-27 10:41:39 -0500 (Tue, 27 May 2008) | 19 lines

Merged revisions 118359 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
r118359 | tilghman | 2008-05-27 10:46:58 -0500 (Tue, 27 May 2008) | 11 lines

Merged revisions 118358 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r118358 | tilghman | 2008-05-27 10:45:37 -0500 (Tue, 27 May 2008) | 3 lines

Add a note that pbx_config.so is needed for Local channels.
(Closes issue ASTERISK-12035)

........

................

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

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