On Thu, Feb 25, 2021 at 05:59:01PM +0100, Lorenzo Bianconi wrote:
> >
> > On Thu, Feb 25, 2021 at 01:49:18PM +0100, Lorenzo Bianconi wrote:
> > > Lorenzo Bianconi (8):
> > >   northd: reduce indentation in build_lrouter_nat_defrag_and_lb
> > >   northd: introduce build_lrouter_lb_flows routine
> > >   northd: introduce build_lrouter_in_unsnat_flow routine
> > >   northd: introduce build_lrouter_in_dnat_flow routine
> > >   northd: introduce build_lrouter_out_undnat_flow routine
> > >   northd: introduce build_lrouter_out_snat_flow routine
> > >   northd: introduce build_lrouter_ingress_flow routine
> > >   northd: introduce lrouter_check_nat_entry routine
> >
> > Does this series change northd behavior?  If so, do you need any help
> > adapting ovn-northd-ddlog to behave similarly?
> >
> 
> Hi Ben,
> 
> there is no behavior change, just code refactoring.

Thanks!

It would be kind to include a sentence that says says that in the
commit messages, something like:

    This commit does not change ovn-northd behavior.

It's helpful not just for ovn-northd-ddlog but for anyone who is trying
to follow along with development, since it means that they don't have to
think so hard.  If they do notice a change in behavior, then it can also
help them decide that it's a bug that they should report.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to