|
Hi Roberto,
Yes absolutely this is a better choice !! thanks for the tips, I will change the fixregards thierry On 11/06/2013 06:35 PM, Roberto Polli wrote: You may consider replacing the whole function checkInstanceBackupFS with the simpler: backup_pattern = os.path.join(backupDir, "backup*.tar.gz") return glob.glob(backup_pattern) It should return an empty list in case of unexistent path On Wednesday 06 November 2013 18:31:40 thierry bordaz wrote:Thanks Roberto and Rich for reviewing this. This second patch takes into account your recommendation to let the logging mechanism to choose rather that to use 'verbose' flag. https://fedorahosted.org/389/attachment/ticket/47584/0002-Ticket-47584-CI-te sts-add-backup-restore-of-an-insta.patch |
-- 389-devel mailing list 389-devel@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/389-devel
