[ovs-dev] [PATCH 2/6] tests: Remove useless run-if-fail commands passed to AT_CHECK

Ben Pfaff blp at ovn.org
Wed Apr 18 18:20:54 UTC 2018


On Wed, Apr 18, 2018 at 06:01:11PM +0200, Jakub Sitnicki wrote:
> Path to ovsdb-server's pidfile has changed long ago when
> ovsdb_start_idltest() helper was introduced in commit 561205007e17
> ("tests: Get rid of overly specific --pidfile and --unixctl options.")
> but the run-if-fail commands were left behind.
> 
> Besides, we don't need to kill the ovs-db-server from the AT_CHECK
> anymore since ovsdb_start_idltest() registers an on_exit hook that will
> do it.
> 
> Clean up any run-if-fail commands that attempt to kill ovsdb-server
> using an invalid pidfile.
> 
> Signed-off-by: Jakub Sitnicki <jkbs at redhat.com>

Good spotting, thanks.  I applied this to master.


More information about the dev mailing list