>
> 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.
>

ack, sorry to not be clear on this. I will add it in the next iteration :)

Regards,
Lorenzo

_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to