diff options
author | Shwetha K Acharya <sacharya@redhat.com> | 2019-07-02 15:00:25 +0530 |
---|---|---|
committer | Sunny Kumar <sunkumar@redhat.com> | 2019-07-22 06:44:49 +0000 |
commit | 74f124619a71df9bdc5ae9fbc07bc19db05bc1d2 (patch) | |
tree | 62defe3c5ccee4eccf3a83149c1c4534751aae7b /geo-replication | |
parent | 41f72ac9453c24bea85709203948c41c5772edec (diff) |
geo-rep: Test case for upgrading config file
Added test case for the patch
https://review.gluster.org/#/c/glusterfs/+/22894/4
Also updated if else structure in gsyncdconfig.py to avoid
repeated occurance of values in new configfile.
fixes: bz#1707731
Change-Id: If97e1d37ac52dbd17d47be6cb659fc5a3ccab6d7
Signed-off-by: Shwetha K Acharya <sacharya@redhat.com>
Diffstat (limited to 'geo-replication')
-rw-r--r-- | geo-replication/syncdaemon/gsyncdconfig.py | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/geo-replication/syncdaemon/gsyncdconfig.py b/geo-replication/syncdaemon/gsyncdconfig.py index 68d1b489e23..0476d5ec05c 100644 --- a/geo-replication/syncdaemon/gsyncdconfig.py +++ b/geo-replication/syncdaemon/gsyncdconfig.py @@ -356,15 +356,14 @@ def config_upgrade(config_file, ret): new_value = "tarssh" else: new_value = "rsync" - config.set('vars', new_key, new_value) - - if key == "timeout": + config.set('vars', new_key, new_value) + elif key == "timeout": new_key = "slave-timeout" config.set('vars', new_key, value) - #for changes like: ignore_deletes to ignore-deletes - new_key = key.replace("_", "-") - config.set('vars', new_key, value) + else: + new_key = key.replace("_", "-") + config.set('vars', new_key, value) with open(config_file, 'w') as configfile: config.write(configfile) |