[X/B/D,linux-kvm,SRU,0/1] UBUNTU: [Config] Enable VXLAN module support
mbox series

Message ID 20190819061559.9851-1-po-hsu.lin@canonical.com
Headers show
Series
  • UBUNTU: [Config] Enable VXLAN module support
Related show

Message

Po-Hsu Lin Aug. 19, 2019, 6:15 a.m. UTC
== SRU Justification ==
VXLAN is used by Flannel and maybe other CNI plugins for Kubernetes,
it will be great to enable the support in KVM kernels.

== Fix ==
Enable the CONFIG_VXLAN in the config file.
CONFIG_NET_UDP_TUNNEL=m and other configs were added automatically with
updateconfigs.

CONFIG_OPENVSWITCH_VXLAN supprt was excluded intentionally as this is
not requested (and to reduce the binary size).

== Test ==
Test kernels could be found here:
https://people.canonical.com/~phlin/kernel/lp-1811819-vxlan/

Kernel smoke tested.
User has also verified that all the patched kernels are working as
expected.

== Regression Potentail ==
Low, this enables the support for VXLAN module on KVM kernels, we might
see some related bugs in the future. Since we already have this in the
generic kernel so the impact should be small.

Po-Hsu Lin (1):
  UBUNTU: [Config] Enable VXLAN module support

 debian.kvm/config/config.common.ubuntu | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Stefan Bader Aug. 26, 2019, 2:32 p.m. UTC | #1
On 19.08.19 08:15, Po-Hsu Lin wrote:
> == SRU Justification ==
> VXLAN is used by Flannel and maybe other CNI plugins for Kubernetes,
> it will be great to enable the support in KVM kernels.
> 
> == Fix ==
> Enable the CONFIG_VXLAN in the config file.
> CONFIG_NET_UDP_TUNNEL=m and other configs were added automatically with
> updateconfigs.
> 
> CONFIG_OPENVSWITCH_VXLAN supprt was excluded intentionally as this is
> not requested (and to reduce the binary size).
> 
> == Test ==
> Test kernels could be found here:
> https://people.canonical.com/~phlin/kernel/lp-1811819-vxlan/
> 
> Kernel smoke tested.
> User has also verified that all the patched kernels are working as
> expected.
> 
> == Regression Potentail ==
> Low, this enables the support for VXLAN module on KVM kernels, we might
> see some related bugs in the future. Since we already have this in the
> generic kernel so the impact should be small.
> 
> Po-Hsu Lin (1):
>   UBUNTU: [Config] Enable VXLAN module support
> 
>  debian.kvm/config/config.common.ubuntu | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
This probably would be a case where some update to the annotations of at least
disco/kvm would be good to make sure this will be not lost moving to eoan. But I
see it does not seem to get used...

Acked-by: Stefan Bader <stefan.bader@canonical.com>
Connor Kuehl Aug. 27, 2019, 2:45 p.m. UTC | #2
On 8/18/19 11:15 PM, Po-Hsu Lin wrote:
> == SRU Justification ==
> VXLAN is used by Flannel and maybe other CNI plugins for Kubernetes,
> it will be great to enable the support in KVM kernels.
> 
> == Fix ==
> Enable the CONFIG_VXLAN in the config file.
> CONFIG_NET_UDP_TUNNEL=m and other configs were added automatically with
> updateconfigs.
> 
> CONFIG_OPENVSWITCH_VXLAN supprt was excluded intentionally as this is
> not requested (and to reduce the binary size).
> 
> == Test ==
> Test kernels could be found here:
> https://people.canonical.com/~phlin/kernel/lp-1811819-vxlan/
> 
> Kernel smoke tested.
> User has also verified that all the patched kernels are working as
> expected.
> 
> == Regression Potentail ==
> Low, this enables the support for VXLAN module on KVM kernels, we might
> see some related bugs in the future. Since we already have this in the
> generic kernel so the impact should be small.
> 
> Po-Hsu Lin (1):
>    UBUNTU: [Config] Enable VXLAN module support
> 
>   debian.kvm/config/config.common.ubuntu | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
> 

Acked-by: Connor Kuehl <connor.kuehl@canonical.com>
Kleber Souza Sept. 3, 2019, 2:58 p.m. UTC | #3
On 8/19/19 8:15 AM, Po-Hsu Lin wrote:
> == SRU Justification ==
> VXLAN is used by Flannel and maybe other CNI plugins for Kubernetes,
> it will be great to enable the support in KVM kernels.
> 
> == Fix ==
> Enable the CONFIG_VXLAN in the config file.
> CONFIG_NET_UDP_TUNNEL=m and other configs were added automatically with
> updateconfigs.
> 
> CONFIG_OPENVSWITCH_VXLAN supprt was excluded intentionally as this is
> not requested (and to reduce the binary size).
> 
> == Test ==
> Test kernels could be found here:
> https://people.canonical.com/~phlin/kernel/lp-1811819-vxlan/
> 
> Kernel smoke tested.
> User has also verified that all the patched kernels are working as
> expected.
> 
> == Regression Potentail ==
> Low, this enables the support for VXLAN module on KVM kernels, we might
> see some related bugs in the future. Since we already have this in the
> generic kernel so the impact should be small.
> 
> Po-Hsu Lin (1):
>   UBUNTU: [Config] Enable VXLAN module support
> 
>  debian.kvm/config/config.common.ubuntu | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 

Applied to xenial, bionic and disco linux-kvm/master-next branches.

Thanks,
Kleber