From 91fd8d7cd5d5a83bf75f5f2972ad2eb2a4a07694 Mon Sep 17 00:00:00 2001 From: Rohit Yadav Date: Wed, 10 Sep 2014 11:29:30 +0200 Subject: [PATCH] CLOUDSTACK-7528: More verbose logging when sending alert fails When sendAlert is called on an AlertManager impl, if it fails it logs that something was wrong but does not log the body of the issue/error. This means we tell the user/admin that there was an issue but don't share the "issue" with them at all as the email alert fail (or that they were not initialized). Signed-off-by: Rohit Yadav (cherry picked from commit 885c02dbd8b40d654f4a84fca8474ff88bc7ca5e) Signed-off-by: Rohit Yadav Conflicts: server/src/com/cloud/alert/AlertManagerImpl.java usage/src/com/cloud/usage/UsageAlertManagerImpl.java --- server/src/com/cloud/alert/AlertManagerImpl.java | 2 +- usage/src/com/cloud/usage/UsageAlertManagerImpl.java | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/server/src/com/cloud/alert/AlertManagerImpl.java b/server/src/com/cloud/alert/AlertManagerImpl.java index 1dba13b6cb2..3f599a19f6a 100755 --- a/server/src/com/cloud/alert/AlertManagerImpl.java +++ b/server/src/com/cloud/alert/AlertManagerImpl.java @@ -252,7 +252,7 @@ public class AlertManagerImpl extends ManagerBase implements AlertManager, Confi _emailAlert.sendAlert(alertType, dataCenterId, podId, null, subject, body); } else { s_alertsLogger.warn(" alertType:: " + alertType + " // dataCenterId:: " + dataCenterId + " // podId:: " + podId + - " // message:: " + subject); + " // message:: " + subject + " // body:: " + body); } } catch (Exception ex) { s_logger.error("Problem sending email alert", ex); diff --git a/usage/src/com/cloud/usage/UsageAlertManagerImpl.java b/usage/src/com/cloud/usage/UsageAlertManagerImpl.java index 67bb7020d20..3c55f57ede0 100644 --- a/usage/src/com/cloud/usage/UsageAlertManagerImpl.java +++ b/usage/src/com/cloud/usage/UsageAlertManagerImpl.java @@ -105,7 +105,7 @@ public class UsageAlertManagerImpl extends ManagerBase implements AlertManager { _emailAlert.sendAlert(alertType, dataCenterId, podId, subject, body); } else { s_alertsLogger.warn(" alertType:: " + alertType + " // dataCenterId:: " + dataCenterId + " // podId:: " + podId + " // clusterId:: " + null + - " // message:: " + subject); + " // message:: " + subject + " // body:: " + body); } } catch (Exception ex) { s_logger.error("Problem sending email alert", ex);