Change in ovirt-engine[master]: gluster: Update volumes by id instead of name

[sanjal at redhat.com]

Shireesh Anjal has submitted this change and it was merged.

Change subject: gluster: Update volumes by id instead of name
......................................................................


gluster: Update volumes by id instead of name

Modifying the logic to compare fetched volumes with existing volumes
to use volume ids instead of name. This handles the rare scenario
where someone deletes a volume and creates a new one with exactly
same name from the gluster CLI, in quick succession.

Change-Id: If9053da4294b7c5aa762645737a2effccd5a9c4d
Bug-Url: https://bugzilla.redhat.com/888144
Signed-off-by: Shireesh Anjal <sanjal at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GlusterManager.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GlusterManagerTest.java
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/gluster/GlusterVolumesListReturnForXmlRpc.java
3 files changed, 36 insertions(+), 31 deletions(-)

Approvals:
  Shireesh Anjal: Verified; Looks good to me, approved


--
To view, visit http://gerrit.ovirt.org/10429
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If9053da4294b7c5aa762645737a2effccd5a9c4d
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shireesh Anjal <sanjal at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Shireesh Anjal <sanjal at redhat.com>