5 May
2025
5 May
'25
5:49 p.m.
On Fri, 2 May 2025 22:02:43 -0700 David Wei wrote:
-def test_default(cfg, netnl) -> None: +def test_default_v4(cfg, netnl) -> None:
- cfg.require_ipver("4")
With this patch, is the previous patch and this hunk still needed? I think you can drop patch 2/3 or both.
The previous patch makes the more "meaty" tests run when either ipver is available. But we still want an indication if both versions are supported by env but reporting *a* skip. Historically this is what the ping test was for, TBH, pretty much a sanity check for the env. But it ended up accumulating random XDP test cases :( Perhaps something to address in net-next..