| Summary: | ASTERISK-23662: Extended Wrap terminated by AMI | ||
| Reporter: | Brett Sutton (bsutton) | Labels: | |
| Date Opened: | 2014-04-23 19:56:10 | Date Closed: | 2014-05-19 13:10:12 |
| Priority: | Major | Regression? | |
| Status: | Closed/Complete | Components: | Applications/app_queue |
| Versions: | Feature Tracker | Frequency of Occurrence | |
| Related Issues: | |||
| Environment: | Attachments: | ||
| Description: | When an agent completes a call they often have additional 'paper work' to complete before they are ready to take the next call.
The current 'wrap' function only provides a fixed wrap interval which is often not appropriate given the variablity of the work required by some calls. The patch enables a queue to be put into 'manager wrap' mode. In manager wrap mode at the end of a call the agent is immediately placed into an 'indefinite wrap' period. When the agent completes their task they can 'via some externally defined UI' end their wrap by sending a 'end manager wrap' AMI call to the queue. This patch can be somewhat dangerous in a production environment as agents can get stuck in wrap if the 'externally defined UI' crashes. This issue is some what mitigated by the fact that if the 'use manager wrap' flag is disabled on a queue all agents in wrap will be immediately released from wrap. The elegance of this method is that it is much cheaper to implement than methods which pause/unpausing the agent in all queues and much more reliable than the pause/unpause method as that method can allow calls to 'slip through' if the pause is not enacted before the standard fixed wrap completes. We also found that in an environment with 40 queues and 30 agents (each agent in every queue) the lock contention of pausing/unpausing agents in every queue was bringing asterisk to its knees. | ||
| Comments: | By: Rusty Newton (rnewton) 2014-04-25 15:42:29.475-0500 Hi Brett, I don't see that you have posted a patch on any of your issues yet. Let me know if you are running into any issues. The [Code Revew|https://wiki.asterisk.org/wiki/display/AST/Code+Review] wiki page should cover just about everything you need to know for the code review process, including signing the license agreement that will allow you to post your patches onto this issue tracker. By: Rusty Newton (rnewton) 2014-05-19 13:10:12.997-0500 Please contact a bug marshal in #asterisk-bugs or #asterisk-dev on irc.freenode.net when you are ready to attach your patches to the issues and we can re-open them. Thanks! | ||