On 9/27/22 02:01, Ilya Maximets wrote: > On 9/20/22 09:12, Ales Musil wrote: >> >> >> On Fri, Sep 16, 2022 at 5:50 PM Dumitru Ceara <[email protected] >> <mailto:[email protected]>> wrote: >> >> Part of the uuidset implementation is taken from the OVN codebase where >> it was added via commit 0e77b3bcbfe2 ("ovn-northd-ddlog: New >> implementation of ovn-northd based on ddlog."). >> >> We now extend that, adding a few helpers and tests. >> >> Co-authored-by: Leonid Ryzhyk <[email protected] >> <mailto:[email protected]>> >> Signed-off-by: Leonid Ryzhyk <[email protected] >> <mailto:[email protected]>> >> Co-authored-by: Justin Pettit <[email protected] <mailto:[email protected]>> >> Signed-off-by: Justin Pettit <[email protected] <mailto:[email protected]>> >> Co-authored-by: Ben Pfaff <[email protected] <mailto:[email protected]>> >> Signed-off-by: Ben Pfaff <[email protected] <mailto:[email protected]>> >> Signed-off-by: Dumitru Ceara <[email protected] >> <mailto:[email protected]>> >> --- >> Note: I wasn't sure if I should keep Leonid as main author so I added a >> "Co-authored-by" tag. I can amend that and send a v2 if needed. >> --- >> lib/automake.mk <http://automake.mk> | 2 + >> lib/uuidset.c | 99 ++++++++++++++++++++++++++++++++++++++++++++ >> lib/uuidset.h | 71 +++++++++++++++++++++++++++++++ >> tests/.gitignore | 1 + >> tests/automake.mk <http://automake.mk> | 1 + >> tests/library.at <http://library.at> | 4 ++ >> tests/test-uuidset.c | 72 ++++++++++++++++++++++++++++++++ >> 7 files changed, 250 insertions(+) >> create mode 100644 lib/uuidset.c >> create mode 100644 lib/uuidset.h >> create mode 100644 tests/test-uuidset.c >> >> Looks good to me, thanks. >> The robot ERROR seems to be a false positive. >> >> Reviewed-by: Ales Musil <[email protected] <mailto:[email protected]>> > > Applied. Thanks! > > Best regards, Ilya Maximets. >
Thanks! _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
