Hi Simon,
On Tue, Apr 28, 2015 at 10:58:56AM +0900, Simon Horman wrote:
> This seems only to lead to excessive verbosity which seems
> much more appropriate for logs than email.
>
> Signed-off-by: Simon Horman <ho...@verge.net.au>
> ---
> src/checks.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/src/checks.c b/src/checks.c
> index 3702d9a4b0fe..efcaff20219b 100644
> --- a/src/checks.c
> +++ b/src/checks.c
> @@ -316,7 +316,6 @@ static void set_server_check_status(struct check *check,
> short status, const cha
>
> Warning("%s.\n", trash.str);
> send_log(s->proxy, LOG_NOTICE, "%s.\n", trash.str);
> - send_email_alert(s, LOG_NOTICE, "%s", trash.str);
Just a question, shouldn't we keep it and send it as LOG_INFO instead ?
That way users can choose whether to have them or not. Just a suggestion,
otherwise I'm fine with this as well.
Willy
