[ovs-git] Open vSwitch: Enable daemon monitoring and automatic restart by default. (next)

dev at openvswitch.org dev at openvswitch.org
Fri Jan 15 23:34:57 UTC 2010


This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Open vSwitch".

The branch, next has been updated
       via  f3ac83df5c4fec3f27a9c8f95ef2753a26c65d24 (commit)
       via  ff8decf1a318b4a611cb08bb3f12833044e8a872 (commit)
       via  7943cd51e7eba1442846a853b6ed62e89a05d110 (commit)
       via  f59613353be350c697f76af2bba1205f7a7ad279 (commit)
       via  24c8ae53303ff25d118bb96d237da21a32085160 (commit)
       via  e3830e909d210145cbe52a1da66058ebf2581fd6 (commit)
       via  4d12270a618bead742d4cabc50601e86fde83f64 (commit)
       via  a723bf84fb639262d2bf7e33b7640a9621bb19b5 (commit)
       via  4a8535f39d5c4f6f34534ea1dcf9f35cddbde0c7 (commit)
      from  686cc0c00bb4cdb3e9ee95d43917ec0cdecd4a01 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit f3ac83df5c4fec3f27a9c8f95ef2753a26c65d24
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=f3ac83df5c4fec3f27a9c8f95ef2753a26c65d24
Author: Ben Pfaff <blp at nicira.com>
		
Enable daemon monitoring and automatic restart by default.
		
Tested on a XenServer by running "kill -SEGV" on the daemon processes and
watching them restart automatically and by running plain "kill" and
watching them terminate normally.


commit ff8decf1a318b4a611cb08bb3f12833044e8a872
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=ff8decf1a318b4a611cb08bb3f12833044e8a872
Author: Ben Pfaff <blp at nicira.com>
		
daemon: Add support for process monitoring and restart.
		


commit 7943cd51e7eba1442846a853b6ed62e89a05d110
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=7943cd51e7eba1442846a853b6ed62e89a05d110
Author: Ben Pfaff <blp at nicira.com>
		
daemon: Refactor code.
		
This commit should not change behavior, but it paves the way for
implementing --monitor in the following commit.


commit f59613353be350c697f76af2bba1205f7a7ad279
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=f59613353be350c697f76af2bba1205f7a7ad279
Author: Ben Pfaff <blp at nicira.com>
		
tests: New macro OVS_WAIT_WHILE.
		
For use by upcoming tests.


commit 24c8ae53303ff25d118bb96d237da21a32085160
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=24c8ae53303ff25d118bb96d237da21a32085160
Author: Ben Pfaff <blp at nicira.com>
		
tests: Factor OVSDB_INIT out of OVS_VSCTL_SETUP for other tests to use.
		
New tests in upcoming commits will use this.


commit e3830e909d210145cbe52a1da66058ebf2581fd6
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=e3830e909d210145cbe52a1da66058ebf2581fd6
Author: Ben Pfaff <blp at nicira.com>
		
fatal-signal: After fork, clear hooks instead of disabling them.
		
Until now, fatal_signal_fork() has simply disabled all the fatal signal
callback hooks.  This worked fine, because a daemon process forked only
once and the parent didn't do much before it exited.

But upcoming commits will introduce a --monitor option, which requires
processes to fork multiple times.  Sometimes the parent process will fork,
then run for a while, then fork again.  It's not good to disable the
hooks in the child process in such a case, because that prevents e.g.
pidfiles from being removed at the child's exit.

So this commit changes the semantics of fatal_signal_fork() to just
clearing out hooks.  After hooks are cleared, new hooks can be added and
will be executed on process termination in the usual way.

This commit also introduces a cancellation callback function so that a
canceled hook can free resources.


commit 4d12270a618bead742d4cabc50601e86fde83f64
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=4d12270a618bead742d4cabc50601e86fde83f64
Author: Ben Pfaff <blp at nicira.com>
		
unixctl: Avoid double error reporting.
		
All of these programs were re-reporting an error that
unixctl_server_create() had already reported.  There's no need for that.


commit a723bf84fb639262d2bf7e33b7640a9621bb19b5
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=a723bf84fb639262d2bf7e33b7640a9621bb19b5
Author: Ben Pfaff <blp at nicira.com>
		
unixctl: Improve error reporting.
		
The ovs_error() function is our standard way of reporting startup errors,
so use it.  (It also outputs the program name at the beginning of the
message.)


commit 4a8535f39d5c4f6f34534ea1dcf9f35cddbde0c7
Diffs: http://openvswitch.org/cgi-bin/gitweb.cgi?p=openvswitch;a=commitdiff;h=4a8535f39d5c4f6f34534ea1dcf9f35cddbde0c7
Author: Ben Pfaff <blp at nicira.com>
		
tests: Improve error reporting for timeval test failures.
		


-----------------------------------------------------------------------

Summary of changes:
 debian/openvswitch-switch.init                     |   13 +-
 debian/openvswitch-switch.template                 |    4 +
 extras/ezio/ovs-switchui.c                         |    4 +-
 extras/ezio/tty.c                                  |    4 +-
 lib/daemon.c                                       |  240 +++++++++++++++-----
 lib/daemon.h                                       |   11 +-
 lib/daemon.man                                     |   11 +
 lib/fatal-signal.c                                 |   77 ++++---
 lib/fatal-signal.h                                 |    6 +-
 lib/netdev.c                                       |    3 +-
 lib/unixctl.c                                      |   13 +-
 ovsdb/ovsdb-server.c                               |    2 +-
 tests/automake.mk                                  |    2 +
 tests/daemon.at                                    |  165 ++++++++++++++
 tests/ovs-vsctl.at                                 |   18 +--
 tests/ovsdb-macros.at                              |   21 ++
 tests/ovsdb-monitor.at                             |    2 +-
 tests/test-dhcp-client.c                           |    4 +-
 tests/test-timeval.c                               |   11 +-
 tests/testsuite.at                                 |   11 +-
 utilities/ovs-controller.c                         |    4 +-
 utilities/ovs-discover.c                           |    4 +-
 utilities/ovs-openflowd.c                          |    2 +-
 vswitchd/ovs-brcompatd.c                           |    2 +-
 vswitchd/ovs-vswitchd.c                            |    4 +-
 xenserver/etc_init.d_vswitch                       |   18 +-
 .../usr_share_vswitch_scripts_sysconfig.template   |    6 +-
 27 files changed, 515 insertions(+), 147 deletions(-)
 create mode 100644 tests/daemon.at
 create mode 100644 tests/ovsdb-macros.at


hooks/post-receive
-- 
Open vSwitch




More information about the git mailing list