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

[Russell Bryant]

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.

-- 
Russell Bryant