Re: [ovs-dev] [PATCH v6 3/3] ovn-northd: Add logical flows to support native DNS

2017-05-02 Thread Ben Pfaff
On Tue, Apr 25, 2017 at 07:59:54PM +0530, nusid...@redhat.com wrote:
> From: Numan Siddique 
> 
> OVN implements native DNS resolution which can be used to resolve the
> internal DNS names belonging to a logical datapath.
> 
> To support this, a new table 'DNS' is added in the NB DB. A new column
> 'dns_records' is added in 'Logical_Switch' table which references to the
> 'DNS' table.
> 
> Following flows are added for each logical switch if configured with
> DNS records in the 'dns_records' column
>  - A logical flow in DNS_LOOKUP stage which uses the action 'dns_lookup'
>to transform the DNS query to DNS reply packet and advances
>to the next stage - DNS_RESPONSE.
> 
>  - A logical flow in DNS_RESPONSE stage which implements the DNS responder
>by sending the DNS reply from previous stage back to the inport.
> 
> Signed-off-by: Numan Siddique 
> Acked-by: Gurucharan Shetty 

This has been a long time coming.  Thank you for persisting!  I think
that it will be a great feature.  I applied this series to master.

I sent out a patch to add conditional monitoring support:
https://mail.openvswitch.org/pipermail/ovs-dev/2017-May/331836.html
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH v6 3/3] ovn-northd: Add logical flows to support native DNS

2017-05-02 Thread Numan Siddique
On Wed, May 3, 2017 at 4:38 AM, Ben Pfaff  wrote:

> On Tue, Apr 25, 2017 at 07:59:54PM +0530, nusid...@redhat.com wrote:
> > From: Numan Siddique 
> >
> > OVN implements native DNS resolution which can be used to resolve the
> > internal DNS names belonging to a logical datapath.
> >
> > To support this, a new table 'DNS' is added in the NB DB. A new column
> > 'dns_records' is added in 'Logical_Switch' table which references to the
> > 'DNS' table.
> >
> > Following flows are added for each logical switch if configured with
> > DNS records in the 'dns_records' column
> >  - A logical flow in DNS_LOOKUP stage which uses the action 'dns_lookup'
> >to transform the DNS query to DNS reply packet and advances
> >to the next stage - DNS_RESPONSE.
> >
> >  - A logical flow in DNS_RESPONSE stage which implements the DNS
> responder
> >by sending the DNS reply from previous stage back to the inport.
> >
> > Signed-off-by: Numan Siddique 
> > Acked-by: Gurucharan Shetty 
>
> This has been a long time coming.  Thank you for persisting!  I think
> that it will be a great feature.  I applied this series to master.
>

Thanks Ben and Guru for the reviews.



>
> I sent out a patch to add conditional monitoring support:
> https://mail.openvswitch.org/pipermail/ovs-dev/2017-May/
> 331836.html
>

Thanks for this patch.

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