| 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 |
| Priority: | Minor | Regression? | No |
| Status: | Closed/Complete | Components: | Applications/app_queue |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| 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 Hmmm... 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! | ||