> On Mar 3, 2016, at 7:18 AM, Russell Bryant <russ...@ovn.org> wrote: > > The "alg=" argument to the ct() action only makes sense when used in > combination with "commit". Add this to the documentation to help make > it clear. > > Signed-off-by: Russell Bryant <russ...@ovn.org> > --- > utilities/ovs-ofctl.8.in | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/utilities/ovs-ofctl.8.in b/utilities/ovs-ofctl.8.in > index 1b280ef..74b88d8 100644 > --- a/utilities/ovs-ofctl.8.in > +++ b/utilities/ovs-ofctl.8.in > @@ -1745,6 +1745,9 @@ connection arrives which is related, the \fBct\fR > action will set the > .RE > . > .IP > +The \fBcommit\fR parameter should be specified to use \fBalg=\fIalg\fR.
RFC-related feedback: Your suggestion sounds nicer, but do you think that should be "must" instead of "should"? I'm fine either way, though. Acked-by: Justin Pettit <jpet...@ovn.org> --Justin _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
