Sahina Bose has posted comments on this change.
Change subject: gluster:Support Striped_replicate volume types
......................................................................
Patch Set 6: (3 inline comments)
Uploaded patchset with Kanagaraj's comments
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/AddBricksToGlusterVolumeCommand.java
Line 163: && getParameters().getReplicaCount() >
getGlusterVolume().getReplicaCount()) {
Line 164: return true;
Line 165: } else {
Line 166: return false;
Line 167: }
Done
Line 168: }
Line 169:
Line 170: private boolean isStripeCountIncreased() {
Line 171: if ((getGlusterVolume().getVolumeType().isStripedType())
Line 172: && getParameters().getStripeCount() >
getGlusterVolume().getStripeCount()) {
Line 173: return true;
Line 174: } else {
Line 175: return false;
Line 176: }
Done
Line 177: }
Line 178:
Line 179: private GlusterStatus getBrickStatus() {
Line 180: return getGlusterVolume().getStatus();
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java
Line 581: int replicaCount = removeBrickModel.getReplicaCount();
Line 582:
Line 583: if (selectedBricks.size() != stripeCount * replicaCount) {
Line 584: return false;
Line 585: }
Yes, I think it should be possible. Will change
Line 586:
Line 587: return true;
Line 588: }
Line 589:
--
To view, visit http://gerrit.ovirt.org/15381
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I517c1e9e41f1bb68626d5a8a45d679208934d98f
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <[email protected]>
Gerrit-Reviewer: Kanagaraj M <[email protected]>
Gerrit-Reviewer: Michael Pasternak <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: Shubhendu Tripathi <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
