On Fri, Apr 28, 2017 at 04:44:32PM -0400, Aaron Conole wrote: > Greetings dev, > > I have whipped up a quick little utility (find below), that I've done a > bit of debugging with and it seems to have made working with dump-flows > from ovs-ofctl a little easier to use. > > If you think it's worthwhile to add to the repository, I'll submit it > formally. We were using it while debugging some strange packet > forwarding in openshift. > > -Aaron
Thanks for working to make the ovs-ofctl formatting better! I prefer to interpret this script as a kind of "feature request" for "ovs-ofctl dump-flows". This command already has some special support, compared to other ovs-ofctl commands, and it might make sense to continue adding to it. ovs-ofctl dump-flows already has one of the features that this script adds, that is, sorting the flows and removing "OFPST_FLOW" lines. You turn this on by using the "--sort" (or "--rsort") option. The other features that this script provides all seem like ones that would be useful to add to ovs-ofctl itself. I'd tend to prefer to continue enhancing it rather than adding wrapper scripts; I think that this is likely to yield a more coherent design in the end, and possibly higher quality. Is that something you're willing to consider? Thanks, Ben. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
