Re: [ovs-dev] [PATCH ovn] controller: add ovn-set-local-ip option

2022-02-09 Thread Vladislav Odintsov
Hi Mohammad, thanks for the review! I’ve added the requested testcase and submitted the v2: https://patchwork.ozlabs.org/project/ovn/patch/20220209233314.51948-1-odiv...@gmail.com/ Regards, Vladislav Odintsov > On 6 Feb 2022, at 12:59, Mohammad Heib wrote: > > Hi Vladislav, > > your change

Re: [ovs-dev] [PATCH ovn] controller: add ovn-set-local-ip option

2022-02-06 Thread Mohammad Heib
Hi Vladislav, your change looks good to me. just if you can add a small test under ./tests to validate your change and make sure that future changes will not break your change (see ./tests/ovn-ipsec.at  for a good example). Thanks, On 1/27/22 18:11, Vladislav Odintsov wrote: When transport

[ovs-dev] [PATCH ovn] controller: add ovn-set-local-ip option

2022-01-27 Thread Vladislav Odintsov
When transport node has multiple interfaces (vlans) and ovn-encap-ip on different hosts need to be configured from different VLANs source IP for encapsulated packet can be not the same, which is expected by remote system. Explicitely setting local_ip resolves such problem. Signed-off-by: