| Summary: | ASTERISK-15243: [patch] keepstats option removed when it shouldn't | ||
| Reporter: | raarts (raarts) | Labels: | |
| Date Opened: | 2009-11-30 12:14:25.000-0600 | Date Closed: | 2010-06-23 10:59:14 |
| Priority: | Minor | Regression? | No |
| Status: | Closed/Complete | Components: | Applications/app_queue |
| Versions: | Frequency of Occurrence | ||
| Related Issues: | |||
| Environment: | Attachments: | ( 0) keepstats.diff ( 1) keepstats-2.patch ( 2) keepstats-3.patch | |
| Description: | In 1.6.2.x the keepstats option in queues.conf was removed, because a queue reset CLI command was introduced. But this option is still useful, because you should be able to set a default for normal CLI reload behaviour. ****** ADDITIONAL INFORMATION ****** I'll add a patch to reinstate this option. | ||
| Comments: | By: raarts (raarts) 2009-12-04 00:42:29.000-0600 License is approved by legal By: raarts (raarts) 2010-01-19 10:26:11.000-0600 I just made a patch against trunk, tested it, and found another problem. A new patch is uploaded. This one is the definitive one as far as I am concerned. By: raarts (raarts) 2010-01-19 10:33:52.000-0600 Whoops, never should say that a patch is the definitive one. I forgot to document the option in configs/queues.conf.sample. So uploaded a new version. Sorry. By: raarts (raarts) 2010-01-19 10:36:33.000-0600 Can someone remove the second patch? I inadvertently uploaded the wrong one. Thanks. By: Nico Kooijman (syspert) 2010-01-20 05:51:14.000-0600 Patch tested on fresh trunk (r 241458) with a positive result. By: Russell Bryant (russell) 2010-03-02 09:58:51.000-0600 Thanks for the patch. However, the change in behavior was intentional. The queue reset command will need to be used instead of a full reload to reset queue stats. | ||