<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">Hi Roberto,<br>
<blockquote>Yes absolutely this is a better choice !! thanks for
the tips, I will change the fix<br>
<br>
</blockquote>
regards<br>
thierry<br>
<br>
On 11/06/2013 06:35 PM, Roberto Polli wrote:<br>
</div>
<blockquote cite="mid:2053041.4EPnQ3DNQ9@rpolli" type="cite">
<pre wrap="">
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:
</pre>
<blockquote type="cite">
<pre wrap="">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.
<a class="moz-txt-link-freetext" href="https://fedorahosted.org/389/attachment/ticket/47584/0002-Ticket-47584-CI-te">https://fedorahosted.org/389/attachment/ticket/47584/0002-Ticket-47584-CI-te</a>
sts-add-backup-restore-of-an-insta.patch
</pre>
</blockquote>
<pre wrap="">
</pre>
</blockquote>
<br>
</body>
</html>