[389-devel] Please review lib389 ticket 47584 (take #2): CI tests: add backup/restore of an instance

Roberto Polli rpolli at babel.it
Wed Nov 6 17:35:46 UTC 2013


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

-- 
Roberto Polli
Community Manager
Babel S.r.l. - http://www.babel.it
T: +39.06.9826.9651 M: +39.340.652.2736 F: +39.06.9826.9680
P.zza S.Benedetto da Norcia, 33 - 00040 Pomezia (Roma)

CONFIDENZIALE: Questo messaggio ed i suoi allegati sono di carattere 
confidenziale per i destinatari in indirizzo.
E' vietato l'inoltro non autorizzato a destinatari diversi da quelli indicati 
nel messaggio originale.
Se ricevuto per errore, l'uso del contenuto e' proibito; si prega di 
comunicarlo al mittente e cancellarlo immediatamente.


More information about the 389-devel mailing list