Re: [ovs-dev] [PATCH ovn] northd: Use LB port_str in northd.

2023-03-02 Thread Dumitru Ceara
On 2/21/23 13:26, Ales Musil wrote: > On Fri, Feb 17, 2023 at 9:41 PM Dumitru Ceara wrote: > >> As mentioned in the structure definition in lb.h, northd should never >> use 'vip_port' instead it should always use 'port_str'. That's because >> it might not always be possible to evaluate the port

Re: [ovs-dev] [PATCH ovn] northd: Use LB port_str in northd.

2023-02-21 Thread Ales Musil
On Fri, Feb 17, 2023 at 9:41 PM Dumitru Ceara wrote: > As mentioned in the structure definition in lb.h, northd should never > use 'vip_port' instead it should always use 'port_str'. That's because > it might not always be possible to evaluate the port value in northd, > e.g., when using LB

[ovs-dev] [PATCH ovn] northd: Use LB port_str in northd.

2023-02-17 Thread Dumitru Ceara
As mentioned in the structure definition in lb.h, northd should never use 'vip_port' instead it should always use 'port_str'. That's because it might not always be possible to evaluate the port value in northd, e.g., when using LB templates. This commit also updates other occurrences of