diff options
| author | Pete Zaitcev <zaitcev@kotori.zaitcev.us> | 2012-11-25 22:57:06 -0700 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-11-29 16:01:00 -0800 |
| commit | 3d83e6c42ee0d43208a7b1ebb0595ee7c43bb066 (patch) | |
| tree | 1c62324ec7a108ea0df2da55491678342416a628 | |
| parent | 01cd59506e8021b4adb7fe8b4877a7e60beb13f6 (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.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/ufo/gluster/swift/common/Glusterfs.py b/ufo/gluster/swift/common/Glusterfs.py index 2d3273ed..f9bb26f8 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: |