> On Oct 19, 2015, at 8:50 AM, Ben Pfaff <b...@nicira.com> wrote: > > On Sun, Oct 18, 2015 at 04:22:22PM -0700, Shashank Shanbhag wrote: >> Fix replace-flows and diff-flows to modify/diff flows in multiple tables. >> Add a --tables(-T) option that allows the user to specify a comma-separated >> list of table indexes to replace/diff. >> >> Signed-off-by: Shashank Shanbhag <shashank.shanb...@gmail.com> >> Acked-by: Romain Lenglet <romain.leng...@oracle.com> > > Hi Jarno, I know that you were talking about a related bug fix a few > days ago. Can you see if this patch does this same thing? I reviewed > earlier versions of it. I have not looked at this version yet.
Shashank, I just rebased and sent a version 2 of the replace-flows fix to the list, as I felt it was a bit less invasive change. It does not have the tables command line parameter, but instead operates on all possible tables. I’m not sure if this helps you in your use case, or if you need to explicitly leave some tables un-touched by replace-flows. If this is the case, I suggest you review this patch (http://patchwork.ozlabs.org/patch/546704/), and rebase your patch on top of it, but maybe have a command line option to exclude tables instead of listing what tables to process? Jarno _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
