Modified not to consider success if the server we are trying to remove is already removed from the cluster.

1 files changed, 1 insertions, 2 deletions

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);