From: Justin Pettit <jpet...@nicira.com>

Signed-off-by: Justin Pettit <jpet...@nicira.com>
Signed-off-by: Ben Pfaff <b...@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 32bbc67..c537b87 100644
--- a/ovn/controller/ovn-controller.c
+++ b/ovn/controller/ovn-controller.c
@@ -87,7 +87,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)
 {
@@ -118,7 +121,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...");
-- 
2.1.3

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

Reply via email to