<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 6, 2018 at 11:02 AM Richard W.M. Jones <<a href="mailto:rjones@redhat.com">rjones@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sun, Aug 05, 2018 at 04:37:31PM +0300, Nir Soffer wrote:<br>
> When splitting long strings over multiple lines, we can use string<br>
> literal concatenation instead of +.<br>
> <br>
> See <a href="https://docs.python.org/3/reference/lexical_analysis.html#string-literal-concatenation" rel="noreferrer" target="_blank">https://docs.python.org/3/reference/lexical_analysis.html#string-literal-concatenation</a><br>
> ---<br>
>  .gnulib                  |  2 +-<br>
<br>
I dropped the accidental gnulib part of this commit :-)<br></blockquote><div><br></div><div>Sorry about that :-)</div><div><br></div><div>I think I did:</div><div>git pull</div><div>git checkout -b ...</div><div>edit file</div><div>git commit</div><div>git format-patch master</div><div><br></div><div>Can we prevent submodules changes to sneak into unrelated patches</div><div>without manual work on the developer side?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Rich.<br>
<br>
>  v2v/rhv-upload-plugin.py | 14 +++++++-------<br>
>  2 files changed, 8 insertions(+), 8 deletions(-)<br>
> <br>
> diff --git a/.gnulib b/.gnulib<br>
> index 5b78831df..646a44e1b 160000<br>
> --- a/.gnulib<br>
> +++ b/.gnulib<br>
> @@ -1 +1 @@<br>
> -Subproject commit 5b78831df03b49408676227604cf16f90dee07ac<br>
> +Subproject commit 646a44e1b190c4a7f6a9f32c63230c619e38d251<br>
> diff --git a/v2v/rhv-upload-plugin.py b/v2v/rhv-upload-plugin.py<br>
> index bdc1e104a..2d686c2da 100644<br>
> --- a/v2v/rhv-upload-plugin.py<br>
> +++ b/v2v/rhv-upload-plugin.py<br>
> @@ -95,8 +95,8 @@ def find_host(connection):<br>
>          # - 'hw_id' equals to 'vdsm_id'<br>
>          # - Its status is 'Up'<br>
>          # - Belongs to the storage domain's datacenter<br>
> -        debug("cannot find a running host with hw_id=%r, " +<br>
> -              "that belongs to datacenter '%s', " +<br>
> +        debug("cannot find a running host with hw_id=%r, "<br>
> +              "that belongs to datacenter '%s', "<br>
>                "using any host" % (vdsm_id, <a href="http://datacenter.name" rel="noreferrer" target="_blank">datacenter.name</a>))<br>
>          return None<br>
>  <br>
> @@ -193,15 +193,15 @@ def open(readonly):<br>
>          if transfer.phase != types.ImageTransferPhase.INITIALIZING:<br>
>              break<br>
>          if time.time() > endt:<br>
> -            raise RuntimeError("timed out waiting for transfer status " +<br>
> +            raise RuntimeError("timed out waiting for transfer status "<br>
>                                 "!= INITIALIZING")<br>
>  <br>
>      # Now we have permission to start the transfer.<br>
>      if params['rhv_direct']:<br>
>          if transfer.transfer_url is None:<br>
> -            raise RuntimeError("direct upload to host not supported, " +<br>
> -                               "requires ovirt-engine >= 4.2 and only works " +<br>
> -                               "when virt-v2v is run within the oVirt/RHV " +<br>
> +            raise RuntimeError("direct upload to host not supported, "<br>
> +                               "requires ovirt-engine >= 4.2 and only works "<br>
> +                               "when virt-v2v is run within the oVirt/RHV "<br>
>                                 "environment, eg. on an oVirt node.")<br>
>          destination_url = urlparse(transfer.transfer_url)<br>
>      else:<br>
> @@ -511,7 +511,7 @@ def close(h):<br>
>                  time.sleep(1)<br>
>                  tmp = transfer_service.get()<br>
>                  if time.time() > endt:<br>
> -                    raise RuntimeError("timed out waiting for transfer " +<br>
> +                    raise RuntimeError("timed out waiting for transfer "<br>
>                                         "to finalize")<br>
>          except sdk.NotFoundError:<br>
>              pass<br>
> -- <br>
> 2.17.1<br>
<br>
-- <br>
Richard Jones, Virtualization Group, Red Hat <a href="http://people.redhat.com/~rjones" rel="noreferrer" target="_blank">http://people.redhat.com/~rjones</a><br>
Read my programming and virtualization blog: <a href="http://rwmj.wordpress.com" rel="noreferrer" target="_blank">http://rwmj.wordpress.com</a><br>
virt-builder quickly builds VMs from scratch<br>
<a href="http://libguestfs.org/virt-builder.1.html" rel="noreferrer" target="_blank">http://libguestfs.org/virt-builder.1.html</a><br>
</blockquote></div></div>