Re: [ovs-dev] [PATCH ovn v2 7/7] ci: Replace clang jemalloc suite with system-test-userspace

2023-03-01 Thread Simon Horman
On Tue, Feb 21, 2023 at 01:32:07PM +0100, Ales Musil wrote: > The test with jemalloc and clang is redundant as > we are already testing jemalloc with gcc. Replace > it with system-test-userspace which runs system tests > over userspace OvS datapath. > > Signed-off-by: Ales Musil FWIIW, I would

Re: [ovs-dev] [PATCH ovn v2 7/7] ci: Replace clang jemalloc suite with system-test-userspace

2023-02-21 Thread 0-day Robot
Bleep bloop. Greetings Ales Musil, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 81 characters long (recommended limit is 79) #27 FILE:

[ovs-dev] [PATCH ovn v2 7/7] ci: Replace clang jemalloc suite with system-test-userspace

2023-02-21 Thread Ales Musil
The test with jemalloc and clang is redundant as we are already testing jemalloc with gcc. Replace it with system-test-userspace which runs system tests over userspace OvS datapath. Signed-off-by: Ales Musil --- .github/workflows/test.yml | 6 +++--- 1 file changed, 3 insertions(+), 3