Manual Server Addition Feature

5 files changed, 342 insertions, 2 deletions

diff --git a/src/com.gluster.storage.management.gui/plugin.xml b/src/com.gluster.storage.management.gui/plugin.xml
index 1850c93e..3c87ffe1 100644
--- a/src/com.gluster.storage.management.gui/plugin.xml
+++ b/src/com.gluster.storage.management.gui/plugin.xml
@@ -462,6 +462,20 @@
                toolbarPath="Normal"
                tooltip="Create a new Volume">
          </action>
+         <action
+               allowLabelUpdate="false"
+               class="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               definitionId="com.gluster.storage.management.gui.commands.ServerAddition"
+               id="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               label="Add Server"
+               menubarPath="com.gluster.storage.management.gui.actionsets.volumes/volumes"
+               mode="FORCE_TEXT"
+               pulldown="false"
+               retarget="false"
+               state="false"
+               style="push"
+               tooltip="Manual Server Addition">
+         </action>
          <menu
                id="com.gluster.storage.management.gui.actionsets.volumes"
                label="&amp;Gluster"
@@ -631,6 +645,20 @@
                toolbarPath="Normal"
                tooltip="Create a new Volume">
          </action>
+         <action
+               allowLabelUpdate="false"
+               class="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               definitionId="com.gluster.storage.management.gui.commands.ServerAddition"
+               id="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               label="Add Server"
+               menubarPath="com.gluster.storage.management.gui.menu.volume/volume"
+               mode="FORCE_TEXT"
+               pulldown="false"
+               retarget="false"
+               state="false"
+               style="push"
+               tooltip="Manual Server Addition">
+         </action>
          <menu
                id="com.gluster.storage.management.gui.menu.volume"
                label="&amp;Gluster"
@@ -675,6 +703,20 @@
                toolbarPath="Normal"
                tooltip="Create a new Volume">
          </action>
+         <action
+               allowLabelUpdate="false"
+               class="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               definitionId="com.gluster.storage.management.gui.commands.ServerAddition"
+               id="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               label="Add Server"
+               menubarPath="com.gluster.storage.management.gui.menu.servers/servers"
+               mode="FORCE_TEXT"
+               pulldown="false"
+               retarget="false"
+               state="false"
+               style="push"
+               tooltip="Manual Server Addition">
+         </action>
          <menu
                id="com.gluster.storage.management.gui.menu.servers"
                label="&amp;Gluster"
@@ -719,6 +761,20 @@
                toolbarPath="Normal"
                tooltip="Create a new Volume">
          </action>
+         <action
+               allowLabelUpdate="false"
+               class="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               definitionId="com.gluster.storage.management.gui.commands.ServerAddition"
+               id="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               label="Add Server"
+               menubarPath="com.gluster.storage.management.gui.menu.glusterserver/glusterserver"
+               mode="FORCE_TEXT"
+               pulldown="false"
+               retarget="false"
+               state="false"
+               style="push"
+               tooltip="Manual Server Addition">
+         </action>
          <menu
                id="com.gluster.storage.management.gui.menu.glusterserver"
                label="&amp;Gluster"
@@ -763,6 +819,20 @@
                toolbarPath="Normal"
                tooltip="Create a new Volume">
          </action>
+         <action
+               allowLabelUpdate="false"
+               class="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               definitionId="com.gluster.storage.management.gui.commands.ServerAddition"
+               id="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               label="Add Server"
+               menubarPath="com.gluster.storage.management.gui.menu.discoveredservers/discoveredservers"
+               mode="FORCE_TEXT"
+               pulldown="false"
+               retarget="false"
+               state="false"
+               style="push"
+               tooltip="Manual Server Addition">
+         </action>
          <menu
                id="com.gluster.storage.management.gui.menu.discoveredservers"
                label="&amp;Gluster"
@@ -807,6 +877,20 @@
                toolbarPath="Normal"
                tooltip="Create a new Volume">
          </action>
