[ovs-dev] [PATCH 2/2] ovn-nbctl: Add lport-set-port-security command.

Ben Pfaff blp at nicira.com
Wed Apr 22 20:28:54 UTC 2015


On Wed, Apr 22, 2015 at 04:19:11PM -0400, Russell Bryant wrote:
> On 04/21/2015 07:37 PM, Ben Pfaff wrote:
> > On Tue, Apr 21, 2015 at 03:58:34PM -0700, Justin Pettit wrote:
> >>
> >>> On Apr 21, 2015, at 2:22 PM, Ben Pfaff <blp at nicira.com> wrote:
> >>>
> >>> +      <dt><code>lport-set-port-security</code> <var>lport</var> [<var>addrs</var>]...</dt>
> >>> +      <dd>
> >>> +        Sets the port security addresses associated with <var>lport</var> to
> >>> +        <var>addrs</var>.  Multiple sets of addresses may be set by using
> >>> +        multiple <var>addrs</var> arguments.  If no <var>addrs</var> argument
> >>> +        is given, <var>lport</var> will not have port security enabled.
> >>> +      </dd>
> >>
> >> Do you think it's worth mentioning that port security limits the
> >> addresses that this lport can use to send and what addresses it is
> >> allowed to receive?  
> > 
> > Yes.  I added a paragraph:
> > 
> > 	<p>
> > 	  Port security limits the addresses from which a logical port may send
> > 	  packets and to which it may receive packets.  See the
> > 	  <code>ovn-nb</code>(5) documentation for the <ref
> > 	  column="port_security" table="Logical_Port"/> column in the <ref
> > 	  table="Logical_Port"/> table for details.
> > 	</p>
> > 
> 
> This is a pretty minor thing, but when you added this paragraph, you
> used tabs instead of spaces.

It's funny that this has started happening in XML files lately.  Weird.

Anyway, I've added an editor hook now that should keep it from recurring.



More information about the dev mailing list