On 10/24/22 20:12, Mike Pattrick wrote: > On Thu, Oct 6, 2022 at 3:37 PM Ilya Maximets <[email protected]> wrote: >> >> Current macros are unable to filter whole seconds, e.g. 'duration:6s'. >> This is causing random test failures, most frequently in CirrusCI: >> >> ./dpif-netdev.at:370: ovs-ofctl -O OpenFlow13 meter-stats br0 | >> strip_timers >> --- - >> +++ /tmp/cirrus-ci-build/tests/testsuite.dir/at-groups/990/stdout >> @@ -1,5 +1,5 @@ >> OFPST_METER reply (OF1.3) (xid=0x2): >> -meter:1 flow_count:1 packet_in_count:10 byte_in_count:600 duration:0.0s >> bands: >> +meter:1 flow_count:1 packet_in_count:10 byte_in_count:600 duration:6s >> bands: >> >> Fix sed matches to correctly handle that scenario. >> Repeating the [0-9\.] twice because it is hard to write a shorter >> portable version with sed. >> >> Signed-off-by: Ilya Maximets <[email protected]> > > Acked-by: Mike Pattrick <[email protected]> >
Thanks! Applied and backported down to 2.13. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
