summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorSelvasundaram <selvam@gluster.com>2011-07-21 18:33:22 +0530
committerSelvasundaram <selvam@gluster.com>2011-07-21 18:33:22 +0530
commit53ba8efce4dafc55204ff673455d6e89d02dc330 (patch)
tree10ae7d14e579833c7c369906b1c7c0aebffb7db0 /src
parentd490f00daba667bb3d52eaf4eaef7d6575f2ed36 (diff)
Server alert bug fix
Diffstat (limited to 'src')
-rw-r--r--src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/AlertsManager.java13
1 files changed, 10 insertions, 3 deletions
diff --git a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/AlertsManager.java b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/AlertsManager.java
index ed700d53..8af61058 100644
--- a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/AlertsManager.java
+++ b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/AlertsManager.java
@@ -101,7 +101,10 @@ public class AlertsManager {
private List<Alert> getServerAlerts() {
List<Alert> serverAlerts = new ArrayList<Alert>();
- serverAlerts.add(getOfflineServerAlerts()); // Single alert for offline servers
+ Alert offlineServerAlert = getOfflineServerAlerts();
+ if (offlineServerAlert != null) {
+ serverAlerts.add(offlineServerAlert); // Single alert for offline servers
+ }
for (GlusterServer server : cluster.getServers()) {
// To check off line servers
@@ -140,8 +143,12 @@ public class AlertsManager {
offlineServers.add(server.getName());
}
}
- return new Alert(ALERT_TYPES.OFFLINE_SERVERS_ALERT, "Server",
- Alert.ALERT_TYPE_STR[ALERT_TYPES.OFFLINE_SERVERS_ALERT.ordinal()] + "(s) " + offlineServers.toString());
+ if (offlineServers.size() > 0) {
+ return new Alert(ALERT_TYPES.OFFLINE_SERVERS_ALERT, "Server",
+ Alert.ALERT_TYPE_STR[ALERT_TYPES.OFFLINE_SERVERS_ALERT.ordinal()] + "(s) "
+ + offlineServers.toString());
+ }
+ return null;
}
private List<Alert> getLowDiskAlerts(GlusterServer server) {