On 5/4/26 9:47 PM, Timothy Redaelli wrote:
> These network testing tools are barely used nowadays. Standard tools
> like iperf are preferred for performance testing,
Thanks, Timothy!
> and OVS system tests
> already cover basic connectivity use cases.
I'd remove this point, since system tests are running at the build time
or in CI, while these tools are for the users to run on their systems
where OVS is deployed. So, the use case is different.
> Remove the utilities,
> their man pages, the supporting ovstest Python module, and all build
> system and packaging references.
Please, make it a patch set removing one thing at a time, the changes
are independent enough to be separate patches. Each patch should
update the NEWS accordingly.
>
> ovs-vlan-test is retained as it is an independent self-contained tool.
As per the ovs-vlan-test man page, this tool is a predcessor of ovs-test
and it doesn't make much sense to keep it if we're removing the newer
tool. It is also fairly irrelevant these days as it was designed to
highlight the vlan bugs in 2.6.x kernels. Should remove this one as well.
> diff --git a/NEWS b/NEWS
> index 1a3044cbf..c14f987db 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -1,5 +1,8 @@
> Post-v3.7.0
> --------------------
> + - Removed ovs-test, ovs-l3ping utilities and the ovstest Python module.
> + Standard tools like iperf are preferred for performance testing, and
> + OVS system tests cover basic connectivity use cases.
Same here, the systems tests are not particularly relevant here.
> - Userspace datapath:
> * ARP/ND lookups for native tunnel are now rate limited. The holdout
> timer can be configured with 'tnl/neigh/retrans_time'.
> diff --git a/debian/control.in b/debian/control.in
> index 836e2564a..1189b4bf3 100644
> --- a/debian/control.in
> +++ b/debian/control.in
> @@ -210,15 +210,8 @@ Description: Open vSwitch switch implementations
> Package: openvswitch-test
> Architecture: all
> Depends:
> - python3-twisted,
> ${misc:Depends},
> ${python3:Depends},
> -Breaks:
> - python3-openvswitch (<< 2.17~),
> - openvswitch-common (<< 2.17~),
> -Replaces:
> - python3-openvswitch (<< 2.17~),
> - openvswitch-common (<< 2.17~),
This change seems unrelated. These clauses are here because the packaging
was completely reworked in OVS 3.0 and it is not compatible with the older
packages. Do you have a reason to remove them?
Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
