Change in ovirt-engine[master]: gluster: handle unsupported gluster volume types

[sabose at redhat.com]

Sahina Bose has submitted this change and it was merged.

Change subject: gluster: handle unsupported gluster volume types
......................................................................


gluster: handle unsupported gluster volume types

   NPE is seen when a new volume type is introduced in Gluster
and that type of volume is created in Gluster CLI.

  Introducing a new volume types 'UNKNOWN' and 'TIER'. Actions like
'ADD' 'REMOVE' bricks and rebalance will not be supported in these
types of volumes. Also tasks with type 'UNKNOWN' will not be sycned.

Change-Id: Iffc6a93987eeafbba4666e2a864038ca1f2258cd
Bug-Url: https://bugzilla.redhat.com/1294473
Signed-off-by: Ramesh Nachimuthu <rnachimu at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterSyncJob.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/gluster/GlusterVolumeType.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/constants/gluster/GlusterConstants.java
M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterTasksListReturnForXmlRpc.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickListModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeModel.java
M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/Enums.java
M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/Enums.properties
10 files changed, 71 insertions(+), 18 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Kanagaraj M: Looks good to me, but someone else must approve
  Sahina Bose: Looks good to me, approved
  Ramesh N: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iffc6a93987eeafbba4666e2a864038ca1f2258cd
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>