[sheepdog] [PATCH] sheep: add an option for selecting the output directory of sheep.log
Ryusuke Konishi
konishi.ryusuke at lab.ntt.co.jp
Wed Dec 11 02:31:54 CET 2013
On Wed, 11 Dec 2013 10:22:05 +0900, Hitoshi Mitake wrote:
> At Tue, 10 Dec 2013 12:36:06 +0800,
> Liu Yuan wrote:
>>
>> On Mon, Dec 09, 2013 at 08:49:45PM +0900, Teruaki Ishizaki wrote:
>> > Previously sheep.log was saved in the Sheepdog store directory,
>> > and used size of Sheepdog store directory included log file size.
>> > So, administrator couldn't do capacity planning simply.
>> >
>> > This patch adds an option for setting the output directory of sheep.log.
>> >
>> > Usage:
>> > $ sheep -L dir=<log_output_dir>
>> >
>> > Example:
>> > $ sheep -L dir=/logdir /datastore
>>
>> I think it is a good time to group logger parameters into a single option, that
>> is
>>
>> -l level=xxx,dir=yyy,formatter=zzz.
>
> I think this patch is helpful for administrators so I want to backport
> it to stable-0.7. I want a patch for grouping the logger parameters to
> be an independent one for minimizing interface change of the stable
> branch.
So, How about separating this change to two patches?
- The first patch simply adds log directory option, and then backport
it for stable users.
- The second patch refactors the log option as above for future releases.
Regards,
Ryusuke Konishi
More information about the sheepdog
mailing list