summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorShireesh Anjal <shireesh@gluster.com>2011-08-03 15:20:35 +0530
committerShireesh Anjal <shireesh@gluster.com>2011-08-03 15:21:10 +0530
commit887157335306976c8926249251c910d6af95c25c (patch)
treeabcf8ef85654af63b46aaaf7b41a80e29dcee0da
parentc08fd957897374d22c1f2d3e87ecbe086487f3d6 (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.java9
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);
}
}
}