Kanagaraj M has submitted this change and it was merged.
Change subject: gluster: During import check host existence by UUID
......................................................................
gluster: During import check host existence by UUID
Modified to check the existence of hosts in the cluster by UUID, while
impporting the hosts into console which got added from CLI.
This makes sure no self entries get listed because of virbr0 bridge and
also resolves the hanging issue while importing the hosts added from
CLI.
Also added logic to discard a host ifrom the list if there is no fingerprint
available for the same.
Change-Id: I60f5fdf16c6b2cf717a041b34f9e62e0ac6b78f3
Bug-Url:
https://bugzilla.redhat.com/1021773
Signed-off-by: Shubhendu Tripathi <shtripat(a)redhat.com>
---
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetAddedGlusterServersQuery.java
M
backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/gluster/GetAddedGlusterServersQueryTest.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/gluster/GlusterDBUtils.java
3 files changed, 90 insertions(+), 17 deletions(-)
Approvals:
Kanagaraj M: Verified; Looks good to me, approved
--
To view, visit
http://gerrit.ovirt.org/20697
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I60f5fdf16c6b2cf717a041b34f9e62e0ac6b78f3
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa(a)redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server