On Thu, Apr 21, 2016 at 4:53 AM, Ramu Ramamurthy <ramu.ramamur...@gmail.com> wrote:
> Tested-by: Ramu Ramamurthy <ramu.ramamur...@us.ibm.com> > > Numan, I tested this patch to work on devstack+ovn without the > openstack-plugin, > with manual configuration. > > Notes: > > 1) In ovn/utilities/ovn-nbctl.c, usage() Can you add a help string to > ovn-nbctl for the new command > lswitch-get-dhcp-options, and lswitch-set-dhcp-options > > 2) When server_mac was not defined, the lflow was created like this, > maybe the error > checks can be tighter in ovn-northd.c > > table=3( ls_in_dhcp), priority= 50, match=(inport == > "7fe086b5-9ab0-4c14-bf62-0291a62a4b14" && eth.src == fa:16:3e:cf:3d:bc > && ip4.src == 0.0.0.0 && ip4.dst == 255.255.255.255 && udp.src == 68 > && udp.dst == 67), action=(dhcp_offer(offerip = 10.0.1.3, netmask = > 255.255.255.0, router = 10.0.1.1, mtu = 1400, server_id = 10.0.1.2, > dns_server = {8.8.8.8,7.7.7.7}, lease_time = 43200); eth.dst = > eth.src; eth.src = (null); ip4.dst = 10.0.1.3; ip4.src = 10.0.1.2; > udp.src = 67; udp.dst = 68; outport = inport; inport = ""; /* Allow > sending out inport. */ output;) > > The ovn-controller failed to parse the above flow at eth.src > > 2016-04-20T23:19:49Z|00070|lflow|WARN|error parsing actions > "dhcp_offer(offerip = 10.0.1.2, netmask = 255.255.255.0, router = > 10.0.1.1, mtu = 1400, server_id = 10.0.1.2, dns_server = > {8.8.8.8,7.7.7.7}, lease_time = 43200); eth.dst = eth.src; eth.src = > (null); ip4.dst = 10.0.1.2; ip4.src = 10.0.1.2; udp.src = 67; udp.dst > = 68; outport = inport; inport = ""; /* Allow sending out inport. */ > output;": Syntax error at `(' expecting constant. > > > Thanks Ramu for testing this out. I will fix the above 2 issues. Regards Numan _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev