URL: https://github.com/SSSD/sssd/pull/404 Title: #404: Log to syslog whether the DP is online or offline
fidencio commented: """ On Wed, Oct 18, 2017 at 2:34 PM, lslebodn notifications@github.com wrote:
On (18/10/17 04:22), fidencio wrote:
@lslebodn: Because I don't see the point of having *a* *lot* of
consecutive patches saying "Backend is offline" for instance.
But if you disagree, I can just drop the patch.
Let's imagine that you will analyze 100MiB file. And you are not sure about state of backend because last change was 17469 lines before.
Tracing debug messages have purpose.
Okay, makes sense. Patch has been dropped from the PR.
Please, as this is something desired for the release on Friday I'd like
to ask for a quick answer.
And I prefer quality instead of half baked features before deadliens (but it is not this case :-)
Can you elaborate a bit more in the "but it is not this case" part of sentence?
I'm basically interested to know whether you meant that this PR is an example of a non-quality half-baked feature. So, can you tell me whether that's what you meant?
LS
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/SSSD/sssd/pull/404#issuecomment-337575604, or mute the thread https://github.com/notifications/unsubscribe-auth/AAG4emLa8YT72Tpt1m_GRI6WazSwVzqtks5stfBFgaJpZM4P2q30 .
"""
See the full comment at https://github.com/SSSD/sssd/pull/404#issuecomment-337577341