diff options
authorCsaba Henk <>2013-02-28 04:18:41 +0100
committerVijay Bellur <>2013-04-27 09:36:58 -0700
commit3490689f29342116cf78cc57bc90ad1979d9e1bb (patch)
parent6113bfe8c9f5528f54b66d31a7bd0f1803ffe092 (diff)
geo-rep: retire old style ssh setup
Users are still using geo-rep with the old, deprecated, insecure, unsupported ssh setup. Not their fault -- the implementation of the new method had the following charasteristics: - old method is possible, but with default settings it's not working - it can be made operational by fiddling with "remote-gsyncd" tunable - with default setting, an unhelpful, actually misleading error message is produced - the UI gave no hint to the changes in the ssh setup tried to fix these; what it accomplished was unrestricted support to the bad practice (by making the default old setup operational). From this on: - we disable the old method by reserving the "remote-gsyncd" tunable - if the old method is attempted, give a hint what to do Change-Id: Icade94725d8d8d2d4c89cab992d4226351637b86 BUG: 895656 Signed-off-by: Csaba Henk <> Reviewed-on: Tested-by: Gluster Build System <> Reviewed-by: Vijay Bellur <>
3 files changed, 9 insertions, 1 deletions
diff --git a/xlators/features/marker/utils/syncdaemon/ b/xlators/features/marker/utils/syncdaemon/
index b29936d596d..53bbcd6f640 100644
--- a/xlators/features/marker/utils/syncdaemon/
+++ b/xlators/features/marker/utils/syncdaemon/
@@ -173,6 +173,12 @@ def log_raise_exception(excont):
logging.error('connection to peer is broken')
if hasattr(gconf, 'transport'):
+ if gconf.transport.returncode == 127:
+ logging.warn("!!!!!!!!!!!!!")
+ logging.warn('!!! getting "No such file or directory" errors '
+ "is most likely due to MISCONFIGURATION, please consult "
+ "")
+ logging.warn("!!!!!!!!!!!!!")
elif isinstance(exc, OSError) and exc.errno in (ENOTCONN, ECONNABORTED):
logging.error('glusterfs session went down [%s]', errorcode[exc.errno])
diff --git a/xlators/mgmt/glusterd/src/glusterd-geo-rep.c b/xlators/mgmt/glusterd/src/glusterd-geo-rep.c
index 847fed49eb8..f46a172d91b 100644
--- a/xlators/mgmt/glusterd/src/glusterd-geo-rep.c
+++ b/xlators/mgmt/glusterd/src/glusterd-geo-rep.c
@@ -38,6 +38,7 @@
static char *gsync_reserved_opts[] = {
+ "remote-gsyncd"
diff --git a/xlators/mgmt/glusterd/src/glusterd.c b/xlators/mgmt/glusterd/src/glusterd.c
index b10b23d579f..3a1b87a2997 100644
--- a/xlators/mgmt/glusterd/src/glusterd.c
+++ b/xlators/mgmt/glusterd/src/glusterd.c
@@ -461,7 +461,8 @@ configure_syncdaemon (glusterd_conf_t *conf)
runinit_gsyncd_setrx (&runner, conf);
- runner_add_args (&runner, "remote-gsyncd", GSYNCD_PREFIX"/gsyncd", ".", "^ssh:", NULL);
+ runner_add_args (&runner, "remote-gsyncd", "/nonexistent/gsyncd",
+ ".", "^ssh:", NULL);
/* gluster-command-dir */