diff mbox

[ovs-dev,ovn-dev] ovn-nbctl: Improvement to the ovn-nbctl manpage

Message ID E6F9594D-66C9-4B71-8E0F-2A847772DD5E@opencloud.tech
State Accepted
Headers show

Commit Message

nickcooper-zhangtonghao July 2, 2016, 5:36 a.m. UTC 
Hi Ben,
	Thank you for your advice.


ovn-nbctl: the improvement to the ovn-nbctl manpage.

If there are multiple logical switches or routers with a duplicate name, 
the configuration is slightly different. You should configure the logical 
switches or routers using the UUID instead of the name.

Signed-off-by: nickcooper-zhangtonghao <nickcooper-zhangtonghao@opencloud.tech>




> On Jul 2, 2016, at 8:35 AM, Ben Pfaff <blp@ovn.org> wrote:
> 
> On Fri, Jul 01, 2016 at 12:38:54AM +0800, nickcooper-zhangtonghao wrote:
>> 	When I create one or more logical router named ’null', using 'ovn-nbctl lr-add’, there is no error information.
>> This is arguably an ovn-nbctl bug.
> 
> It's not a bug.  Any number of unnamed logical switches or logical
> routers may exist.
> 
> If the documentation is unclear then please submit an improvement to the
> ovn-nbctl manpge.
> 
> Thanks,
> 
> Ben.


Thanks,
nick

Comments

Ben Pfaff July 3, 2016, 7:13 p.m. UTC | #1 
On Sat, Jul 02, 2016 at 01:36:31PM +0800, nickcooper-zhangtonghao wrote:
> Hi Ben,
> 	Thank you for your advice.
> 
> 
> ovn-nbctl: the improvement to the ovn-nbctl manpage.
> 
> If there are multiple logical switches or routers with a duplicate name, 
> the configuration is slightly different. You should configure the logical 
> switches or routers using the UUID instead of the name.
> 
> Signed-off-by: nickcooper-zhangtonghao <nickcooper-zhangtonghao@opencloud.tech>

Applied, thanks!
diff mbox

Patch

diff --git a/ovn/utilities/ovn-nbctl.8.xml b/ovn/utilities/ovn-nbctl.8.xml
index c2ca420..b4b8501 100644
--- a/ovn/utilities/ovn-nbctl.8.xml
+++ b/ovn/utilities/ovn-nbctl.8.xml
@@ -50,7 +50,9 @@ 
           <code>--may-exist</code>, adding a duplicate name succeeds but does
           not create a new logical switch.  With <code>--add-duplicate</code>,
           the command really creates a new logical switch with a duplicate
-          name.  It is an error to specify both options.
+          name.  It is an error to specify both options.  If there are multiple
+          logical switches with a duplicate name, configure the logical switches
+          using the UUID instead of the <var>switch</var> name.
         </p>
       </dd>

@@ -267,7 +269,9 @@ 
           <code>--may-exist</code>, adding a duplicate name succeeds but does
           not create a new logical router.  With <code>--add-duplicate</code>,
           the command really creates a new logical router with a duplicate
-          name.  It is an error to specify both options.
+          name.  It is an error to specify both options.  If there are multiple
+          logical routers with a duplicate name, configure the logical routers
+          using the UUID instead of the <var>router</var> name.
         </p>
       </dd>