+         <action
+               allowLabelUpdate="false"
+               class="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               definitionId="com.gluster.storage.management.gui.commands.ServerAddition"
+               id="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               label="Add Server"
+               menubarPath="com.gluster.storage.management.gui.menu.discoveredserver/discoveredserver"
+               mode="FORCE_TEXT"
+               pulldown="false"
+               retarget="false"
+               state="false"
+               style="push"
+               tooltip="Manual Server Addition">
+         </action>
          <menu
                id="com.gluster.storage.management.gui.menu.discoveredserver"
                label="&amp;Gluster"
@@ -939,6 +1023,20 @@
                toolbarPath="Normal"
                tooltip="Create a new Volume">
          </action>
+         <action
+               allowLabelUpdate="false"
+               class="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               definitionId="com.gluster.storage.management.gui.commands.ServerAddition"
+               id="com.gluster.storage.management.gui.actions.ServerAdditionAction"
+               label="Add Server"
+               menubarPath="com.gluster.storage.management.gui.menu.gluster/gluster"
+               mode="FORCE_TEXT"
+               pulldown="false"
+               retarget="false"
+               state="false"
+               style="push"
+               tooltip="Manual Server Addition">
+         </action>
          <menu
                id="com.gluster.storage.management.gui.menu.gluster"
                label="&amp;Gluster"
