On Wed, Jul 27, 2016 at 2:07 AM, Ben Pfaff <b...@ovn.org> wrote:
> On Wed, Jul 27, 2016 at 12:55:00AM +0530, Numan Siddique wrote:
> > This patch adds a new OVN action 'put_dhcpv6_opts' to support native
> > DHCPv6 in OVN.
> >
> > ovn-controller parses this action and adds a NXT_PACKET_IN2
> > OF flow with 'pause' flag set and the DHCPv6 options stored in
> > 'userdata' field.
> >
> > When the valid DHCPv6 packet is received by ovn-controller, it frames a
> > new DHCPv6 reply packet with the DHCPv6 options present in the
> > 'userdata' field and resumes the packet and stores 1 in the 1-bit
> subfield.
> > If the packet is invalid, it resumes the packet without any modifying and
> > stores 0 in the 1-bit subfield.
> >
> > Eg. reg0[3] = put_dhcpv6_opts(IA_ADDR = aef0::4, SERVER_ID =
> 00:00:00:00:10:02,
> > DNS_RECURSIVE_SERVER={ae70::1,ae70::2}....)
> >
> > A new 'DHCPv6_Options' table is added in SB DB which stores
> > the supported DHCPv6 options with DHCPv6 code and type. ovn-northd is
> > expected to popule this table.
> >
> > Upcoming patch will add logical flows with this action.
> >
> > Signed-off-by: Numan Siddique <nusid...@redhat.com>
>
> Same comment here as previously, that the put_dhcpv6_opts action needs
> documentation in ovn-sb.xml.
>
I think I have added the documentation in ovn-sb.xml. I will recheck again.
I will submit another patch set addressing the comments.
Thanks again for the reviews.
Numan
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
