Merge pull request #274 from Selvasundaram/master

Modified Volume#equals to correctly check for added/removed bricks 

1 files changed, 8 insertions, 0 deletions

diff --git a/src/com.gluster.storage.management.core/src/com/gluster/storage/management/core/model/Volume.java b/src/com.gluster.storage.management.core/src/com/gluster/storage/management/core/model/Volume.java
index 65bec003..9356f14a 100644
--- a/src/com.gluster.storage.management.core/src/com/gluster/storage/management/core/model/Volume.java
+++ b/src/com.gluster.storage.management.core/src/com/gluster/storage/management/core/model/Volume.java
@@ -330,6 +330,14 @@ public class Volume extends Entity {
 		if(oldBricks.size() != newBricks.size()) {
 			return false;
 		}
+		
+		if(!GlusterCoreUtil.getAddedEntities(oldBricks, newBricks, false).isEmpty()) {
+			return false;
+		}
+
+		if(!GlusterCoreUtil.getAddedEntities(newBricks, oldBricks, false).isEmpty()) {
+			return false;
+		}
 
 		Map<Brick, Brick> modifiedBricks = GlusterCoreUtil.getModifiedEntities(oldBricks, newBricks);
 		if(modifiedBricks.size() > 0) {