Hi All, Please review the patch.
Fixes : https://fedorahosted.org/freeipa/ticket/6031 -- Thanks, Abhijeet Kasurde IRC: akasurde http://akasurde.github.io
From 5cbf144f0157146ef73185b029259f7cdeffb5b9 Mon Sep 17 00:00:00 2001 From: Abhijeet Kasurde <akasu...@redhat.com> Date: Thu, 7 Jul 2016 14:22:50 +0530 Subject: [PATCH] Added fix for correct IPA backup file name Fixes: https://fedorahosted.org/freeipa/ticket/6031 Signed-off-by: Abhijeet Kasurde <akasu...@redhat.com> --- ipaserver/install/ipa_backup.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ipaserver/install/ipa_backup.py b/ipaserver/install/ipa_backup.py index 18a60ecd13c7e7f5381b61ec70ea308a1931b7ec..75c5462ac9a2aa3459382b85e9a97e12de8a4257 100644 --- a/ipaserver/install/ipa_backup.py +++ b/ipaserver/install/ipa_backup.py @@ -289,7 +289,7 @@ class Backup(admintool.AdminTool): self.log.info('Stopping Directory Server') dirsrv.stop(capture_output=False) else: - self.log.info('Stopping IPA services') + self.log.info('Stopping IPA service') run(['ipactl', 'stop']) instance = installutils.realm_to_serverid(api.env.realm) @@ -578,10 +578,10 @@ class Backup(admintool.AdminTool): if data_only: backup_dir = os.path.join(paths.IPA_BACKUP_DIR, time.strftime('ipa-data-%Y-%m-%d-%H-%M-%S')) - filename = os.path.join(backup_dir, "ipa-data.tar") + filename = os.path.join(backup_dir, "ipa-data.tar.gz") else: backup_dir = os.path.join(paths.IPA_BACKUP_DIR, time.strftime('ipa-full-%Y-%m-%d-%H-%M-%S')) - filename = os.path.join(backup_dir, "ipa-full.tar") + filename = os.path.join(backup_dir, "ipa-full.tar.gz") os.mkdir(backup_dir) os.chmod(backup_dir, 0o700) -- 2.4.11
-- Manage your subscription for the Freeipa-devel mailing list: https://www.redhat.com/mailman/listinfo/freeipa-devel Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code
