[asterisk-bugs] [Asterisk 0012858]: "queue show" shows old (cached) information, whilst "queue show QUEUENAME" shows current information when using realtime.
noreply at bugs.digium.com
noreply at bugs.digium.com
Thu Jun 26 16:00:16 CDT 2008
A NOTE has been added to this issue.
======================================================================
http://bugs.digium.com/view.php?id=12858
======================================================================
Reported By: bcnit
Assigned To: putnopvut
======================================================================
Project: Asterisk
Issue ID: 12858
Category: Applications/app_queue
Reproducibility: always
Severity: minor
Priority: normal
Status: closed
Asterisk Version: 1.6.0-beta9
SVN Branch (only for SVN checkouts, not tarball releases): N/A
SVN Revision (number only!):
Disclaimer on File?: N/A
Request Review:
Resolution: fixed
Fixed in Version:
======================================================================
Date Submitted: 06-14-2008 07:54 CDT
Last Modified: 06-26-2008 16:00 CDT
======================================================================
Summary: "queue show" shows old (cached) information, whilst
"queue show QUEUENAME" shows current information when using realtime.
Description:
Whenever 'queue show' is used, realtime is not queried for any updates.
When 'queue show QUEUENAME' is used, it is.
This is defined in apps/app_queue.c:
/* We only want to load realtime queues when a specific queue is asked
for. */
if (argc == 3) /* specific queue */
load_realtime_queue(argv[2]);
I appreciate that this has been deliberately coded this way, but in my
opinion, this is erroneous behaviour.
If realtime information is not loaded for each queue, then the output of
'queue show' is just plain wrong and can't be relied on - the command
simply becomes useless.
The response to bug http://bugs.digium.com/view.php?id=0012556 was that the AMI
should be used to collect
status information. At the moment, one must be aware of each queue name in
order to retrieve status information for it - The problem is that third
party applications cannot report on the status of queues using the AMI
without having this knowledge (i.e. access to the realtime data store). In
addition, even if they were aware of this, they would have to make one
'queue show QUEUENAME' call for each of the queues they require information
on - this is clearly inefficient at best.
I believe that the fact that 'queue show' returns incorrect information
should be considered a bug.
======================================================================
----------------------------------------------------------------------
svnbot - 06-26-08 16:00
----------------------------------------------------------------------
Repository: asterisk
Revision: 125479
_U team/seanbright/resolve-shadow-warnings/
U team/seanbright/resolve-shadow-warnings/apps/app_queue.c
------------------------------------------------------------------------
r125479 | seanbright | 2008-06-26 15:59:57 -0500 (Thu, 26 Jun 2008) | 84
lines
Merged revisions 125332-125333,125385-125386,125438,125477 via svnmerge
from
https://origsvn.digium.com/svn/asterisk/trunk
................
r125332 | russell | 2008-06-26 11:37:01 -0400 (Thu, 26 Jun 2008) | 5 lines
- add get_max_rate timing API call
- change ast_settimeout() to honor max rate in edge cases of file playback
(this will make some warning messages go away at the end of playing back
a file)
................
r125333 | kpfleming | 2008-06-26 11:50:07 -0400 (Thu, 26 Jun 2008) | 13
lines
Merged revisions 125327 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r125327 | kpfleming | 2008-06-26 10:30:33 -0500 (Thu, 26 Jun 2008) | 5
lines
ensure that (whenever possible) if we generate a log message because an
ioctl() call to DAHDI/Zaptel failed, that we include the reason it failed
by including the stringified error number
(issue AST-80)
........
................
r125385 | oej | 2008-06-26 12:54:22 -0400 (Thu, 26 Jun 2008) | 12 lines
Merged revisions 125384 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r125384 | oej | 2008-06-26 18:32:08 +0200 (Tor, 26 Jun 2008) | 3 lines
Add support for peer realm based auth (a few missing lines, the rest is
well documented but never worked)
........
................
r125386 | tilghman | 2008-06-26 13:06:17 -0400 (Thu, 26 Jun 2008) | 6
lines
Convert casts to unions, to fix alignment issues on Solaris
(closes issue http://bugs.digium.com/view.php?id=12932)
Reported by: snuffy
Patches:
bug_12932_20080627.diff uploaded by snuffy (license 35)
................
r125438 | tilghman | 2008-06-26 13:40:25 -0400 (Thu, 26 Jun 2008) | 6
lines
Don't play "your message has been saved" twice.
(closes issue http://bugs.digium.com/view.php?id=12893)
Reported by: jaroth
Patches:
duplicate_saved.patch uploaded by jaroth (license 50)
................
r125477 | mmichelson | 2008-06-26 16:57:41 -0400 (Thu, 26 Jun 2008) | 19
lines
Merged revisions 125476 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4
........
r125476 | mmichelson | 2008-06-26 15:56:01 -0500 (Thu, 26 Jun 2008) | 11
lines
Prior to this patch, the "queue show" command used cached
information for realtime queues instead of giving up-to-date
info. Now realtime is queried for the latest and greatest in
queue info.
(closes issue http://bugs.digium.com/view.php?id=12858)
Reported by: bcnit
Patches:
queue_show.patch uploaded by putnopvut (license 60)
........
................
------------------------------------------------------------------------
http://svn.digium.com/view/asterisk?view=rev&revision=125479
Issue History
Date Modified Username Field Change
======================================================================
06-26-08 16:00 svnbot Checkin
06-26-08 16:00 svnbot Note Added: 0089317
======================================================================
More information about the asterisk-bugs
mailing list