On Tue, Apr 5, 2016 at 9:43 AM, Na Zhu <na...@cn.ibm.com> wrote: > This patch add column "enabled" to table Logical_Router > for setting router administrative state. > > The type of "enabled" is bool. > > If the administrative state is false, delete all the flows > relevant to the logical router from table Logical_Flow. > > Signed-off-by: Na Zhu <na...@cn.ibm.com> > Acked-by:
Did you intend to have a name and email address with "Acked-by" here? Otherwise, just remove it. I also can't seem to apply this patch. I tried applying it from the mailing list as well as patchwork. Please try posting it with "git send-email", as that seems to get the best results. It would also be nice to have some kind of test coverage for this feature to help make sure we don't accidentally break it in the future. > --- > ovn/northd/ovn-northd.8.xml | 4 ++++ > ovn/northd/ovn-northd.c | 10 ++++++++++ > ovn/ovn-nb.ovsschema | 5 +++-- > ovn/ovn-nb.xml | 7 +++++++ > 4 files changed, 24 insertions(+), 2 deletions(-) > > diff --git a/ovn/northd/ovn-northd.8.xml b/ovn/northd/ovn-northd.8.xml > index da776e1..f7a11c4 100644 > --- a/ovn/northd/ovn-northd.8.xml > +++ b/ovn/northd/ovn-northd.8.xml > @@ -397,6 +397,10 @@ output; > > <h2>Logical Router Datapaths</h2> > > + <p> > + Logical router datapaths will only exist for <ref > table="Logical_Router" db="OVN_Northbound"/> rows in the <ref > db="OVN_Northbound"/> database that do not have <ref column="enabled" > table="Logical_Router" db="OVN_Northbound"/> set to <code>false</code> This should be line wrapped. -- Russell Bryant _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
