Re: [ovs-dev] [PATCH ovn v2 2/2] controller: Fix IPv6 prefix delegation

2022-07-25 Thread Ales Musil
Hi Mark, thank you for the review. On Mon, Jul 25, 2022 at 6:09 PM Mark Michelson wrote: > Hi Ales, I have one small finding below, but other than that, it looks > good. I don't think you need to roll a new version to address this issue, > so > > Acked-by: Mark Michelson > > On 7/25/22 06:49,

Re: [ovs-dev] [PATCH ovn v2 2/2] controller: Fix IPv6 prefix delegation

2022-07-25 Thread Mark Michelson
Hi Ales, I have one small finding below, but other than that, it looks good. I don't think you need to roll a new version to address this issue, so Acked-by: Mark Michelson On 7/25/22 06:49, Ales Musil wrote: The local_datapaths are populated by iterating through all available port bindings o

[ovs-dev] [PATCH ovn v2 2/2] controller: Fix IPv6 prefix delegation

2022-07-25 Thread Ales Musil
The local_datapaths are populated by iterating through all available port bindings or new ones added during I-C. The port bindings are not ordered, it might happen that the order between engine runs are different. This can cause the prefix delegation shash to be with NULL local_datapath. Even in ca