diff options
| author | Kotresh HR <khiremat@redhat.com> | 2018-09-14 03:42:26 -0400 |
|---|---|---|
| committer | Amar Tumballi <amarts@redhat.com> | 2018-09-18 04:27:59 +0000 |
| commit | b977b44dd0adfcd7a3b432844260de4b8d1c4adf (patch) | |
| tree | 987b9fd83f993ebf8bd1c1bded7907b0733c7f23 /xlators/mgmt/glusterd/src/glusterd-utils.c | |
| parent | 484f417da945cf83afdbf136bb4817311862a8d2 (diff) | |
geo-rep: Fix issues related config set
1. '--ignore-mising-args' option for rsync is not
being used even though the rsync version is
greater than 3.1.0. Fixed the same.
2. '--existing' option for rsync is also not being
used. Fixed the same.
3. geo-rep config fails to set rsync-options as the
value contains '--'. Interestingly, python argsparse
treats the value with '--' (e.g., --ignore-missing-args)
as option. But when passed with something like
--value=--ignore-missing-args, it succeeds. Fixed the
same.
Change-Id: Iaeb838acaff1c2920fee9c7f920c99edce13a0a1
Signed-off-by: Kotresh HR <khiremat@redhat.com>
fixes: bz#1629561
Diffstat (limited to 'xlators/mgmt/glusterd/src/glusterd-utils.c')
| -rw-r--r-- | xlators/mgmt/glusterd/src/glusterd-utils.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xlators/mgmt/glusterd/src/glusterd-utils.c b/xlators/mgmt/glusterd/src/glusterd-utils.c index d1efce4aef1..1214b6cbb19 100644 --- a/xlators/mgmt/glusterd/src/glusterd-utils.c +++ b/xlators/mgmt/glusterd/src/glusterd-utils.c @@ -8151,8 +8151,8 @@ glusterd_start_gsync(glusterd_volinfo_t *master_vol, char *slave, runner_add_args(&runner, GSYNCD_PREFIX "/gsyncd", path_list, "-c", NULL); runner_argprintf(&runner, "%s", conf_path); runner_argprintf(&runner, ":%s", master_vol->volname); - runner_add_args(&runner, slave, "--config-set", "session-owner", uuid_str, - NULL); + runner_add_args(&runner, slave, "--config-set", "session-owner", NULL); + runner_argprintf(&runner, "--value=%s", uuid_str); synclock_unlock(&priv->big_lock); ret = runner_run(&runner); synclock_lock(&priv->big_lock); |