On 7/25/22 23:34, Han Zhou wrote: > When ovn-controller is restarted, it may need multiple iterations of > main loop before completely download all related data from SB DB, > especially when ovn-monitor-all=false, so after restart, before it sees > the related localnet ports from SB DB, it treats the related patch > ports on the chassis as not needed, and deleted them. Later when it > downloads thoses port-bindings it recreates them. For a graceful > upgrade, we don't want this to happen, because it would break the > traffic. > > This is especially problematic at ovn-k8s setups because the external > side of the patch port is used to program some flows for external > network communication. When the patch ports are recreated, the OF port > number changes and those flows are broken. The CMS would detect and fix > the flows but it would result in even longer downtime. > > This patch postpone the deletion of the patch ports, with the assumption > that leaving the unused ports on the chassis for little longer is not > harmful. > > Signed-off-by: Han Zhou <[email protected]> > ---
Looks good to me, thanks! Acked-by: Dumitru Ceara <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
