[ovs-git] [openvswitch/ovs] 8a16ab: tests: Fix issue in use of OVS_APP_EXIT_AND_WAIT.
GitHub
noreply at github.com
Thu Jun 23 23:05:18 UTC 2016
Branch: refs/heads/master
Home: https://github.com/openvswitch/ovs
Commit: 8a16ab90b27ca28fbc5e8a42a02eb8c7133183ae
https://github.com/openvswitch/ovs/commit/8a16ab90b27ca28fbc5e8a42a02eb8c7133183ae
Author: Lance Richardson <lrichard at redhat.com>
Date: 2016-06-23 (Thu, 23 Jun 2016)
Changed paths:
M tests/daemon.at
M tests/ovn-controller.at
M tests/ovn-sbctl.at
M tests/ovs-macros.at
M tests/ovs-vsctl.at
M tests/ovs-vswitchd.at
M tests/ovsdb-idl.at
M tests/ovsdb-monitor.at
M tests/ovsdb-server.at
Log Message:
-----------
tests: Fix issue in use of OVS_APP_EXIT_AND_WAIT.
Commit f9b11f2a09b4 introduced a loop to wait for process exit
in OVS_APP_EXIT_AND_WAIT after the "exit" command has been sent.
Unfortunately, this does not work for cases where a unixctl socket
has to be used to send the "exit" command because the process
ID cannot be determined from the socket path.
OVS_APP_EXIT_AND_WAIT_BY_TARGET has since been introduced to enable
graceful termination of daemons via unixctl sockets.
This set of changes addresses the problem described above by
making OVS_APP_EXIT_AND_WAIT_BY_TARGET take the unixctl socket
path and corresponding process ID as separate parameters. In order
to better detect issues in this logic in the future, checks have
been added to verify that the pidfile exists before using its
contents.
Tested on a Linux system.
Signed-off-by: Lance Richardson <lrichard at redhat.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>
More information about the git
mailing list