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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Kanagaraj M <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Shireesh Anjal <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
