From a6b287e89cf2c84f43326f58bcd81621f89f3a47 Mon Sep 17 00:00:00 2001 From: Shireesh Anjal Date: Fri, 19 Aug 2011 11:25:29 +0530 Subject: Modified not to consider success if the server we are trying to remove is already removed from the cluster. --- .../gluster/storage/management/console/actions/RemoveServerAction.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'src') diff --git a/src/com.gluster.storage.management.console/src/com/gluster/storage/management/console/actions/RemoveServerAction.java b/src/com.gluster.storage.management.console/src/com/gluster/storage/management/console/actions/RemoveServerAction.java index 936de599..8f52657c 100644 --- a/src/com.gluster.storage.management.console/src/com/gluster/storage/management/console/actions/RemoveServerAction.java +++ b/src/com.gluster.storage.management.console/src/com/gluster/storage/management/console/actions/RemoveServerAction.java @@ -70,7 +70,6 @@ public class RemoveServerAction extends AbstractMonitoredActionDelegate { } catch (Exception e) { if (!serverExists(server.getName())) { modelManager.removeGlusterServer((GlusterServer) server); - successServers.add(server); } errMsg += "[" + server.getName() + "] : " + e.getMessage() + CoreConstants.NEWLINE; } @@ -98,7 +97,7 @@ public class RemoveServerAction extends AbstractMonitoredActionDelegate { String errMsg) { if (successServers.size() == selectedServers.size()) { if(selectedServers.size() == 1) { - showInfoDialog(dialogTitle, "Server [" + selectedServers.iterator().next() + "] removed successfully!" + CoreConstants.NEWLINE + errMsg); + showInfoDialog(dialogTitle, "Server [" + selectedServers.iterator().next() + "] removed successfully!"); } else { showInfoDialog(dialogTitle, "Following servers removed successfully: " + CoreConstants.NEWLINE + selectedServers + CoreConstants.NEWLINE + errMsg); -- cgit