<div dir="ltr"><div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Nov 18, 2019 at 1:51 AM Nir Soffer <<a href="mailto:nsoffer@redhat.com">nsoffer@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Daniel, please review this one.<br>
<br>
On Mon, Nov 18, 2019 at 1:04 AM Nir Soffer <<a href="mailto:nirsof@gmail.com" target="_blank">nirsof@gmail.com</a>> wrote:<br>
><br>
> Checking status more frequently save a couple of seconds. Here is<br>
> an example flow tested with oVirt upload_disk.py example:<br>
><br>
> With 5 seconds wait:<br>
><br>
> Created disk in 11.085111 seconds<br>
> Created transfer in 1.857502 seconds<br>
><br>
> With 1 second wait:<br>
><br>
> Created disk in 4.991227 seconds<br>
> Created transfer in 1.961243 seconds<br>
> ---<br>
>  v2v/rhv-upload-plugin.py | 4 ++--<br>
>  1 file changed, 2 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/v2v/rhv-upload-plugin.py b/v2v/rhv-upload-plugin.py<br>
> index fdd2012f5..472d483f2 100644<br>
> --- a/v2v/rhv-upload-plugin.py<br>
> +++ b/v2v/rhv-upload-plugin.py<br>
> @@ -161,7 +161,7 @@ def open(readonly):<br>
><br>
>      endt = time.time() + timeout<br>
>      while True:<br>
> -        time.sleep(5)<br>
> +        time.sleep(1)<br>
>          disk = disk_service.get()<br>
>          if disk.status == types.DiskStatus.OK:<br>
>              break<br>
> @@ -197,7 +197,7 @@ def open(readonly):<br>
>              transfer_service.cancel()<br>
>              raise RuntimeError("timed out waiting for transfer status "<br>
>                                 "!= INITIALIZING")<br>
> -        time.sleep(5)<br>
> +        time.sleep(1)<br>
><br>
>      # Now we have permission to start the transfer.<br>
>      if params['rhv_direct']:<br>
> --<br>
> 2.21.0<br>
><br>
<br></blockquote><div><br></div><div>looks good, ack.</div></div></div>