On Wed, Mar 29, 2017 at 11:29:27PM +0530, Numan Siddique wrote: > On Wed, Mar 29, 2017 at 11:12 PM, Ben Pfaff <b...@ovn.org> wrote: > > > On Mon, Mar 27, 2017 at 07:42:58PM +0530, nusid...@redhat.com wrote: > > > From: Numan Siddique <nusid...@redhat.com> > > > > > > This patch adds a new OVN action 'dns_lookup' to support native DNS. > > > ovn-controller parses this action and adds a NXT_PACKET_IN2 > > > OF flow with 'pause' flag set. > > > > > > A new table 'DNS' is added in the SB DB to look up and resolve > > > the DNS queries. When a valid DNS packet is received by > > > ovn-controller, it looks up the DNS name in the 'DNS' table > > > and if successful, it frames a DNS reply, resumes the packet > > > and stores 1 in the 1-bit subfield. If the packet is invalid > > > or cannot be resolved, it resumes the packet without any > > > modifications and stores 0 in the 1-bit subfield. > > > > > > reg0[4] = dns_lookup(); next; > > > > > > An upcoming patch will use this action and adds logical flows. > > > > > > Signed-off-by: Numan Siddique <nusid...@redhat.com> > > > > I didn't do a thorough review, but here's a suggested incremental for > > the documentation. (I started by noticing that a DNS request should > > have udp.dst 53, not udp.src 53, and then got carried away a little.) > > > > > Thanks Ben for pointing out and correcting it. Before I update the patch, I > would like to know your opinion on the approach suggested by Guru - ( > https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/330235.html) as > that may require some changes in this patch as well.
I just spent some time talking to Guru and I think that his design is a good one and I support shifting to it. I think that it will scale better. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
