On 5/20/25 2:27 PM, Dumitru Ceara wrote: > On 5/19/25 9:43 PM, Ilya Maximets wrote: >> On 5/8/25 9:23 PM, Alexandra Rukomoinikova wrote: >>> Added checksum which is calculated for pipeline stages. >>> Also, the tool calculate-pipeline-cksum is introduced. This tool >>> calculates the cksum of a pipeline stages. >>> >>> Signed-off-by: Alexandra Rukomoinikova <arukomoinikova@k2.cloud> >>> --- > > Hi Alexandra, Ilya, > > Thanks for the patch and for the review! > >>> v4 --> v5: addressed Ilya comments. >>> rebased >>> --- >>> Makefile.am | 10 ++++++++++ >>> build-aux/automake.mk | 2 ++ >>> build-aux/calculate-pipeline-cksum | 10 ++++++++++ >>> build-aux/cksum-pipeline-check | 19 +++++++++++++++++++ >>> lib/ovn-util.c | 6 +++++- >>> 5 files changed, 46 insertions(+), 1 deletion(-) >>> create mode 100755 build-aux/calculate-pipeline-cksum >>> create mode 100755 build-aux/cksum-pipeline-check >> >> Thanks, Alexandra, for v5! I left a few nits below that maybe maintainers >> can address while applying. The patch seems to work fine otherwise. >> >> Acked-by: Ilya Maximets <i.maxim...@ovn.org> >> > > I folded in the minor changes Ilya requested and applied this patch to main.
Thanks! Should this also be backported down to e.g. 24.03? So we do not forget to update internal version in backports. This will make backporting potential pipeline changes a bit more difficult. But that's exactly the point. WDYT? Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
