diff options
author | Dhandapani <dhandapani@gluster.com> | 2011-10-27 14:52:05 +0530 |
---|---|---|
committer | Dhandapani <dhandapani@gluster.com> | 2011-10-27 18:54:35 +0530 |
commit | 9f57717a53a9e7d62975d992a5f3e3e294a33b32 (patch) | |
tree | b789f9c9c613d65e684bcc3881f4e079c1dd1c4b /src/com.gluster.storage.management.console/src | |
parent | 3526939541bfa4a08317de7203ccb763ffcd218c (diff) | |
parent | a650d155c77c1fea290eb7dfb1f30ce163c0359e (diff) |
Review comments incorporated
Merge branch 'master' of git@github.com:Dhandapani/console
Diffstat (limited to 'src/com.gluster.storage.management.console/src')
-rw-r--r-- | src/com.gluster.storage.management.console/src/com/gluster/storage/management/console/actions/ForceStartVolumeAction.java | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/src/com.gluster.storage.management.console/src/com/gluster/storage/management/console/actions/ForceStartVolumeAction.java b/src/com.gluster.storage.management.console/src/com/gluster/storage/management/console/actions/ForceStartVolumeAction.java index f6f0e44b..101b7982 100644 --- a/src/com.gluster.storage.management.console/src/com/gluster/storage/management/console/actions/ForceStartVolumeAction.java +++ b/src/com.gluster.storage.management.console/src/com/gluster/storage/management/console/actions/ForceStartVolumeAction.java @@ -12,6 +12,7 @@ import com.gluster.storage.management.console.views.VolumeBricksView; import com.gluster.storage.management.core.model.Brick; import com.gluster.storage.management.core.model.Brick.BRICK_STATUS; import com.gluster.storage.management.core.model.Volume; +import com.gluster.storage.management.core.utils.StringUtil; public class ForceStartVolumeAction extends AbstractActionDelegate { @@ -28,17 +29,18 @@ public class ForceStartVolumeAction extends AbstractActionDelegate { protected void performAction(IAction action) { // volume brick service will be started, do you want to continue? final String actionDesc = action.getDescription(); - boolean confirmed = showConfirmDialog(actionDesc, - "Volume brick service will be started, do you want to continue?"); + boolean confirmed = showConfirmDialog( + actionDesc, + "The offline Bricks [" + StringUtil.collectionToString(bricks, ", ") + "] of Volume [" + + volume.getName() + "] will be started. Are you sure you want to continue?"); if (!confirmed) { return; } try { new VolumesClient().startVolume(volume.getName(), true); - showInfoDialog(actionDesc, "Volume [" + volume.getName() + "] forcefully started successfully!"); + showInfoDialog(actionDesc, "Offline Bricks of Volume [" + volume.getName() + "] started successfully!"); } catch (Exception e) { - showErrorDialog(actionDesc, - "Failed to forcefully start volume [" + volume.getName() + "]! Error: [" + e.getMessage() + "]"); + showErrorDialog(actionDesc, e.getMessage()); } } |