On 10/6/26 2:34 PM, Aaron Conole wrote: > Aaron Conole <[email protected]> writes: > >> Ilya Maximets via dev <[email protected]> writes: >> >>> STP synchronization takes 30 seconds, which is way too long of a time >>> to wait in a test, and it's also on an edge of what the macros are >>> willing to wait for. Stop and warp the time instead. RSTP doesn't >>> suffer that much, but it's better to use the same pattern in the test. >>> >>> Also, there should be no reason to wait on disabling STP/RSTP. The >>> changes should be already applied when ovs-vsctl returns. >>> >>> Fixes: 332ca1dad732 ("ofproto-dpif: Fix bundle floodable flag when >>> disabling STP/RSTP.") >>> Signed-off-by: Ilya Maximets <[email protected]> >>> --- >> >> Reviewed-by: Aaron Conole <[email protected]> > > Ugh - I went on auto-pilot when typing my review tag. The email is > wrong and should actually be <[email protected]>
Thanks, Aaron! I fixed up the tag and applied the change. Backported down to 3.3. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
