On 8/8/22 16:17, Aaron Conole wrote: > Ilya Maximets <[email protected]> writes: > >> On 8/5/22 20:32, Aaron Conole wrote: >>> Ilya Maximets <[email protected]> writes: >>> >>>> The permanent neighbor entry for fc00::1 is added into a wrong >>>> namespace, so in order to reply to a ping from at_ns1, the >>>> address of fc00::1 has to be discovered. >>> >>> This is strange - how did it end up in wrong namespace? Is it a race >>> with the veth setup? I guess we could possibly fix that? >> >> We have: >> NS_CHECK_EXEC([at_ns0], [ip -6 neigh add fc00::1 lladdr e4:11:22:33:44:55 >> dev p0]) >> >> But it should be: >> NS_CHECK_EXEC([at_ns1], [ip -6 neigh add fc00::1 lladdr e4:11:22:33:44:55 >> dev p1]) >> >> because fc00::1 is an IP of p0 in at_ns0. There is no point adding the >> entry to the namespace where it is directly accessible. > > Ugh, I glanced at the NS setup block too quickly and missed that. > >> We could just fix that, but I re-worked the test a little bit instead. > > Makes sense to me. > > Acked-by: Aaron Conole <[email protected]>
Thanks, Salem, Michael and Aaron! I also left the test running during weekend and it didn't fail after 30K iterations. Should be stable enough. :) Applied and backported down to 2.13. best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
