The IDLs as created by ovn-controller always retry failed connections,
which means that ovsdb_idl_is_alive() will always report that they are
alive.

Signed-off-by: Ben Pfaff <b...@nicira.com>
Acked-by: Russell Bryant <rbry...@redhat.com>
---
 ovn/controller/ovn-controller.c | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git a/ovn/controller/ovn-controller.c b/ovn/controller/ovn-controller.c
index bd3ef0d..6f94187 100644
--- a/ovn/controller/ovn-controller.c
+++ b/ovn/controller/ovn-controller.c
@@ -210,22 +210,6 @@ main(int argc, char *argv[])
             break;
         }
 
-        if (!ovsdb_idl_is_alive(ctx.ovnsb_idl)) {
-            int retval = ovsdb_idl_get_last_error(ctx.ovnsb_idl);
-            VLOG_ERR("%s: database connection failed (%s)",
-                     ovnsb_remote, ovs_retval_to_string(retval));
-            retval = EXIT_FAILURE;
-            break;
-        }
-
-        if (!ovsdb_idl_is_alive(ctx.ovs_idl)) {
-            int retval = ovsdb_idl_get_last_error(ctx.ovs_idl);
-            VLOG_ERR("%s: database connection failed (%s)",
-                     ovs_remote, ovs_retval_to_string(retval));
-            retval = EXIT_FAILURE;
-            break;
-        }
-
         ofctrl_clear_flows();
 
         chassis_run(&ctx);
-- 
2.1.3

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

Reply via email to