[PATCH] CTDB logging improvements

Amitay Isaacs amitay at gmail.com
Tue Jun 13 05:55:03 UTC 2017


On Tue, Jun 13, 2017 at 3:46 PM, Martin Schwenke via samba-technical <
samba-technical at lists.samba.org> wrote:

> 2 separate commits:
>
> * ctdb-common: Log a count of dropped messages with non-blocking logging
>
>   The non-blocking logging variants can currently silently drop messages
>   when the socket queue fills.
>
>   In this case, count the number of dropped messages and attempt to log
>   a message about dropped log messages when the next message is logged.
>
> * ctdb-scripts: Compact server-end TCP connection killing output
>
>   When thousands of connections are being killed the logs are flooded
>   with information about connections that should be killed.  When some
>   connections are not killed then the number not killed is printed.
>   This is the wrong way around!  When debugging "fail-back" problems,
>   it is important to know details of connections that were *not*
>   killed.  It is almost never important to know the full list of all
>   connections that were *supposed* to be killed.
>
>   Instead, print a summary showing how many connections of the total
>   were killed.  If any were not killed then print a list of remaining
>   connections.
>
>   Update unit tests: infrastructure for fake TCP connections,
>   existing, test cases, add new test cases.
>
> Please review and maybe push...
>
> peace & happiness,
> martin
>

Reviewed-by: Amitay Isaacs <amitay at gmail.com>

Amitay.


More information about the samba-technical mailing list