Shireesh Anjal has posted comments on this change. Change subject: engine: Get Gluster Servers query ......................................................................
Patch Set 7: Alon, No, the sequence is different. We don't ask for a list of IP addresses from the user. We ask for only one - any server of the existing cluster. From this server, we automatically (using gluster cli) fetch list of all other servers. Since this server may not have been bootstrapped, we need to execute the gluster cli using SSH to fetch the list of servers. This patch is for that purpose. So the only change I would expect is that this query should accept the already verified fingerprint of the server, and validate it before authenticating the user. -- To view, visit http://gerrit.ovirt.org/7584 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic69a9a48bf227c8fa805c8aa9c4f08fa36ea9425 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Dhandapani Gopal <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Dhandapani Gopal <[email protected]> Gerrit-Reviewer: Kanagaraj M <[email protected]> Gerrit-Reviewer: Michael Kublin <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Selvasundaram <[email protected]> Gerrit-Reviewer: Shireesh Anjal <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