diff --git a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/AddServerAction.java b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/AddServerAction.java
index 81661942..6d21e537 100644
--- a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/AddServerAction.java
+++ b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/AddServerAction.java
@@ -52,7 +52,7 @@ public class AddServerAction extends AbstractActionDelegate {
 					guiHelper.setStatusMessage("Adding server [" + server.getName() + "]...");
 
 					try {
-						glusterServersClient.addServer(server);
+						glusterServersClient.addServer(server.getName());
 						modelManager.addGlusterServer(glusterServersClient.getGlusterServer(server.getName()));
 						successServers.add(server);
 					} catch (Exception e) {
diff --git a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/ServerAdditionAction.java b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/ServerAdditionAction.java
new file mode 100644
index 00000000..410b7bea
--- /dev/null
+++ b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/actions/ServerAdditionAction.java
@@ -0,0 +1,34 @@
+package com.gluster.storage.management.gui.actions;
+
+import org.apache.log4j.Logger;
+import org.eclipse.jface.action.IAction;
+import org.eclipse.jface.viewers.ISelection;
+import org.eclipse.ui.IWorkbenchWindow;
+import org.eclipse.ui.IWorkbenchWindowActionDelegate;
+
+import com.gluster.storage.management.gui.dialogs.ChangePasswordDialog;
+import com.gluster.storage.management.gui.dialogs.ServerAdditionDialog;
+import com.gluster.storage.management.gui.views.pages.AbstractDisksPage;
+
+public class ServerAdditionAction extends AbstractActionDelegate {
+	private static final Logger logger = Logger.getLogger(ServerAdditionAction.class);
+	@Override
+	public void dispose() {
+		// TODO Auto-generated method stub
+		
+	}
+
+	@Override
+	protected void performAction(IAction action) {
+		
+		try {
+			// To open a dialog for server addition
+			ServerAdditionDialog dialog = new ServerAdditionDialog(getShell());
+			dialog.open();
+		} catch (Exception e) {
+			logger.error("Error in Manual server addition", e);
+			e.printStackTrace();
+		}
+	}
+
+}
diff --git a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/InitializeDiskTypeSelection.java b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/InitializeDiskTypeSelection.java
index bb700de5..5fce27e4 100644
--- a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/InitializeDiskTypeSelection.java
+++ b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/InitializeDiskTypeSelection.java
@@ -56,7 +56,7 @@ public class InitializeDiskTypeSelection extends Dialog  {
 	protected void configureShell(Shell newShell) {
 		super.configureShell(newShell);
 
-		newShell.setText("Gluster Management Console - Select Cluster");
+		newShell.setText("Gluster Management Console - Select FS Type");
 		addEscapeListener(newShell);
 	}
 	
diff --git a/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/ServerAdditionDialog.java b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/ServerAdditionDialog.java
new file mode 100644
index 00000000..cbaabb9c
--- /dev/null
+++ b/src/com.gluster.storage.management.gui/src/com/gluster/storage/management/gui/dialogs/ServerAdditionDialog.java
@@ -0,0 +1,208 @@
+/**
+ * ServerAdditionDialog.java
+ *
+ * Copyright (c) 2011 Gluster, Inc. <http://www.gluster.com>
+ * This file is part of Gluster Management Console.
+ *
+ * Gluster Management Console is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Affero General Public License as published
+ * by the Free Software Foundation; either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Gluster Management Console is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of MERCHANTABILITY
+ * or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU Affero General Public License
+ * for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program. If not, see
+ * <http://www.gnu.org/licenses/>.
+ */
+package com.gluster.storage.management.gui.dialogs;
+
+import java.util.HashSet;
+import java.util.List;
+import java.util.Set;
+
+import org.eclipse.core.databinding.DataBindingContext;
+import org.eclipse.core.databinding.UpdateValueStrategy;
+import org.eclipse.core.databinding.beans.PojoProperties;
+import org.eclipse.jface.databinding.swt.SWTObservables;
+import org.eclipse.jface.databinding.swt.WidgetProperties;
+import org.eclipse.jface.dialogs.Dialog;
+import org.eclipse.jface.dialogs.ErrorDialog;
+import org.eclipse.jface.dialogs.IDialogConstants;
+import org.eclipse.jface.dialogs.MessageDialog;
+import org.eclipse.jface.fieldassist.ControlDecoration;
+import org.eclipse.swt.SWT;
+import org.eclipse.swt.events.ModifyEvent;
+import org.eclipse.swt.events.ModifyListener;
+import org.eclipse.swt.events.TraverseEvent;
+import org.eclipse.swt.events.TraverseListener;
+import org.eclipse.swt.layout.GridData;
+import org.eclipse.swt.layout.GridLayout;
+import org.eclipse.swt.widgets.Button;
+import org.eclipse.swt.widgets.Composite;
+import org.eclipse.swt.widgets.Control;
+import org.eclipse.swt.widgets.Display;
+import org.eclipse.swt.widgets.Label;
+import org.eclipse.swt.widgets.Shell;
+import org.eclipse.swt.widgets.Text;
+
+import com.gluster.storage.management.client.GlusterDataModelManager;
+import com.gluster.storage.management.client.GlusterServersClient;
+import com.gluster.storage.management.core.model.GlusterServer;
+import com.gluster.storage.management.core.model.Server;
+import com.gluster.storage.management.core.model.Volume;
+import com.gluster.storage.management.gui.utils.GUIHelper;
+import com.gluster.storage.management.gui.validators.StringRequiredValidator;
+
+public class ServerAdditionDialog extends Dialog {
+	public static final int RETURN_CODE_ERROR = 2;
+	private Text serverName;
+	private Button addButton;
+
+	private final GUIHelper guiHelper = GUIHelper.getInstance();
+	private ControlDecoration errDecoration;
+
+	private Composite composite;
+
+	public ServerAdditionDialog(Shell shell) {
+		super(shell);
+	}
+
+	@Override
+	protected void configureShell(Shell newShell) {
+		super.configureShell(newShell);
+
+		newShell.setText("Gluster Management Console - Add Server");
+		addEscapeListener(newShell);
+	}
+
+	private void addEscapeListener(Shell shell) {
+		shell.addTraverseListener(new TraverseListener() {
+
+			@Override
+			public void keyTraversed(TraverseEvent e) {
+				if (e.keyCode == SWT.ESC) {
+					cancelPressed();
+				}
+			}
+		});
+	}
+
+	/**
+	 * Overriding to make sure that the dialog is centered in screen
+	 */
+	@Override
+	protected void initializeBounds() {
+		super.initializeBounds();
+
+		guiHelper.centerShellInScreen(getShell());
+	}
+
+	private void configureDialogLayout(Composite composite) {
+		GridLayout layout = (GridLayout) composite.getLayout();
+		layout.numColumns = 2;
+		layout.marginLeft = 20;
+		layout.marginRight = 20;
+		layout.marginTop = 20;
+		layout.horizontalSpacing = 20;
+		layout.verticalSpacing = 20;
+	}
+
+	// ------------------------------------------
+
+	private void createLabel(Composite composite, String label) {
+		Label passwordLabel = new Label(composite, SWT.NONE);
+		passwordLabel.setText(label);
+		passwordLabel.setLayoutData(new GridData(GridData.END, GridData.CENTER, false, false));
+	}
+
+	private Text createServerNameText(Composite composite) {
+		Text field = new Text(composite, SWT.BORDER );
+		GridData layoutData = new GridData(SWT.FILL, GridData.FILL, true, false);
+		layoutData.widthHint = convertWidthInCharsToPixels(32);
+		field.setLayoutData(layoutData);
+		return field;
+	}
+
+	@Override
+	protected Control createDialogArea(Composite parent) {
+
+		composite = (Composite) super.createDialogArea(parent);
+		configureDialogLayout(composite);
+
+		createLabel(composite, "Server Name:");
+		serverName = createServerNameText(composite);
+		errDecoration = guiHelper.createErrorDecoration(serverName);
+
+		createListeners();
+
+		return composite;
+	}
+
+	private void createListeners() {
+		ModifyListener listener = new ModifyListener() {
+			@Override
+			public void modifyText(ModifyEvent e) {
+				updateButtonStatus();
+			}
+		};
+		
+		serverName.addModifyListener(listener);
+	}
+
+	private void updateButtonStatus() {
+		addButton.setEnabled(true);
+		errDecoration.hide();
+
+		if(!serverExists(serverName.getText())) {
+			addButton.setEnabled(false);
+			errDecoration.setDescriptionText("Server name already exists.");
+			errDecoration.show();
+		}
+		
+		if(serverName.getText().isEmpty()) {
+			addButton.setEnabled(false);
+			errDecoration.setDescriptionText("Please enter server name!");
+			errDecoration.show();
+		}
+	}
+
+	@Override
+	protected void createButtonsForButtonBar(Composite parent) {
+		addButton = createButton(parent, IDialogConstants.OK_ID, "&Add Server", true);
+		addButton.setEnabled(false);
+		createButton(parent, IDialogConstants.CANCEL_ID, IDialogConstants.CANCEL_LABEL, false);
+	}
+	
+	public Boolean serverExists(String serverName) {
+		List<GlusterServer> servers = GlusterDataModelManager.getInstance().getModel().getCluster().getServers();
+		for (GlusterServer server : servers) {
+			if (server.getName().equals(serverName)) {
+				return false;
+			}
+		}
+		return true;
+	}
+
+	protected void okPressed() {
+		GlusterServersClient serversClient = new GlusterServersClient();
+		GlusterDataModelManager modelManager = GlusterDataModelManager.getInstance();
+
+		try {
+			String serverNameText = serverName.getText();
+			serversClient.addServer(serverNameText);
+
+			modelManager.addGlusterServer(serversClient.getGlusterServer(serverNameText));
+
+			MessageDialog
+					.openInformation(getShell(), "Add Server", "Server " + serverNameText + " added successfully!");
+		} catch (Exception e) {
+			MessageDialog.openError(getShell(), "Server addition Failed", e.getMessage());
+			setReturnCode(RETURN_CODE_ERROR);
+		}
+		this.close();
+	}
+}