Signed-off-by: Justin Pettit <jpet...@nicira.com>
---
 ovn/controller/ovn-controller.8.xml |    5 +++++
 ovn/controller/ovn-controller.c     |    6 ++++--
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/ovn/controller/ovn-controller.8.xml 
b/ovn/controller/ovn-controller.8.xml
index ca7fa43..56900b9 100644
--- a/ovn/controller/ovn-controller.8.xml
+++ b/ovn/controller/ovn-controller.8.xml
@@ -110,5 +110,10 @@
           </p>
         </li>
       </ul>
+      <p>
+        <code>ovn-controller</code> does not support changing the
+        chassis name, integration bridge, or OVN database mid-run.  If
+        these values need to change, the daemon must be restarted.
+      </p>
     </p>
 </manpage>
diff --git a/ovn/controller/ovn-controller.c b/ovn/controller/ovn-controller.c
index dd3c30d..2898b41 100644
--- a/ovn/controller/ovn-controller.c
+++ b/ovn/controller/ovn-controller.c
@@ -84,7 +84,10 @@ get_bridge(struct controller_ctx *ctx, const char *name)
 /* Retrieve the OVN integration bridge from the "external-ids:ovn-bridge"
  * key, the remote location from the "external-ids:ovn-remote" key, and
  * the chassis name from the "external-ids:system-id" key in the
- * Open_vSwitch table of the OVS database instance. */
+ * Open_vSwitch table of the OVS database instance.
+ *
+ * ovn-controller does not support changing any of these mid-run.
+ */
 static void
 get_core_config(struct controller_ctx *ctx)
 {
@@ -115,7 +118,6 @@ get_core_config(struct controller_ctx *ctx)
             goto try_again;
         }
 
-        /* xxx This does not support changing OVN Southbound OVSDB mid-run. */
         remote = smap_get(&cfg->external_ids, "ovn-remote");
         if (!remote) {
             VLOG_INFO("OVN OVSDB remote not specified.  Waiting...");
-- 
1.7.5.4

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to