On 3/14/23 09:04, Eelco Chaudron wrote: > > > On 14 Mar 2023, at 7:50, Faicker Mo wrote: > >> The error message when test failed, >> >> >>> 2023-03-14T02:41:16.010Z|00077|unixctl|DBG|received request >>> revalidator/purge[], id=0 >>> 2023-03-14T02:41:16.021Z|00001|dpif(revalidator6)|WARN|system@ovs-system: >>> failed to flow_get (No such file or directory) >>> ufid:de773de7-f35f-4abd-bc98-b1e1ee80787a <empty>, packets:0, >> bytes:0, used:never >>> 2023-03-14T02:41:16.021Z|00002|ofproto_dpif_upcall(revalidator6)|WARN|Failed >>> to acquire udpif_key corresponding to unexpected flow (No such file or >>> directory): ufid:de773de7-f35f-4abd >> -bc98-b1e1ee80787a >>> 2023-03-14T02:41:16.041Z|00078|unixctl|DBG|replying with success, id=0: "" >>> 2023-03-14T02:41:16.045Z|00079|unixctl|DBG|received request >>> dpctl/dump-flows[], id=0 >>> 2023-03-14T02:41:16.045Z|00080|unixctl|DBG|replying with success, id=0: >>> "recirc_id(0),in_port(3),eth(src=96:3f:94:a0:e4:5a,dst=aa:1a:54:e9:c5:56),eth_type(0x0800),ipv4(frag=no), >>> packe >> ts:1, bytes:84, used:0.270s, actions:2 >>> " >> >> I run the test for 3000 loops. > > I guess this is hard to troubleshoot, but I guess this is happening due to > flow deletion right in the middle of a revalidator run. It sounds odd that a > short delay solves it, but if it is we might leave it in for now. Any other > idea on this Ilya/Simon?
I don't like an idea of having a split-second sleep in the test, because it seems very error prone. If we need to wait, let's sleep for the whole second then. I'm not sure how to generally avoid the error that you see otherwise at the moment. And I'm a bit lost in this conversation, so I'll mark this patch as 'Changes Requested'. Please, submit v9 with the test changes that work for you folks, but without split-second sleeps. And please, don't send v9 in reply to v8, new versions should be new email threads. Thanks. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
