[Home]

Summary:ASTERISK-06384: [patch] Add new queue_log event
Reporter:Johann Hoehn (johann)Labels:
Date Opened:2006-02-21 09:11:03.000-0600Date Closed:2006-05-03 15:48:23
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_queue
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20060310__bug6559.diff.txt
Description:There is a conidition that can be met causing a call to be removed from the queue and no event entry in the queue_log file will correpsond to the action causing some problems with stats tracking.

If a queue is setup using the following options and a dialplan with a priority after the Queue() call will be executed if there are no agents logged in.  The call will get an entry for joining, however nothing is recorded when it leaves.

leavewhenempty = strict
joinempty = yes

Suggest adding a new event to cover the leavewhenempty option being used to the queue_log.

<timestamp>:<uniqueid>:NOAGENTSAVAIL:<position>:<origpostiion>:<holdtime>

****** ADDITIONAL INFORMATION ******

The use of joinempty = yes is to get the ENTERQUEUE event.  The leavewhenempty allows calls to go to the next prioity instead of waiting in the queue if no one is available.  In 1.2 the strict option was added allowing Callback Agents to be checked.
Comments:By: Tilghman Lesher (tilghman) 2006-03-10 23:27:00.000-0600

Please test the attached patch and give me feedback on whether the patch adds the functionality that you want.

By: Tilghman Lesher (tilghman) 2006-03-22 11:01:44.000-0600

I need feedback on this patch or this issue will be closed.

By: Johann Hoehn (johann) 2006-03-27 14:48:19.000-0600

We are running 1.2.4.  (not sure why the bug was put in with SVN head).  Had to make some minor changes(the last part wasn't needed).  We now have Asterisk up and running with it.

However since this was put in we have have people pausing themselves making it harder for the event to occur.

When the new compiled module was copied over to the directory, Asterisk died on itself.  Wasn't even able to issue a reload app_queue.so before this occured.  After a restart though everything is running fine.

By: Tilghman Lesher (tilghman) 2006-03-27 15:04:26.000-0600

The patch is against SVN trunk, because this is a feature and will never be added to 1.2.

By: Serge Vecher (serge-v) 2006-05-03 15:30:50

johann: did you try testing this patch with /trunk?

By: BJ Weschke (bweschke) 2006-05-03 15:48:23

Committed to /trunk. Thanks!