summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorPete Zaitcev <zaitcev@kotori.zaitcev.us>2012-11-25 22:57:06 -0700
committerPeter Portante <peter.portante@redhat.com>2013-04-29 16:35:56 -0400
commitf1b668029a6adec9b00d964e8b1c975ecccae6bc (patch)
tree354f72b5825bf64f8a8bd5c13ceba14d4bb0b154
parent57ebd5ae177614ac18308965328a6a75f66ac7e8 (diff)
Replace "ssh foo gluster" with "gluster --remote-host=foo"
With the current lack of authentication, there is no security improvement with passwordless ssh replaced. However, it is more convenient to install UFO if you do not have to generate ssh keys every time. There is a downside the gluster-server RPM becomes required. BUG: 880050 Change-Id: I04f4ad666ac96713055723bd432131a4db325b4f Signed-off-by: Pete Zaitcev <zaitcev@redhat.com> Reviewed-on: http://review.gluster.org/4231 Reviewed-by: Peter Portante <pportant@redhat.com> Reviewed-by: Jeff Darcy <jdarcy@redhat.com> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com> Reviewed-by: Niels de Vos <ndevos@redhat.com> Tested-by: Anand Avati <avati@redhat.com>
-rw-r--r--ufo/gluster/swift/common/Glusterfs.py8
1 files changed, 4 insertions, 4 deletions
diff --git a/ufo/gluster/swift/common/Glusterfs.py b/ufo/gluster/swift/common/Glusterfs.py
index 2d3273e..f9bb26f 100644
--- a/ufo/gluster/swift/common/Glusterfs.py
+++ b/ufo/gluster/swift/common/Glusterfs.py
@@ -101,7 +101,7 @@ def unmount(full_mount_path):
def _get_export_list():
if REMOTE_CLUSTER:
- cmnd = 'ssh %s gluster volume info' % MOUNT_IP
+ cmnd = 'gluster --remote-host=%s volume info' % MOUNT_IP
else:
cmnd = 'gluster volume info'
@@ -109,10 +109,10 @@ def _get_export_list():
if os.system(cmnd + ' >> /dev/null'):
if REMOTE_CLUSTER:
- logging.error('Getting volume info failed %s, make sure to have '\
- 'passwordless ssh on %s', NAME, MOUNT_IP)
+ logging.error('Getting volume info failed for %s, make sure '\
+ 'gluster --remote-host=%s works', NAME, MOUNT_IP)
else:
- logging.error('Getting volume failed %s', NAME)
+ logging.error('Getting volume info failed for %s', NAME)
else:
fp = os.popen(cmnd)
while True: