Re: [ovs-dev] [PATCH ovn] binding: handle ovs ofport update

2023-07-31 Thread Mohammad Heib
Hi Mark, thank you for reviewing my patch is submitted v2 addressing your comments. On 7/28/23 23:16, Mark Michelson wrote: Hi Mohammad, The actual code change looks good, but I have some other minor comments to address. See inline below. On 7/26/23 12:12, Mohammad Heib wrote: Currently wh

Re: [ovs-dev] [PATCH ovn] binding: handle ovs ofport update

2023-07-28 Thread Mark Michelson
Hi Mohammad, The actual code change looks good, but I have some other minor comments to address. See inline below. On 7/26/23 12:12, Mohammad Heib wrote: Currently when ovs interface ofport is updated after setting external_ids:iface_id, the ovn-controller will see this change but will not do

[ovs-dev] [PATCH ovn] binding: handle ovs ofport update

2023-07-26 Thread Mohammad Heib
Currently when ovs interface ofport is updated after setting external_ids:iface_id, the ovn-controller will see this change but will not do much if it handles this change incrementally. This behavior leads to a mismatch between the ovs openflow flows in table=0 (inaccurate in_port) and the ofport