[libvirt] [PATCH] news: document libxl tunnelled migration support

Joao Martins posted 1 patch 7 years, 2 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/1487199384-30276-1-git-send-email-joao.m.martins@oracle.com
docs/news.xml | 10 ++++++++++
1 file changed, 10 insertions(+)
[libvirt] [PATCH] news: document libxl tunnelled migration support
Posted by Joao Martins 7 years, 2 months ago
Signed-off-by: Joao Martins <joao.m.martins@oracle.com>
---
 docs/news.xml | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/docs/news.xml b/docs/news.xml
index b756a97..b0629b5 100644
--- a/docs/news.xml
+++ b/docs/news.xml
@@ -53,6 +53,16 @@
           was <code>virtio-net</code>.
         </description>
       </change>
+      <change>
+        <summary>
+          libxl: add tunnelled migration support
+        </summary>
+        <description>
+          Add tunnelled migration to libxl driver, which is always capable of
+          strong encryption and doesn't require any extra network connection
+          other than what's required for remote access of libvirtd.
+        </description>
+      </change>
     </section>
     <section title="Improvements">
       <change>
-- 
2.1.4

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] news: document libxl tunnelled migration support
Posted by Jim Fehlig 7 years, 2 months ago
Joao Martins wrote:
> Signed-off-by: Joao Martins <joao.m.martins@oracle.com>
> ---
>  docs/news.xml | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/docs/news.xml b/docs/news.xml
> index b756a97..b0629b5 100644
> --- a/docs/news.xml
> +++ b/docs/news.xml
> @@ -53,6 +53,16 @@
>            was <code>virtio-net</code>.
>          </description>
>        </change>
> +      <change>
> +        <summary>
> +          libxl: add tunnelled migration support
> +        </summary>
> +        <description>
> +          Add tunnelled migration to libxl driver, which is always capable of
> +          strong encryption and doesn't require any extra network connection
> +          other than what's required for remote access of libvirtd.
> +        </description>
> +      </change>
>      </section>
>      <section title="Improvements">
>        <change>

Pushed, but only after realizing tunneled is misspelled :-(. No use perpetuating
the misspelling of tunneled, so I've pushed a trivial followup.

Regards,
Jim

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] news: document libxl tunnelled migration support
Posted by Joao Martins 7 years, 2 months ago
On 02/15/2017 11:41 PM, Jim Fehlig wrote:
> Joao Martins wrote:
>> Signed-off-by: Joao Martins <joao.m.martins@oracle.com>
>> ---
>>  docs/news.xml | 10 ++++++++++
>>  1 file changed, 10 insertions(+)
>>
>> diff --git a/docs/news.xml b/docs/news.xml
>> index b756a97..b0629b5 100644
>> --- a/docs/news.xml
>> +++ b/docs/news.xml
>> @@ -53,6 +53,16 @@
>>            was <code>virtio-net</code>.
>>          </description>
>>        </change>
>> +      <change>
>> +        <summary>
>> +          libxl: add tunnelled migration support
>> +        </summary>
>> +        <description>
>> +          Add tunnelled migration to libxl driver, which is always capable of
>> +          strong encryption and doesn't require any extra network connection
>> +          other than what's required for remote access of libvirtd.
>> +        </description>
>> +      </change>
>>      </section>
>>      <section title="Improvements">
>>        <change>
> 
> Pushed, but only after realizing tunneled is misspelled :-(. No use perpetuating
> the misspelling of tunneled, so I've pushed a trivial followup.

Interesting, I didn't know that. Greping the whole repo for "tunnelled" and you
will find a *lot* of matches:

$ git grep tunneled | wc -l
10
$ git grep tunnelled | wc -l
1242

Hmm, but the internets aren't really clear. Some hits say tunneled vs tunnelled
being both correct.

Joao

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] news: document libxl tunnelled migration support
Posted by Jim Fehlig 7 years, 2 months ago
On 02/15/2017 05:06 PM, Joao Martins wrote:
> On 02/15/2017 11:41 PM, Jim Fehlig wrote:
>> Joao Martins wrote:
>>> Signed-off-by: Joao Martins <joao.m.martins@oracle.com>
>>> ---
>>>  docs/news.xml | 10 ++++++++++
>>>  1 file changed, 10 insertions(+)
>>>
>>> diff --git a/docs/news.xml b/docs/news.xml
>>> index b756a97..b0629b5 100644
>>> --- a/docs/news.xml
>>> +++ b/docs/news.xml
>>> @@ -53,6 +53,16 @@
>>>            was <code>virtio-net</code>.
>>>          </description>
>>>        </change>
>>> +      <change>
>>> +        <summary>
>>> +          libxl: add tunnelled migration support
>>> +        </summary>
>>> +        <description>
>>> +          Add tunnelled migration to libxl driver, which is always capable of
>>> +          strong encryption and doesn't require any extra network connection
>>> +          other than what's required for remote access of libvirtd.
>>> +        </description>
>>> +      </change>
>>>      </section>
>>>      <section title="Improvements">
>>>        <change>
>>
>> Pushed, but only after realizing tunneled is misspelled :-(. No use perpetuating
>> the misspelling of tunneled, so I've pushed a trivial followup.
>
> Interesting, I didn't know that. Greping the whole repo for "tunnelled" and you
> will find a *lot* of matches:
>
> $ git grep tunneled | wc -l
> 10
> $ git grep tunnelled | wc -l
> 1242

So the incorrect spelling is used over 100x more than the correct one :-)

