On Thu, Mar 21, 2013 at 2:31 PM, Ben Pfaff <[email protected]> wrote: > On Thu, Mar 21, 2013 at 02:07:39PM -0700, Gurucharan Shetty wrote: >> This patch adds two new scripts that run "ovs-ofctl show" and >> "ovs-ofctl dump-flows" on each bridge. >> >> Signed-off-by: Gurucharan Shetty <[email protected]> > >> +for bridge in `ovs-vsctl list-br` >> +do > > The quoting here is really curious: > >> + echo "ovs-ofctl dump-flows "${bridge}"" I will get rid of the extra quotes and push this to master and 1.10
Do you think we need a new script that does "ovs-appctl dpif/dump-flows" on each bridge? The current "ovs-dpctl dump-flows system@ovs-system" may give a lot of o/p. >> + ovs-ofctl dump-flows "$bridge" >> + echo "" >> +done > > There is something similar in the other script. > > Otherwise this looks good, thank you. > > I guess that you will backport these patches to 1.10? > > Thanks, > > Ben. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
