On 10/17/25 8:31 AM, Felix Huettner wrote:
> On Thu, Oct 16, 2025 at 12:17:40PM +0200, Dumitru Ceara wrote:
>> On 10/16/25 10:40 AM, Felix Huettner via dev wrote:
>>> On Tue, Oct 07, 2025 at 05:27:43PM +0300, Alexandra Rukomoinikova wrote:
>>>> Restore inter-AZ connectivity during OVN upgrades by allowing
>>>> legacy routes (without lr-id) from different availability zones
>>>> to be learned. Previously, updated nodes skipped all routes
>>>> without lr-id, breaking connectivity to non-upgraded AZs.
>>>>
>>>> The fix checks availability zone for legacy routes instead of
>>>> skipping them entirely, maintaining backward compatibility.
>>>
>>> Hi Alexandra,
>>>
>>> sorry for taking so long to take a look.
>>> That seems reasonable to me.
>>>
>>> However there seems to be some issue with applying the patch in the CI.
>>> Still take this:
>>> Signed-off-by: Felix Huettner <[email protected]>
>>>
>>
>> Hi Felix,
>>
>> Hmm, I'm a bit confused about this.
>>
>> This patch from Alexandra was already accepted:
>> https://github.com/ovn-org/ovn/commit/951d992
>>
>> Do you mean to say you should've been co-author or did you actually want
>> to ack the patch?
> 
> Hi Dumitru,
> 

Hi Felix,

> i'm sorry i was probably too tired yesterday. I actually wanted to ack
> it and did not realized it was already accepted.
> 
> Sorry for the confusion.
> 

No worries, thanks for the review!

>>
>> There's a backport patch available for review for branches <= 25.03:
>>
>> https://patchwork.ozlabs.org/project/ovn/patch/[email protected]/
>>
>> That's because there were too many non-trivial conflicts when I tried to
>> backport the original fix to older branches, see:
>> https://mail.openvswitch.org/pipermail/ovs-dev/2025-October/426866.html
>>
>> Thanks,
>> Dumitru
>>
>>> Thanks,
>>> Felix
>>>

Regards,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to