> Hmm, but the internets aren't really clear. Some hits say tunneled vs tunnelled
> being both correct.

Heh, as a native speaker I'm not sure which spelling is correct, but seem to 
recall a prior discussion on the list proclaiming 'tunneled'. If folks prefer I 
can revert the s/tunnelled/tunneled/ commit.

Regards,
Jim

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] news: document libxl tunnelled migration support
Posted by Chris Lalancette 7 years, 2 months ago
On Wed, Feb 15, 2017 at 8:24 PM, Jim Fehlig <jfehlig@suse.com> wrote:

>
> Heh, as a native speaker I'm not sure which spelling is correct, but seem
> to recall a prior discussion on the list proclaiming 'tunneled'. If folks
> prefer I can revert the s/tunnelled/tunneled/ commit.
>
>
I think you have me to blame for "tunnelled".  At the time I initially
wrote tunnelled migration support, I did a Google search similar to what
you have done, and determined that tunnelled seemed to be slightly
preferred.  Subsequently almost everyone has balked at it, but I think the
horse has left the barn on it and we should probably stick with "tunnelled".

Chris
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] news: document libxl tunnelled migration support
Posted by Jim Fehlig 7 years, 2 months ago
On 02/15/2017 06:35 PM, Chris Lalancette wrote:
> On Wed, Feb 15, 2017 at 8:24 PM, Jim Fehlig <jfehlig@suse.com
> <mailto:jfehlig@suse.com>> wrote:
>
>
>     Heh, as a native speaker I'm not sure which spelling is correct, but seem to
>     recall a prior discussion on the list proclaiming 'tunneled'. If folks
>     prefer I can revert the s/tunnelled/tunneled/ commit.
>
>
> I think you have me to blame for "tunnelled".  At the time I initially wrote
> tunnelled migration support, I did a Google search similar to what you have
> done, and determined that tunnelled seemed to be slightly preferred.
> Subsequently almost everyone has balked at it, but I think the horse has left
> the barn on it and we should probably stick with "tunnelled".

Ok. I also failed to realize that the commit providing the news item used 
'tunnelled'. I've sent a patch to revert the spelling change, but I'll wait for 
an explicit ACK this time

https://www.redhat.com/archives/libvir-list/2017-February/msg00770.html

Regards,
Jim

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH] news: document libxl tunnelled migration support
Posted by Joao Martins 7 years, 2 months ago
On 02/16/2017 01:24 AM, Jim Fehlig wrote:
> On 02/15/2017 05:06 PM, Joao Martins wrote:
>> On 02/15/2017 11:41 PM, Jim Fehlig wrote:
>>> Joao Martins wrote:
>>>> Signed-off-by: Joao Martins <joao.m.martins@oracle.com>
>>>> ---
>>>>  docs/news.xml | 10 ++++++++++
>>>>  1 file changed, 10 insertions(+)
>>>>
>>>> diff --git a/docs/news.xml b/docs/news.xml
>>>> index b756a97..b0629b5 100644
>>>> --- a/docs/news.xml
>>>> +++ b/docs/news.xml
>>>> @@ -53,6 +53,16 @@
>>>>            was <code>virtio-net</code>.
>>>>          </description>
>>>>        </change>
>>>> +      <change>
>>>> +        <summary>
>>>> +          libxl: add tunnelled migration support
>>>> +        </summary>
>>>> +        <description>
>>>> +          Add tunnelled migration to libxl driver, which is always capable of
>>>> +          strong encryption and doesn't require any extra network connection
>>>> +          other than what's required for remote access of libvirtd.
>>>> +        </description>
>>>> +      </change>
>>>>      </section>
>>>>      <section title="Improvements">
>>>>        <change>
>>>
>>> Pushed, but only after realizing tunneled is misspelled :-(. No use perpetuating
>>> the misspelling of tunneled, so I've pushed a trivial followup.
>>
>> Interesting, I didn't know that. Greping the whole repo for "tunnelled" and you
>> will find a *lot* of matches:
>>
>> $ git grep tunneled | wc -l
>> 10
>> $ git grep tunnelled | wc -l
>> 1242
> 
> So the incorrect spelling is used over 100x more than the correct one :-)

Hehe :D

>> Hmm, but the internets aren't really clear. Some hits say tunneled vs tunnelled
>> being both correct.
> 
> Heh, as a native speaker I'm not sure which spelling is correct, but seem to 
> recall a prior discussion on the list proclaiming 'tunneled'. If folks prefer I 
> can revert the s/tunnelled/tunneled/ commit.

Sorry I may have mis-expressed myself before - didn't meant this being an issue.
I was just curious about the word because I made that same mistake throughout
the patches. Probably there's no need for revert with both appearing correct (as
folks are suggesting in followup messages)

Joao

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list