[SSSD] Re: [PATCH] Revert "DEBUG: Preventing chown_debug_file if journald on"
On Tue, Feb 23, 2016 at 04:11:14PM +0100, Lukas Slebodnik wrote:
On (23/02/16 15:33), Jakub Hrozek wrote:
>On Mon, Jan 25, 2016 at 12:00:11PM +0100, Lukas Slebodnik wrote:
>> ehlo,
>>
>> The first patch fixes regression introduced in 1.13.2.
>> https://fedorahosted.org/sssd/ticket/2493
>>
>> I'm sorry I didn't catch it in review.
>> Fortunatelly there is a simple but annoying workaround.
>>
>> Change user to "non-root" + remove old log files.
>>
>> The second patch should fix ticket #2493 in different way.
>>
>> LS
>
>I just wonder if it makes sense to print an INTERNAL DEBUG message to
>make it clear we went through that codepath, but otherwise ACK.
>
This message would be logged very often in case of
logging to journal and empty directory /var/log/sssd.
IMHO it woudl be just a confusing debug message
even though it would be logged with high debug level.
LS
Good point. ACK; CI:
http://sssd-ci.duckdns.org/logs/job/37/79/summary.html