diff options
author | Shireesh Anjal <shireesh@gluster.com> | 2011-08-03 15:20:35 +0530 |
---|---|---|
committer | Shireesh Anjal <shireesh@gluster.com> | 2011-08-03 15:21:10 +0530 |
commit | 887157335306976c8926249251c910d6af95c25c (patch) | |
tree | abcf8ef85654af63b46aaaf7b41a80e29dcee0da | |
parent | c08fd957897374d22c1f2d3e87ecbe086487f3d6 (diff) |
Fixed Bug 3318 - Deleting multiple volumes simultaneously throws unmarshalling errors
-rw-r--r-- | src/com.gluster.storage.management.gateway/src/com/gluster/storage/management/gateway/resources/v1_0/VolumesResource.java | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/src/com.gluster.storage.management.gateway/src/com/gluster/storage/management/gateway/resources/v1_0/VolumesResource.java b/src/com.gluster.storage.management.gateway/src/com/gluster/storage/management/gateway/resources/v1_0/VolumesResource.java index 8c22c67b..31dff630 100644 --- a/src/com.gluster.storage.management.gateway/src/com/gluster/storage/management/gateway/resources/v1_0/VolumesResource.java +++ b/src/com.gluster.storage.management.gateway/src/com/gluster/storage/management/gateway/resources/v1_0/VolumesResource.java @@ -694,17 +694,16 @@ public class VolumesResource extends AbstractResource { } private void postDelete(String volumeName, List<Brick> bricks, boolean deleteFlag) { - Status result; for (Brick brick : bricks) { String brickDirectory = brick.getBrickDirectory(); // String mountPoint = brickDirectory.substring(0, brickDirectory.lastIndexOf("/")); - result = (Status) serverUtil.executeScriptOnServer(true, brick.getServerName(), + Object output = serverUtil.executeScriptOnServer(true, brick.getServerName(), VOLUME_DIRECTORY_CLEANUP_SCRIPT + " " + brickDirectory + " " + (deleteFlag ? "-d" : ""), - Status.class); - if (!result.isSuccess()) { + String.class); + if (output instanceof Status) { throw new GlusterRuntimeException("Error in post-delete operation of volume [" + volumeName + "]: " - + result); + + output); } } } |