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