> > Subject: Re: [dpdk-dev] [PATCH] ci: increase unit test timeout > > > > 28/01/2020 21:53, Aaron Conole: > > > Ferruh Yigit <ferruh.yi...@intel.com> writes: > > > > > > > Timeout multiplier was 3, which gives 30 seconds for unit test but > > > > still some unit test was timing out time to time and travis > > > > reporting false positive failures. > > > > > > > > Increasing the multiplier to 10, which makes timeout duration > > > > 100seconds. > > > > > > > > Signed-off-by: Ferruh Yigit <ferruh.yi...@intel.com> > > > > --- > > > > > > It's okay to me. I thought there was an effort to split out > > > performance part of this test from the functional part, but that > > > seems to not have gone anywhere. > > > > > > Acked-by: Aaron Conole <acon...@redhat.com> > > > > NACK > > The fix should be to split perf tests out of fast-tests. > > > > The following patch is splitting hash_readwrite_autotest: > > https://patchwork.dpdk.org/patch/58726/ Thomas/David, is it possible to merge this patch? It is acked already. It would be good to have these changes for LF functional tests.
> > But we are still waiting for a patch splitting hash_readwrite_lf_autotest. > > Please consider working on unit tests as a HIGH PRIORITY (using uppercase > > ;). > > We should not have to wait so long to see performance tests removed > > from fast unit tests (while keeping the functional coverage). > Apologies, it slipped from my radar. Will send out a patch soon. > > > > > >
