[Home]

Summary:ASTERISK-16389: [patch] Reload logger with alternate configuration file
Reporter:Tilghman Lesher (tilghman)Labels:
Date Opened:2010-07-18 10:58:34Date Closed:2010-12-31 03:21:50.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Core/Configuration
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20100718__logger_reload_altconf__2.diff.txt
( 1) issue17668.patch.v2
Description:On Friday 16 July 2010 12:59:43 Steve Edwards wrote:
> "Request For Comments on a Feature Suggestion" -- just wondering if others
> would find this useful.
>
> It occurs to me that being able to just enter:
>
>          logger reload maximum-verbosity.conf
>
> would be a useful feature and would be useful in many situations like
> "iax2 reload" or "extensions reload."
Comments:By: Russell Bryant (russell) 2010-07-18 12:26:19

case CLI_INIT:
- e->command = "logger reload";
- e->usage =
+ e->command = "logger reload [<alt-conf>]";
+ e->usage =
"Usage: logger reload\n"
"       Reloads the logger subsystem state.  Use after restarting syslogd(8) if you are using syslog logging.\n";
return NULL;



You probably want to update usage, but leave command alone.
By: Tilghman Lesher (tilghman) 2010-07-18 13:47:49

Fixed.
By: Paul Belanger (pabelanger) 2010-07-22 10:22:28

v2:
- Fix broken compile.
By: Russell Bryant (russell) 2010-07-23 05:05:01

Ship it.
By: Leif Madsen (lmadsen) 2010-07-29 15:10:56

I'd set this to "Ship It" if I could!
By: Digium Subversion (svnbot) 2010-12-31 03:21:49.000-0600

Repository: asterisk
Revision: 300044

U   trunk/CHANGES
U   trunk/main/logger.c

------------------------------------------------------------------------
r300044 | tilghman | 2010-12-31 03:21:48 -0600 (Fri, 31 Dec 2010) | 9 lines

Support an alternate configuration file for the 'logger reload' command.

(closes issue ASTERISK-16389)
Reported by: tilghman
Patches:
      20100718__logger_reload_altconf__2.diff.txt uploaded by tilghman (license 14)

Review: (by lmadsen, russell within comments on issue tracker)

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=300044