From 998bb652ecb494af8aa38ed0ae4e653cd8dff709 Mon Sep 17 00:00:00 2001 From: Ethan Jackson Date: Wed, 22 Sep 2010 12:40:39 -0700 Subject: [PATCH] xenserver: monitor-external-ids should run with --monitor The init script starts monitor-external-ids with --monitor when configured to do so. Also made changes to guarantee that --monitor actually restarts ovs-external-ids. Signed-off-by: Ethan Jackson --- python/ovs/daemon.py | 7 +++++++ xenserver/etc_init.d_openvswitch | 2 +- xenserver/usr_share_openvswitch_scripts_ovs-external-ids | 2 +- 3 files changed, 9 insertions(+), 2 deletions(-) diff --git a/python/ovs/daemon.py b/python/ovs/daemon.py index eaaaa519b..6dff3a02f 100644 --- a/python/ovs/daemon.py +++ b/python/ovs/daemon.py @@ -52,6 +52,8 @@ _monitor = False # File descriptor used by daemonize_start() and daemonize_complete(). _daemonize_fd = None +RESTART_EXIT_CODE = 5 + def make_pidfile_name(name): """Returns the file name that would be used for a pidfile if 'name' were provided to set_pidfile().""" @@ -266,6 +268,11 @@ def _fork_notify_startup(fd): os.close(fd) def _should_restart(status): + global RESTART_EXIT_CODE + + if os.WIFEXITED(status) and os.WEXITSTATUS(status) == RESTART_EXIT_CODE: + return True + if os.WIFSIGNALED(status): for signame in ("SIGABRT", "SIGALRM", "SIGBUS", "SIGFPE", "SIGILL", "SIGPIPE", "SIGSEGV", "SIGXCPU", "SIGXFSZ"): diff --git a/xenserver/etc_init.d_openvswitch b/xenserver/etc_init.d_openvswitch index e4641097d..5f18196cf 100755 --- a/xenserver/etc_init.d_openvswitch +++ b/xenserver/etc_init.d_openvswitch @@ -353,7 +353,7 @@ function start { # Start daemon to monitor external ids PYTHONPATH=/usr/share/openvswitch/python \ /usr/share/openvswitch/scripts/ovs-external-ids \ - --pidfile --detach "$VSWITCHD_OVSDB_SERVER" + --pidfile --detach $monitor_opt "$VSWITCHD_OVSDB_SERVER" touch /var/lock/subsys/openvswitch } diff --git a/xenserver/usr_share_openvswitch_scripts_ovs-external-ids b/xenserver/usr_share_openvswitch_scripts_ovs-external-ids index c69fe3775..fc27aaae1 100755 --- a/xenserver/usr_share_openvswitch_scripts_ovs-external-ids +++ b/xenserver/usr_share_openvswitch_scripts_ovs-external-ids @@ -279,4 +279,4 @@ if __name__ == '__main__': raise except: s_log.exception("traceback") - sys.exit(1) + sys.exit(ovs.daemon.RESTART_EXIT_CODE) -- 2.43.0