Shubhendu Tripathi has uploaded a new change for review. Change subject: gluster: fixed the issue introduced by path 20697 ......................................................................
gluster: fixed the issue introduced by path 20697 Because of the patch http://gerrit.ovirt.org/#/c/20697 the notification regarding added hosts from the CLI was not getting shown. Corrected to use the original logic. Change-Id: Ibc6d586c1c82b0c415a4e898c77b1cedc89e7970 Signed-off-by: Shubhendu Tripathi <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetAddedGlusterServersQuery.java 1 file changed, 1 insertion(+), 4 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/35/21535/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetAddedGlusterServersQuery.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetAddedGlusterServersQuery.java index 87a27ab..e785f36 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetAddedGlusterServersQuery.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/gluster/GetAddedGlusterServersQuery.java @@ -63,10 +63,7 @@ fingerprint = returnValue.getReturnValue().toString(); } } - // Keep server details in the map only for the servers which are reachable - if (fingerprint != null && !(fingerprint.isEmpty())) { - serversAndFingerprint.put(server.getHostnameOrIp(), fingerprint); - } + serversAndFingerprint.put(server.getHostnameOrIp(), fingerprint == null ? "" : fingerprint); } } return serversAndFingerprint; -- To view, visit http://gerrit.ovirt.org/21535 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ibc6d586c1c82b0c415a4e898c77b1cedc89e7970 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shubhendu Tripathi <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
