On 10/14/24 15:01, Vladislav Odintsov wrote: > > On 14.10.2024 14:47, Dumitru Ceara wrote: >> On 10/13/24 10:19, Vladislav Odintsov wrote: >>> This picks up the following relevant OVS changes: >>> a15ce086d ofproto-dpif: Improve load balancing in dp_hash select groups. >>> 76ba41b5c vconn: Always properly free flow stats reply. >>> 64cb90507 ovsdb-idl: Fix IDL memory leak. >>> ... and others. >>> >>> Reported-at: >>> https://mail.openvswitch.org/pipermail/ovs-dev/2024-October/417627.html >>> Signed-off-by: Vladislav Odintsov <[email protected]> >>> --- >> Hi Vladislav, >> >>> ovs | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/ovs b/ovs >>> index c598c05c8..a15ce086d 160000 >>> --- a/ovs >>> +++ b/ovs >>> @@ -1 +1 @@ >>> -Subproject commit c598c05c85b2d38874a0ce8f7f088f6aae4fdabc >>> +Subproject commit a15ce086d41f9dfe6c1589333413b8e777401ef0 >> This would make branch-24.09 use a newer submodule version than the OVN >> main branch does. >> >> I think we need this commit on main too, what do you think? > > Hi Dumitru, > > Agree. > > The patch: > https://patchwork.ozlabs.org/project/ovn/patch/[email protected]/ >
I was talking offline to Ilya about all these bumps and he made a good point: while the tip of OVS branch-3.y and the latest v3.y.z get almost the same amount of testing in the OVS repo it might be a bit better to use v3.y.z instead. That's because likely external users of OVS run tagged OVS releases in production so those might get more external testing. I had a quick look at the main differences between choosing the tip of branch-3.y and v3.y.z on all branches and I think we'd only miss: 99e7cf9cce1c vconn: Always properly free flow stats reply. f59f19bf69a4 ovsdb-idl: Fix IDL memory leak. which might be OK. If you agree I can change your patches on all branches (no need to post new ones) and apply them. What do you think? Thanks, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
