Summary:ASTERISK-17421: [patch] Proposed better handling of negative penalty's
Reporter:Gregory Hinton Nietsky (irroot)Labels:
Date Opened:2011-02-17 02:04:40.000-0600Date Closed:2011-06-29 01:50:22
Versions: Frequency of
Environment:Attachments:( 0) app_queue_negpenalty.patch
Description:Hi there im not happy with the current behavior of app_queue when the agent penalty is negative i personally believe this should indicate a "logged out/deleted" the patch does this as well as enabling cli agent remove of realtime agents by using the proposed behavior.


This is a change in behavior and is only eligible for Trunk if accepted.
Comments:By: Gregory Hinton Nietsky (irroot) 2011-02-18 09:06:46.000-0600


By: Leif Madsen (lmadsen) 2011-02-18 13:33:34.000-0600

I would also suggest making this a topic of discussion on the asterisk-dev list since this changes behaviour.

By: Gregory Hinton Nietsky (irroot) 2011-02-19 14:08:07.000-0600

in the latest patch its a "global" option with no change unless set. thx for the feedback.

By: Gregory Hinton Nietsky (irroot) 2011-06-29 01:50:22.290-0500

r325483 | irroot | 2011-06-29 08:39:26 +0200 (Wed, 29 Jun 2011) | 22 lines

Commit "distrotech" app_queue changes to Trunk

* Added general option negative_penalty_invalid default off. when set
  members are seen as invalid/logged out when there penalty is negative.  
  for realtime members when set remove from queue will set penalty to -1.  
* Added queue option autopausedelay when autopause is enabled it will be
  delayed for this number of seconds since last successful call if there
  was no prior call the agent will be autopaused immediately.
* Added member option ignorebusy this when set and ringinuse is not  
  will allow per member control of multiple calls as ringinuse does for
  the Queue.
- Mark QUEUE_MEMBER_PENALTY Deprecated it never worked for realtime members
- QUEUE_MEMBER is now R/W supporting setting paused, ignorebusy and penalty.

(closes issue ASTERISK-17421)
(closes issue ASTERISK-17391)
Reported by: irroot
Tested by: irroot, jrose
Review: https://reviewboard.asterisk.org/r/1119/