[ovs-dev] [PATCH ovn 10/11] ovn-nbctl: Take default database target from OVN_NB_DB in environment.

Ben Pfaff blp at nicira.com
Sun Jun 14 19:19:04 UTC 2015


This makes life easier for testing at the point you start to separate your
environment into multiple machines.

Also work on the manpage a little.

Signed-off-by: Ben Pfaff <blp at nicira.com>
---
 ovn/ovn-nbctl.8.xml   | 38 +++++++++++++++++++++++++++-----------
 ovn/ovn-nbctl.c       |  5 ++++-
 python/build/nroff.py |  2 +-
 3 files changed, 32 insertions(+), 13 deletions(-)

diff --git a/ovn/ovn-nbctl.8.xml b/ovn/ovn-nbctl.8.xml
index 7100652..0e89229 100644
--- a/ovn/ovn-nbctl.8.xml
+++ b/ovn/ovn-nbctl.8.xml
@@ -182,20 +182,36 @@
     </dl>
 
     <h1>Options</h1>
-    <p><code>-d</code> <var>database</var> | <code>--db</code> <var>database</var></p>
-    <p><code>-h</code> | <code>--help</code></p>
-    <p><code>-o</code> | <code>--options</code></p>
-    <p><code>-V</code> | <code>--version</code></p>
+
+    <dl>
+    <dt><code>-d</code> <var>database</var></dt>
+    <dt><code>--db</code> <var>database</var></dt>
+    <dd>
+      The OVSDB database remote to contact.  If the <env>OVN_NB_DB</env>
+      environment variable is set, its value is used as the default.
+      Otherwise, the default is <code>unix:@RUNDIR@/db.sock</code>, but this
+      default is unlikely to be useful outside of single-machine OVN test
+      environments.
+    </dd>
+
+    <dt><code>-h</code> | <code>--help</code></dt>
+    <dt><code>-o</code> | <code>--options</code></dt>
+    <dt><code>-V</code> | <code>--version</code></dt>
+    </dl>
 
     <h1>Logging options</h1>
-    <p><code>-v</code><var>spec</var>, <code>--verbose=</code><var>spec</var></p>
-    <p><code>-v</code>, <code>--verbose</code></p>
-    <p><code>--log-file</code>[<code>=</code><var>file</var>]</p>
-    <p><code>--syslog-target=</code><var>host</var><code>:</code><var>port</var></p>
+    <dl>
+    <dt><code>-v</code><var>spec</var>, <code>--verbose=</code><var>spec</var></dt>
+    <dt><code>-v</code>, <code>--verbose</code></dt>
+    <dt><code>--log-file</code>[<code>=</code><var>file</var>]</dt>
+    <dt><code>--syslog-target=</code><var>host</var><code>:</code><var>port</var></dt>
+    </dl>
 
     <h1>PKI configuration (required to use SSL)</h1>
-    <p><code>-p</code>, <code>--private-key=</code><var>file</var>  file with private key</p>
-    <p><code>-c</code>, <code>--certificate=</code><var>file</var>  file with certificate for private key</p>
-    <p><code>-C</code>, <code>--ca-cert=</code><var>file</var>      file with peer CA certificate</p>
+    <dl>
+    <dt><code>-p</code>, <code>--private-key=</code><var>file</var>  file with private key</dt>
+    <dt><code>-c</code>, <code>--certificate=</code><var>file</var>  file with certificate for private key</dt>
+    <dt><code>-C</code>, <code>--ca-cert=</code><var>file</var>      file with peer CA certificate</dt>
+    </dl>
 
 </manpage>
diff --git a/ovn/ovn-nbctl.c b/ovn/ovn-nbctl.c
index 4dc77c0..a8e5d08 100644
--- a/ovn/ovn-nbctl.c
+++ b/ovn/ovn-nbctl.c
@@ -771,7 +771,10 @@ default_db(void)
 {
     static char *def;
     if (!def) {
-        def = xasprintf("unix:%s/db.sock", ovs_rundir());
+        def = getenv("OVN_NB_DB");
+        if (!def) {
+            def = xasprintf("unix:%s/db.sock", ovs_rundir());
+        }
     }
     return def;
 }
diff --git a/python/build/nroff.py b/python/build/nroff.py
index 55dc878..050518c 100644
--- a/python/build/nroff.py
+++ b/python/build/nroff.py
@@ -65,7 +65,7 @@ def inlineXmlToNroff(node, font, to_upper=False):
         else:
             return textToNroff(node.data, font)
     elif node.nodeType == node.ELEMENT_NODE:
-        if node.tagName in ['code', 'em', 'option']:
+        if node.tagName in ['code', 'em', 'option', 'env']:
             s = r'\fB'
             for child in node.childNodes:
                 s += inlineXmlToNroff(child, r'\fB')
-- 
2.1.3




More information about the dev mailing list