Change in ovirt-engine[master]: gluster: During import check host existence by UUID

kmayilsa at redhat.com kmayilsa at redhat.com
Thu Nov 21 04:45:54 UTC 2013


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 at 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 at redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayilsa at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtripat at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list