Ramesh N has uploaded a new change for review.

Change subject: webadmin: fix add brick validation issue
......................................................................

webadmin: fix add brick validation issue

  Fix the add brick validation issue for replicate and
stripe volume.

Bug-Url: https://bugzilla.redhat.com/1143828
Signed-off-by: Ramesh Nachimuthu <rnach...@redhat.com>
Change-Id: I005c32f2d030c35ec43149ee4f4aeaa7e731ee3c
---
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/35/39635/1

diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
index 552dded..640e91b 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
@@ -506,14 +506,14 @@
             break;
 
         case DISTRIBUTED_REPLICATE:
-            if (brickCount <= replicaCount || (brickCount % replicaCount) != 0)
+            if ((brickCount % replicaCount) != 0)
             {
                 valid = false;
             }
             break;
 
         case DISTRIBUTED_STRIPE:
-            if (brickCount <= stripeCount || (brickCount % stripeCount) != 0)
+            if ((brickCount % stripeCount) != 0)
             {
                 valid = false;
             }


-- 
To view, visit https://gerrit.ovirt.org/39635
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I005c32f2d030c35ec43149ee4f4aeaa7e731ee3c
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5-gluster
Gerrit-Owner: Ramesh N <rnach...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to