Sahina Bose has posted comments on this change.
Change subject: gluster:Support Striped_replicate volume types
......................................................................
Patch Set 2: (3 inline comments)
Will update patchset based on Shubhendu's comments
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/AddBricksToGlusterVolumeCommand.java
Line 41: if (getParameters().getBricks() == null ||
getParameters().getBricks().size() == 0) {
Line 42:
addCanDoActionMessage(VdcBllMessages.ACTION_TYPE_FAILED_BRICKS_REQUIRED);
Line 43: return false;
Line 44: }
Line 45: if (getGlusterVolume().getVolumeType() ==
GlusterVolumeType.REPLICATE
Thanks, Shubhendu. Instead, I think I will create methods in GlusterVolumeType
to determine if Striped or Replicate.
Line 46: || getGlusterVolume().getVolumeType() ==
GlusterVolumeType.DISTRIBUTED_REPLICATE
Line 47: || getGlusterVolume().getVolumeType() ==
GlusterVolumeType.STRIPED_REPLICATE
Line 48: || getGlusterVolume().getVolumeType() ==
GlusterVolumeType.DISTRIBUTED_STRIPED_REPLICATE) {
Line 49: if (getParameters().getReplicaCount() >
getGlusterVolume().getReplicaCount() + 1) {
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java
Line 416: return;
Line 417: }
Line 418:
Line 419: GlusterVolumeType selectedVolumeType = (GlusterVolumeType)
getTypeList().getSelectedItem();
Line 420: if (selectedVolumeType == GlusterVolumeType.REPLICATE
Will change to use the enum methods to check if replicated/striped
Line 421: || selectedVolumeType ==
GlusterVolumeType.DISTRIBUTED_REPLICATE)
Line 422: {
Line 423:
getReplicaCount().setEntity(volumeBrickModel.getReplicaCount().getEntity());
Line 424: }
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java
Line 585: volume.setClusterId(clusterId);
Line 586: volume.setName((String) volumeModel.getName().getEntity());
Line 587: GlusterVolumeType type = (GlusterVolumeType)
volumeModel.getTypeList().getSelectedItem();
Line 588:
Line 589: if (type == GlusterVolumeType.STRIPE || type ==
GlusterVolumeType.DISTRIBUTED_STRIPE) {
Will change.
Line 590: volume.setStripeCount(volumeModel.getStripeCountValue());
Line 591: } else if (type == GlusterVolumeType.REPLICATE || type ==
GlusterVolumeType.DISTRIBUTED_REPLICATE) {
Line 592:
volume.setReplicaCount(volumeModel.getReplicaCountValue());
Line 593: } else if ( type == GlusterVolumeType.STRIPED_REPLICATE
--
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: 2
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]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
