|Summary:||ASTERISK-10412: [patch] app_queue tweak for QueueSummary manager event|
|Reporter:||Paul W (kwakwaversal)||Labels:|
|Date Opened:||2007-09-28 16:59:50||Date Closed:||2007-10-04 17:40:48|
|Environment:||Attachments:||( 0) app_queue_summary_patch.patch|
|Description:||The manager_queues_summary() function doesn't behave correctly when no queue filter is set. It's the common mistake of not resetting the variables inside a loop resulting in available members and calls being summed up through every queue iteration.|
This patch fixes that and also adds another useful statistic giving the longest hold time (in seconds) for each queue. Very handy for providing useful real-time stats.
|Comments:||By: Paul W (kwakwaversal) 2007-09-28 17:07:29|
Apologies for attaching the file twice, it hung for about 4 mins so I resubmitted it.
By: Mark Michelson (mmichelson) 2007-10-04 17:19:59
Are you using a patched version of Asterisk? My copy of the 1.4.11 branch does not have the qmemcount, qmemavail, or qchancount variables in it.
Edit: Those variables are in the trunk version of Asterisk, though. I'll just assume you accidentally put the wrong version in your description.
By: Mark Michelson (mmichelson) 2007-10-04 17:28:52
Thanks for the patch, by the way. I'll also add in the longest hold time change, too!
By: Mark Michelson (mmichelson) 2007-10-04 17:40:48
Fixed in Asterisk trunk, rev 84726. Thanks for the patch!