[sheepdog] [PATCH] dog: log dog operation in syslog

[Liu Yuan]

On Fri, Dec 27, 2013 at 02:18:31PM +0900, Hitoshi Mitake wrote:
> At Fri, 27 Dec 2013 12:52:21 +0800,
> Liu Yuan wrote:
> > 
> > On Fri, Dec 27, 2013 at 11:01:40AM +0900, Hitoshi Mitake wrote:
> > > This patch is based on a request from our internal team. Currently,
> > > sheepdog doesn't provide a way of logging admin's operation. The lack
> > > of this feature is serious because we cannot find any evidence of
> > > operation.
> > > 
> > > This patch adds the logging feature in dog command. If the environment
> > > variable, SHEEPDOG_DOG_LOG, has value, dog command logs its command
> > > line argument with syslog.
> > 
> > syslog is full of logs unrelated to sheepdog. How about a dedicated log file
> > e.g, /var/log/sheepdog/dog.log to log the dog operation? SHEEPDOG_DOG_LOG can
> > be used as a hint for user defined locatin for log file. I'd like to make loging
> > of dog operation default on.
> 
> A dedicated log file should be supported by dog, too. But current
> standard syslog should also be supported. Some users may use rsyslog,
> etc and sheepdog should fit into their way.
> 
> I'll implement the custom log location soon. Could you apply this one
> first? 

Let's make it as simple as possible. Support both syslog and dedicated log file
look unnecessary to me. One log file is good enough.

Thanks
Yuan