diff options
| author | Kotresh HR <khiremat@redhat.com> | 2019-10-16 14:25:47 +0530 |
|---|---|---|
| committer | Kotresh HR <khiremat@redhat.com> | 2019-11-06 22:29:14 +0530 |
| commit | fa1d2b24c5e4f0f5af8b70065c4195495ac5fd02 (patch) | |
| tree | 3f8791c06299b88b1af21521b672664f3ce89fc6 /geo-replication | |
| parent | 3cec3924e46bb4860bccd38bb8c8464f8c36351f (diff) | |
geo-rep: Fix config upgrade on non-participating node
After upgrade, if the config files are of old format, it
gets migrated to new format. Monitor process migrates it.
Since monitor doesn't run on nodes where bricks are not
hosted, it doesn't get migrated there. So this patch fixes
the config upgrade on nodes which doesn't host bricks.
This happens during config either on get/set/reset.
Backport of:
Patch: https://review.gluster.org/23555
Change-Id: Ibade2f2310b0f3affea21a3baa1ae0eb71162cba
Signed-off-by: Kotresh HR <khiremat@redhat.com>
BUG: 1762220
Change-Id: Ibade2f2310b0f3affea21a3baa1ae0eb71162cba
fixes: bz#1764028
Signed-off-by: Kotresh HR <khiremat@redhat.com>
Diffstat (limited to 'geo-replication')
| -rw-r--r-- | geo-replication/syncdaemon/gsyncd.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/geo-replication/syncdaemon/gsyncd.py b/geo-replication/syncdaemon/gsyncd.py index 6ae5269542f..7b48d82ee97 100644 --- a/geo-replication/syncdaemon/gsyncd.py +++ b/geo-replication/syncdaemon/gsyncd.py @@ -255,7 +255,8 @@ def main(): if args.subcmd == "slave": override_from_args = True - if args.subcmd == "monitor": + if config_file is not None and \ + args.subcmd in ["monitor", "config-get", "config-set", "config-reset"]: ret = gconf.is_config_file_old(config_file, args.master, extra_tmpl_args["slavevol"]) if ret is not None: gconf.config_upgrade(config_file, ret) |