[ovs-dev] [v4 07/12] test/sytem-dpdk: Add unit test for mfex autovalidator
fbl at sysclose.org
Tue Jun 29 14:21:29 UTC 2021
On Tue, Jun 29, 2021 at 03:50:22PM +0200, Eelco Chaudron wrote:
> On 28 Jun 2021, at 4:57, Flavio Leitner wrote:
> > Hi,
> > On Thu, Jun 17, 2021 at 09:57:49PM +0530, Kumar Amber wrote:
> >> Tests:
> >> 6: OVS-DPDK - MFEX Autovalidator
> >> 7: OVS-DPDK - MFEX Autovalidator Fuzzy
> >> Added a new directory to store the PCAP file used
> >> in the tests and a script to generate the fuzzy traffic
> >> type pcap to be used in fuzzy unit test.
> > I haven't tried this yet but am I right that these tests are
> > going to pass a pcap to send traffic in a busy loop for 5
> > seconds in the first case and 20 seconds in the second case?
> > I see that when autovalidator is set OVS will crash if one
> > implementation returns a different value, so I wonder why
> > we need to run for that long.
> I think we should remove the assert (already suggested by Harry),
> so it will not crass by accident if someone selects autovalidator
> in the field (and runs into an issue).
> Failure will then be detected by the ERROR log entries on shutdown.
That's true for the testsuite, but not in production as there is
nothing to disable that.
Perhaps if autovalidator detects an issue, it should log an ERROR
level log to report to testsuite, disable the failing mode and make
sure OVS is either in default or in another functional mode.
> I’m wondering if there is another way than a simple delay, as these tend to cause issues later on. Can we check packets processed or something?
Yeah, maybe we can pass all packets like 5x at least.
> > It is storing a python tool in the pcap directory. I think the
> > fuzzy tool could be called 'mfex_fuzzy.py' and stay in tests/
> > with other similar testing tools.
> > Also, I don't think the test environment sets OVS_DIR. The
> > 'tests/' is actually $srcdir, but I could be wrong here.
> > BTW, scapy is not mandatory to build or test OVS, so if that
> > tool is not available, the test should be skipped and not fail.
> > Thanks,
> > fbl
More information about the dev