Hi Mike, Thanks for reworking the tests. One minor nit inline, otherwise patch LGTM,
> -----Original Message----- > From: Phelan, Michael <[email protected]> > Sent: Monday, July 11, 2022 7:19 PM > To: [email protected] > Cc: Stokes, Ian <[email protected]>; [email protected]; > [email protected]; [email protected]; Phelan, Michael > <[email protected]>; Pai G, Sunil <[email protected]> > Subject: [PATCH v3] tests: Add OVS-DPDK MTU unit tests. The title should be "system-dpdk:" instead of "tests:", didn't catch this earlier, but I guess this can be changed when applying, might not require another revision. > > This adds 8 new unit tests to the 'check-dpdk' subsystem that will test > Maximum Transmission Unit (MTU) functionality. > > Signed-off-by: Michael Phelan <[email protected]> > Signed-off-by: Sunil Pai G <[email protected]> > Co-authored-by: Sunil Pai G <[email protected]> > > --- > v3: > - Capitalize first letter in comments. > - Add [] in AT_CHECK commands. > - Remove unnecessary "," in grep commands. > - Establish connection with vhost front end (virtio-user) using testpmd > first in all vport tests. > Acked-by: Sunil Pai G <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
