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 <dgo...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Dhandapani Gopal <dgo...@redhat.com> Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com> Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com> Gerrit-Reviewer: Selvasundaram <sesub...@redhat.com> Gerrit-Reviewer: Shireesh Anjal <san...@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
