Thomas Monjalon <tho...@monjalon.net> writes: > 15/06/2019 08:42, David Marchand: >> This is a joint effort to make the unit tests ready for CI. > > Applied, thanks > > Remaining work below from your list: > [...] >> - the tests on --file-prefix are still ko, and have been isolated in a >> test that we could disable while waiting for the fixes,
Yes, I think it's good to do that for now. >> - rwlock_autotest and hash_readwrite_lf_autotest are taking a little more >> than 10s, Occasionally the distributor test times out as well. I've moved them as part of a separate patch, that I'll post along with a bigger series to enable the unit tests under travis. Michael and I are leaning toward introducing a new variable called RUN_TESTS which will do the docs and unit testing since those combined would add quite a bit to the execution time of each job (and feel free to bike shed the name, since the patches aren't final). >> - librte_table unit test crashes on ipv6 [2], I guess we're waiting on a patch from Jananee (CC'd)? >> - the "perf" tests are taking way too long for my taste, Agreed, so I think we'll disable the perf tests in the CI environment anyway. >> - the shared build unit tests all fail when depending on mempool since >> the mempool drivers are not loaded, I think Michael is working on a fix for this right now. >> 2: https://bugs.dpdk.org/show_bug.cgi?id=285