From: Justin Pettit <[email protected]>
Signed-off-by: Justin Pettit <[email protected]>
Signed-off-by: Ben Pfaff <[email protected]>
---
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
[email protected]
http://openvswitch.org/mailman/listinfo/dev
