Re: [PATCH 1/2] MEDIUM: Do not send email alerts corresponding to log-health-checks messages
On Tue, Apr 28, 2015 at 09:24:42AM +0200, Willy Tarreau wrote: On Tue, Apr 28, 2015 at 02:25:02PM +0900, Simon Horman wrote: On Tue, Apr 28, 2015 at 06:43:38AM +0200, Willy Tarreau wrote: 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. Good idea, I'll re-spin. In the mean time could you look at the second patch of the series? It is (currently) independent of this one. Sorry, I wasn't clear, I did so and found it fine. I can merge it if you want but just like you I know that merging only parts of series causes more trouble than they solve. Understood, I'll resubmit the entire series.
Re: [PATCH 1/2] MEDIUM: Do not send email alerts corresponding to log-health-checks messages
On Tue, Apr 28, 2015 at 02:25:02PM +0900, Simon Horman wrote: On Tue, Apr 28, 2015 at 06:43:38AM +0200, Willy Tarreau wrote: 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. Good idea, I'll re-spin. In the mean time could you look at the second patch of the series? It is (currently) independent of this one. Sorry, I wasn't clear, I did so and found it fine. I can merge it if you want but just like you I know that merging only parts of series causes more trouble than they solve. Willy
Re: [PATCH 1/2] MEDIUM: Do not send email alerts corresponding to log-health-checks messages
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
[PATCH 1/2] MEDIUM: Do not send email alerts corresponding to log-health-checks messages
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); } } -- 2.1.4
Re: [PATCH 1/2] MEDIUM: Do not send email alerts corresponding to log-health-checks messages
On Tue, Apr 28, 2015 at 06:43:38AM +0200, Willy Tarreau wrote: 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. Good idea, I'll re-spin. In the mean time could you look at the second patch of the series? It is (currently) independent of this one.