On 8/14/23 21:01, Mark Michelson wrote:
> On 8/14/23 12:25, Han Zhou wrote:
>> On Mon, Aug 14, 2023 at 6:54 AM Dumitru Ceara <[email protected]> wrote:
>>>
>>> Fixes: 596ea7acbe68 ("ovn-controller: Detect and use L4_SYM dp-hash if
>> available.")
>>> Signed-off-by: Dumitru Ceara <[email protected]>
>>> ---
>>>   NEWS | 3 +++
>>>   1 file changed, 3 insertions(+)
>>>
>>> diff --git a/NEWS b/NEWS
>>> index 8275877f99..f99334c1b8 100644
>>> --- a/NEWS
>>> +++ b/NEWS
>>> @@ -10,6 +10,9 @@ Post v23.06.0
>>>     - To allow optimizing ovn-controller's monitor conditions for the
>> regular
>>>       VIF case, ovn-controller now unconditionally monitors all
>>> sub-ports
>>>       (ports with parent_port set).
>>> +  - ECMP routes use L4_SYM dp-hash by default if the datapath supports
>> it.
>>> +    Existing sessions might get re-hashed to a different ECMP path when
>>> +    OVN detects the algorithm support in the datapath.
>>
>> nit: probably add "during the upgrade" to avoid misunderstanding.
>>
>> Acked-by: Han Zhou <[email protected]>
> 
> I updated the sentence to be "Existing sessions might get re-hashed to a
> different ECMP path when OVN detects the algorithm support in the
> datapath during an upgrade or restart of ovn-controller."
> 
> I pushed the change to main. Thanks Dumitru and Han.
> 

Thanks, Han, Mark!

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

Reply via email to