From 35b2c77c6810d3c3eeffcfffeb8f99351dd8dd7a Mon Sep 17 00:00:00 2001 From: Dhandapani Date: Thu, 21 Jul 2011 17:09:02 +0530 Subject: Bug 3203 - GUI allows removing single brick from a distributed-replicated volume & tells its succeeded --- .../storage/management/server/resources/v1_0/VolumesResource.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/com.gluster.storage.management.server/src/com/gluster/storage/management/server/resources/v1_0/VolumesResource.java b/src/com.gluster.storage.management.server/src/com/gluster/storage/management/server/resources/v1_0/VolumesResource.java index a00ed974..fb3923fc 100644 --- a/src/com.gluster.storage.management.server/src/com/gluster/storage/management/server/resources/v1_0/VolumesResource.java +++ b/src/com.gluster.storage.management.server/src/com/gluster/storage/management/server/resources/v1_0/VolumesResource.java @@ -475,15 +475,19 @@ public class VolumesResource extends AbstractResource { } public void removeBricks(String clusterName, String volumeName, List brickList, GlusterServer onlineServer) { + Status status; try { - glusterUtil.removeBricks(volumeName, brickList, onlineServer.getName()); + status = glusterUtil.removeBricks(volumeName, brickList, onlineServer.getName()); } catch (ConnectionException e) { // online server has gone offline! try with a different one. onlineServer = clusterService.getNewOnlineServer(clusterName); if (onlineServer == null) { throw new GlusterRuntimeException("No online servers found in cluster [" + clusterName + "]"); } - glusterUtil.removeBricks(volumeName, brickList, onlineServer.getName()); + status = glusterUtil.removeBricks(volumeName, brickList, onlineServer.getName()); + } + if (!status.isSuccess()) { + throw new GlusterRuntimeException(status.toString()); } } -- cgit