Hi Roberto,

   Yes absolutely this is a better choice !! thanks for the tips, I
   will change the fix

regards
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-de...@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel

Reply via email to