Change in ovirt-engine[master]: webadmin: add arbiter support for gluster volume

Code Review gerrit at ovirt.org
Mon Nov 28 15:54:54 UTC 2016


>From Sahina Bose <sabose at redhat.com>:

Sahina Bose has submitted this change and it was merged.

Change subject: webadmin: add arbiter support for gluster volume
......................................................................


webadmin: add arbiter support for gluster volume

 Adding support for arbiter volume creation. Arbiter Volume
check box will be added in new volume dialog and add brick
dialog and it will be shown when user creates a replica volume.
Arbiter volume should be a replica 3 volume and every third brick
in the brick list will be treated as arbiter brick.

 Arbiter Count should be 1. Hence it is shown as a check box
in UI and converted to 1 or 0 during volume creation.

Change-Id: I7d24653a633b0ccc6b6676af24b985bdd3a0be8a
Signed-off-by: Ramesh Nachimuthu <rnachimu at redhat.com>
---
A frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/GlusterFeaturesUtil.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/volumes/VolumeListModel.java
M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIConstants.properties
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumePopupView.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumePopupView.ui.xml
M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/webadmin/ApplicationConstants.properties
9 files changed, 59 insertions(+), 5 deletions(-)

Approvals:
  Alexander Wels: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Ramesh N: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7d24653a633b0ccc6b6676af24b985bdd3a0be8a
Gerrit-PatchSet: 14
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Jenkins CI RO
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list