Update OVS to 2.17.0

[antoninbas]

There is no update that significantly impacts Antrea, now that we no
longer rely on the userspace datapath.

We also apply a patch that is available in the 'master' branch, but
hasn't been released yet. The patch fixes #2003.

Signed-off-by: Antonin Bas abas@vmware.com

[codecov-commenter]

Codecov Report

Merging #3591 (dad6d85) into main (7be763e) will decrease coverage by 13.84%.
The diff coverage is 53.30%.

❗ Current head dad6d85 differs from pull request most recent head 7237044. Consider uploading reports for the commit 7237044 to get more accurate results

@@             Coverage Diff             @@
##             main    #3591       +/-   ##
===========================================
- Coverage   64.16%   50.32%   -13.85%     
===========================================
  Files         278      248       -30     
  Lines       27825    35657     +7832     
===========================================
+ Hits        17854    17944       +90     
- Misses       8063    15920     +7857     
+ Partials     1908     1793      -115     

Flag	Coverage Δ
e2e-tests	50.32% <53.30%> (?)
kind-e2e-tests	?
unit-tests	?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
pkg/agent/cniserver/pod_configuration.go	42.77% <0.00%> (-11.05%)	⬇️
pkg/agent/openflow/pipeline_other.go	4.00% <0.00%> (-1.13%)	⬇️
pkg/features/antrea_features.go	11.11% <ø> (+2.02%)	⬆️
pkg/agent/proxy/proxier.go	47.64% <3.70%> (-13.56%)	⬇️
pkg/agent/cniserver/ipam/antrea_ipam_controller.go	32.87% <46.96%> (-47.13%)	⬇️
pkg/ipam/poolallocator/allocator.go	30.36% <50.00%> (-19.27%)	⬇️
pkg/agent/cniserver/ipam/antrea_ipam.go	46.82% <50.96%> (-32.96%)	⬇️
pkg/agent/cniserver/server.go	41.41% <51.57%> (-24.36%)	⬇️
pkg/agent/controller/networkpolicy/reject.go	76.47% <55.00%> (-11.44%)	⬇️
pkg/agent/openflow/pipeline.go	69.35% <56.96%> (-4.77%)	⬇️
... and 284 more

[antoninbas]

/test-all

[wenyingd]

Hi, do you think we should also bump up Windows OVS version? @antoninbas @jianjuns @tnqn

[antoninbas]

@wenyingd that would be good. If you have new binaries, you can upload them to our S3 bucket and submit a PR.

[antoninbas]

/test-e2e

[antoninbas]

/test-e2e
/test-integration

[hongliangl]

There is no update that significantly impacts Antrea, now that we no
longer rely on the userspace datapath.

Can we remove the code of userspace in openflow now like in this link @antoninbas

[antoninbas]

There is no update that significantly impacts Antrea, now that we no
longer rely on the userspace datapath.

Can we remove the code of userspace in openflow now like in this link @antoninbas

I don't have a preference. We don't use that code at the moment, so we could remove it. There have been talks in the past of supporting ovs-dpdk for the primary Pod network, which would require that code. But if this support is needed, we can always add the code back. It is likely that since we no longer test with the userspace datapath anyway, this code path is already broken (or will be soon), so it may be a good idea to just remove it for now and simplify the code base.