[ovs-dev] [PATCH v2] tests: Queue for termination all OVSDB IDL pids
Ilya Maximets
i.maximets at ovn.org
Fri Oct 9 15:11:09 UTC 2020
On 9/23/20 1:22 PM, Alin Gabriel Serdean wrote:
> When running OVSDB cluster tests on Windows not all the ovsdb processes are terminated.
> Queue up the pids of the started processes for termination when the test stops.
>
> Signed-off-by: Alin Gabriel Serdean <aserdean at ovn.org>
> ---
> v2: Address Ilya's comments
> ---
> tests/ovsdb-idl.at | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Thanks!
Applied to master and backported down to 2.12.
Best regards, Ilya Maximets.
>
> diff --git a/tests/ovsdb-idl.at b/tests/ovsdb-idl.at
> index 261f4f323..b46258591 100644
> --- a/tests/ovsdb-idl.at
> +++ b/tests/ovsdb-idl.at
> @@ -43,13 +43,13 @@ m4_define([OVSDB_CLUSTER_START_IDLTEST],
> AT_CHECK([ovsdb-tool join-cluster s$i.db \
> $schema_name unix:s$i.raft unix:s1.raft])
> done
> + on_exit 'kill $(cat s*.pid)'
> for i in $(seq $n); do
> AT_CHECK([ovsdb-server -vraft -vconsole:warn --detach --no-chdir \
> --log-file=s$i.log --pidfile=s$i.pid --unixctl=s$i \
> --remote=punix:s$i.ovsdb \
> m4_if([$2], [], [], [--remote=$2]) s$i.db])
> done
> - on_exit 'kill $(cat s*.pid)'
>
> for i in $(seq $n); do
> OVS_WAIT_UNTIL([ovs-appctl -t $(pwd)/s$i cluster/status ${schema_name} \
>
More information about the dev
mailing list