Bala.FA has posted comments on this change.
Change subject: gluster: Setup and verify ssl connection between nodes.
......................................................................
Patch Set 1: Code-Review-1
(1 comment)
....................................................
File vdsm/gluster/api.py
Line 404: except paramiko.AuthenticationException, e:
Line 405: raise ge.GlusterSshHostAuthException(err=[str(e)])
Line 406:
Line 407: self._validateSshConnection(hostname, fingerprint, username)
Line 408:
Can we move this stuff to new file (may be ssh.py) to expand the usage of these functions
output of vdsm verbs?
Line 409:
Line 410: def getGlusterMethods(gluster):
Line 411: l = []
Line 412: for name in dir(gluster):
--
To view, visit
http://gerrit.ovirt.org/18355
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6f040b1343998de4f8e28419c63e380240368db
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Timothy Asir <tjeyasin(a)redhat.com>
Gerrit-Reviewer: Aravinda VK <avishwan(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: ndarshan <dnarayan(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes