On 12/5/24 14:45, David Marchand wrote:
> On Thu, Dec 5, 2024 at 2:22 PM Ilya Maximets <[email protected]> wrote:
>>
>> On 12/5/24 14:12, David Marchand wrote:
>>> On Wed, Dec 4, 2024 at 9:45 PM Ilya Maximets <[email protected]> wrote:
>>>>
>>>> Tunnel support is in upstream Linux kernel since 2013.  However,
>>>> despite the FAQ saying so, I'm not aware of actual attempts to bring
>>>> support for LISP tunnels upstream.  The only available implementation
>>>> is in OOT kernel module shipped with OVS 2.17.  It is deprecated and
>>>> will reach EoL in Feb 2025.
>>>>
>>>> Mark the tunnel port type as deprecated, so we can fully remove the
>>>> support once the only available implementation reaches end of life
>>>> together with OVS 2.17.
>>>>
>>>> Signed-off-by: Ilya Maximets <[email protected]>
>>>> ---
>>>>  Documentation/faq/releases.rst | 16 +++++-----------
>>>>  Documentation/howto/lisp.rst   |  4 ++++
>>>
>>> I noticed a mention of LISP tunnels in Documentation/intro/install/rhel.rst.
>>> This was introduced with f9ee9dcb378f ("rhel: Remove the firewall hole
>>> that we create for GRE.") though I don't think LISP was ever supported
>>> in RHEL.
>>> This could be removed now, or it can wait for the full removal post 3.5.
>>
>> It can be supported on RHEL if the OOT kernel module is loaded.  And
>> we did support building OOT module for RHEL.  We'll clean this up once
>> the support is fully removed.
> 
> Ok.
> 
>>>
>>> Reading https://github.com/openvswitch/ovs, STT and LISP tunnels are
>>> still advertised as supported.
>>> Flagging in README.rst would make this deprecation more visible.
>>
>> I added deprecation warnings in all places where we tell how to set
>> them up, but I didn't want to pollute simple lists of features like
>> this one.  Especially because 'deprecated' doesn't mean they are not
>> supported.
>>
>> We have a NEWS entry, warnings in documentation about specific features
>> and a runtime warning.  It should be enough, I think.  If someone reads
>> the readme for the first time, they'll likely look at the other parts
>> of the documentation to figure out how to set it up.
> 
> Well, I was trying to look at it from a first user pov.
> Hence the README displayed in github seems more visible than the NEWS file.
> 
> But if you prefer to keep it like this, fine by me.

I think, it's fine as is.  It'll be removed from the readme in about 2
months anyway.

Thanks, Eelco and David!  Applied the set to main.
Will be working on the removal once 3.5 is out.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to