On Wed, Dec 7, 2016 at 1:49 PM, Numan Siddique <nusid...@redhat.com> wrote:
> > > On Tue, Dec 6, 2016 at 4:12 AM, Ben Pfaff <b...@ovn.org> wrote: > >> On Sun, Oct 23, 2016 at 02:32:11PM -0700, Ben Pfaff wrote: >> > The put_dhcp_opts(v6) logical action didn't really work because >> ovn-trace >> > didn't handle DHCP options. This fixes the problem. >> > >> > This also makes the put_dhcp_opts(v6) logical provide useful tracing >> output >> > showing what's happening and the assumptions. >> > >> > Signed-off-by: Ben Pfaff <b...@ovn.org> >> > > > > Acked-by: > Numan Siddique > < > <b...@ovn.org>nusid...@redhat.com > > > > Looks like my previous email wasn't formatted properly. Acked-by: Numan Siddique <nusid...@redhat.com> Thanks Numan Numan > > > > --- >> > v1->v2: Improve output in a few minor ways: make --detailed mode mention >> > the DHCPDISCOVER/DHCPREQUEST assumption, make --minimal mode omit >> register >> > modifications. >> > >> > ovn/utilities/ovn-trace.c | 63 ++++++++++++++++++++++++++++++ >> ++++++++++++++--- >> > 1 file changed, 59 insertions(+), 4 deletions(-) >> >> This still needs a review, which shouldn't be difficult. >> _______________________________________________ >> dev mailing list >> d...@openvswitch.org >> https://mail.openvswitch.org/mailman/listinfo/ovs-dev >> > > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev