From 919e9c54d2d1aff67544c231c8b862164a02013b Mon Sep 17 00:00:00 2001 From: Pavlo Golub Date: Wed, 11 Dec 2019 11:25:41 +0100 Subject: [PATCH] Make `dest` argument default value of `backup()` cross platform (#1324) Fixes #1325 --- features/environment.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/features/environment.py b/features/environment.py index d7ba4e82..3ffca4df 100644 --- a/features/environment.py +++ b/features/environment.py @@ -258,7 +258,7 @@ class PatroniController(AbstractController): def backup_source(self): return 'postgres://{username}:{password}@{host}:{port}/{database}'.format(**self._replication) - def backup(self, dest='data/basebackup'): + def backup(self, dest=os.path.join('data', 'basebackup')): subprocess.call([PatroniPoolController.BACKUP_SCRIPT, '--walmethod=none', '--datadir=' + os.path.join(self._work_directory, dest), '--dbname=' + self.backup_source])