Sahina Bose has posted comments on this change.
Change subject: gluster:Support Striped_replicate volume types
......................................................................
Patch Set 4: (7 inline comments)
Uploading patchset with additional validations for the new volume types
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/CreateGlusterVolumeCommand.java
Line 86: addValidationGroup(CreateReplicatedVolume.class);
Line 87: break;
Line 88: default:
Line 89: addValidationGroup(CreateEntity.class);
Line 90: }
Done
Line 91: return super.getValidationGroups();
Line 92: }
Line 93:
Line 94: @Override
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeEntity.java
Line 60: @NotNull(message =
"VALIDATION.GLUSTER.VOLUME.STRIPE_COUNT.NOT_NULL", groups = {
CreateStripedVolume.class })
Line 61: private Integer stripeCount;
Line 62:
Line 63: @Valid
Line 64: private final Map<String, GlusterVolumeOptionEntity> options = new
LinkedHashMap<String, GlusterVolumeOptionEntity>();
Added automatically by eclipse code formatter. Removed now
Line 65:
Line 66: @NotNull(message = "VALIDATION.GLUSTER.VOLUME.BRICKS.NOT_NULL",
groups = {CreateEntity.class, CreateReplicatedVolume.class,
CreateStripedVolume.class})
Line 67: @Valid
Line 68: private List<GlusterBrickEntity> bricks = new
ArrayList<GlusterBrickEntity>();
....................................................
File
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeType.java
Line 55: }
Line 56:
Line 57: public boolean isReplicatedType() {
Line 58: return value().contains("REPLICATE");
Line 59: }
The Gluster volume type name encapsulates this information. I think adding
fields for this is an overkill. Can revisit if needed.
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java
Line 130:
Line 131: getTypeList().getSelectedItemChangedEvent().addListener(new
IEventListener() {
Line 132:
Line 133: @Override
Line 134: public void eventRaised(Event ev, Object sender,
EventArgs args) {
Done
Line 135: if (getTypeList().getSelectedItem() ==
GlusterVolumeType.REPLICATE
Line 136: || getTypeList().getSelectedItem() ==
GlusterVolumeType.DISTRIBUTED_REPLICATE
Line 137: || getTypeList().getSelectedItem() ==
GlusterVolumeType.STRIPED_REPLICATE
Line 138: || getTypeList().getSelectedItem() ==
GlusterVolumeType.DISTRIBUTED_STRIPED_REPLICATE) {
Line 135: if (getTypeList().getSelectedItem() ==
GlusterVolumeType.REPLICATE
Line 136: || getTypeList().getSelectedItem() ==
GlusterVolumeType.DISTRIBUTED_REPLICATE
Line 137: || getTypeList().getSelectedItem() ==
GlusterVolumeType.STRIPED_REPLICATE
Line 138: || getTypeList().getSelectedItem() ==
GlusterVolumeType.DISTRIBUTED_STRIPED_REPLICATE) {
Line 139: getReplicaCount().setIsAvailable(true);
Done
Line 140: }
Line 141: else {
Line 142: getReplicaCount().setIsAvailable(false);
Line 143: }
Line 144:
Line 145: if (getTypeList().getSelectedItem() ==
GlusterVolumeType.STRIPE
Line 146: || getTypeList().getSelectedItem() ==
GlusterVolumeType.DISTRIBUTED_STRIPE
Line 147: || getTypeList().getSelectedItem() ==
GlusterVolumeType.STRIPED_REPLICATE
Line 148: || getTypeList().getSelectedItem() ==
GlusterVolumeType.DISTRIBUTED_STRIPED_REPLICATE) {
Done
Line 149: getStripeCount().setIsAvailable(true);
Line 150: }
Line 151: else {
Line 152: getStripeCount().setIsAvailable(false);
....................................................
File
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java
Line 601:
volume.setReplicaCount(volumeModel.getReplicaCountValue());
Line 602: break;
Line 603: default:
Line 604: break;
Line 605: }
Done
Line 606:
Line 607: volume.setVolumeType(type);
Line 608:
Line 609: if ((Boolean) volumeModel.getTcpTransportType().getEntity()) {
--
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: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtri...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
