public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
From: Stefano Brivio <sbrivio@redhat.com>
To: David Gibson <david@gibson.dropbear.id.au>
Cc: passt-dev@passt.top
Subject: Re: [PATCH 12/14] test: Use "nstool exec" to slightly simplify tests
Date: Wed, 5 Apr 2023 13:59:12 +0200	[thread overview]
Message-ID: <20230405135912.4b40c5a0@elisabeth> (raw)
In-Reply-To: <20230404014638.3225556-13-david@gibson.dropbear.id.au>

On Tue,  4 Apr 2023 11:46:36 +1000
David Gibson <david@gibson.dropbear.id.au> wrote:

> Using this, rather than using "nstool info" to get the pid then manually
> connecting with nsenter makes things a little simpler.
> 
> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
> ---
>  test/lib/context | 14 +++++++++-----
>  test/lib/setup   | 33 +++++++++++++--------------------
>  test/run         |  2 +-
>  3 files changed, 23 insertions(+), 26 deletions(-)
> 
> diff --git a/test/lib/context b/test/lib/context
> index ee6b683..d9d8260 100644
> --- a/test/lib/context
> +++ b/test/lib/context
> @@ -13,6 +13,8 @@
>  # Copyright Red Hat
>  # Author: David Gibson <david@gibson.dropbear.id.au>
>  
> +NSTOOL="${BASEPATH}/nstool"
> +
>  # context_setup_common() - Create outline of a new context
>  # $1:	Context name
>  context_setup_common() {
> @@ -30,15 +32,17 @@ context_setup_host() {
>  	echo sh -c > "${__enter}"
>  }
>  
> -# context_setup_nsenter() - Create a new context for running commands in a namespace
> +# context_setup_nstool() - Create a new context for running commands with nstool exec
>  # $1:	Context name
> -# $2:	Namespace PID
> -context_setup_nsenter() {
> +# $2:	nstool control socket
> +context_setup_nstool() {
>  	__name="$1"
> -	shift
> +        __sock="$2"
>  	__enter="${STATESETUP}/context_${__name}.enter"
> +        # Wait for the ns to be ready
> +        ${NSTOOL} info -w "${__sock}" > /dev/null

Bad indentation here.

>  	context_setup_common "${__name}"
> -	echo "nsenter $@ sh -c" > "${__enter}"
> +	echo "${NSTOOL} exec ${__sock} -- sh -c" > "${__enter}"
>  }
>  
>  # context_setup_guest() - Create a new context for running commands in a guest
> diff --git a/test/lib/setup b/test/lib/setup
> index 6d7644a..7b17336 100755
> --- a/test/lib/setup
> +++ b/test/lib/setup
> @@ -17,7 +17,6 @@ INITRAMFS="${BASEPATH}/mbuto.img"
>  VCPUS="$( [ $(nproc) -ge 8 ] && echo 6 || echo $(( $(nproc) / 2 + 1 )) )"
>  __mem_kib="$(sed -n 's/MemTotal:[ ]*\([0-9]*\) kB/\1/p' /proc/meminfo)"
>  VMEM="$((${__mem_kib} / 1024 / 4))"
> -NSTOOL="${BASEPATH}/nstool"
>  
>  # setup_build() - Set up pane layout for build tests
>  setup_build() {
> @@ -78,9 +77,8 @@ setup_pasta() {
>  	layout_pasta
>  
>  	context_run_bg unshare "unshare -rUnpf ${NSTOOL} hold ${STATESETUP}/ns.hold"
> -	__target_pid=$(${NSTOOL} info -pw ${STATESETUP}/ns.hold)
>  
> -	context_setup_nsenter ns -U -n -p --preserve-credentials -t ${__target_pid}
> +	context_setup_nstool ns ${STATESETUP}/ns.hold
>  
>  	# Ports:
>  	#
> @@ -94,7 +92,7 @@ setup_pasta() {
>  	[ ${DEBUG} -eq 1 ] && __opts="${__opts} -d"
>  	[ ${TRACE} -eq 1 ] && __opts="${__opts} --trace"
>  
> -	context_run_bg passt "./pasta ${__opts} -f -t 10002 -T 10003 -u 10002 -U 10003 -P ${STATESETUP}/passt.pid ${__target_pid}"
> +	context_run_bg passt "./pasta ${__opts} -f -t 10002 -T 10003 -u 10002 -U 10003 -P ${STATESETUP}/passt.pid $(${NSTOOL} info -pw ${STATESETUP}/ns.hold)"
>  
>  	# pidfile isn't created until pasta is ready
>  	wait_for [ -f "${STATESETUP}/passt.pid" ]
> @@ -128,11 +126,10 @@ setup_passt_in_ns() {
>  
>  	context_run_bg pasta "./pasta ${__opts} -t 10001,10002,10011,10012 -T 10003,10013 -u 10001,10002,10011,10012 -U 10003,10013 -P ${STATESETUP}/pasta.pid --config-net ${NSTOOL} hold ${STATESETUP}/ns.hold"
>  	wait_for [ -f "${STATESETUP}/pasta.pid" ]
> -	__ns_pid=$(${NSTOOL} info -pw ${STATESETUP}/ns.hold)
>  
> -	context_setup_nsenter qemu "-t ${__ns_pid} -U -n -p --preserve-credentials"
> -	context_setup_nsenter ns "-t ${__ns_pid} -U -n -p --preserve-credentials"
> -	context_setup_nsenter passt "-t ${__ns_pid} -U -n -p --preserve-credentials"
> +	context_setup_nstool qemu ${STATESETUP}/ns.hold
> +	context_setup_nstool ns ${STATESETUP}/ns.hold
> +	context_setup_nstool passt ${STATESETUP}/ns.hold
>  
>  	__opts=
>  	[ ${PCAP} -eq 1 ] && __opts="${__opts} -p ${LOGDIR}/passt_in_pasta.pcap"
> @@ -190,19 +187,17 @@ setup_two_guests() {
>  	[ ${DEBUG} -eq 1 ] && __opts="${__opts} -d"
>  	[ ${TRACE} -eq 1 ] && __opts="${__opts} --trace"
>  	context_run_bg pasta_1 "./pasta ${__opts} --trace -l /tmp/pasta1.log -P ${STATESETUP}/pasta_1.pid -t 10001,10002 -T 10003,10004 -u 10001,10002 -U 10003,10004 --config-net ${NSTOOL} hold ${STATESETUP}/ns1.hold"
> -	__ns1_pid=$(${NSTOOL} info -pw ${STATESETUP}/ns1.hold)
> -	context_setup_nsenter passt_1 -U -n -p --preserve-credentials -t ${__ns1_pid}
> +	context_setup_nstool passt_1 ${STATESETUP}/ns1.hold
>  
>  	__opts=
>  	[ ${PCAP} -eq 1 ] && __opts="${__opts} -p ${LOGDIR}/pasta_2.pcap"
>  	[ ${DEBUG} -eq 1 ] && __opts="${__opts} -d"
>  	[ ${TRACE} -eq 1 ] && __opts="${__opts} --trace"
>  	context_run_bg pasta_2 "./pasta ${__opts} --trace -l /tmp/pasta2.log -P ${STATESETUP}/pasta_2.pid -t 10004,10005 -T 10003,10001 -u 10004,10005 -U 10003,10001 --config-net ${NSTOOL} hold ${STATESETUP}/ns2.hold"
> -	__ns2_pid=$(${NSTOOL} info -pw ${STATESETUP}/ns2.hold)
> -	context_setup_nsenter passt_2 -U -n -p --preserve-credentials -t ${__ns2_pid}
> +	context_setup_nstool passt_2 ${STATESETUP}/ns2.hold
>  
> -	context_setup_nsenter qemu_1 -U -n -p --preserve-credentials -t ${__ns1_pid}
> -	context_setup_nsenter qemu_2 -U -n -p --preserve-credentials -t ${__ns2_pid}
> +	context_setup_nstool qemu_1 ${STATESETUP}/ns1.hold
> +	context_setup_nstool qemu_2 ${STATESETUP}/ns2.hold
>  
>  	__ifname="$(context_run qemu_1 "ip -j link show | jq -rM '.[] | select(.link_type == \"ether\").ifname'")"
>  
> @@ -310,15 +305,13 @@ teardown_passt_in_ns() {
>  
>  # teardown_two_guests() - Exit namespaces, kill qemu processes, passt and pasta
>  teardown_two_guests() {
> -	__ns1_pid=$(${NSTOOL} info -pw "${STATESETUP}/ns1.hold")
> -	__ns2_pid=$(${NSTOOL} info -pw "${STATESETUP}/ns2.hold")
> -	nsenter -U -p --preserve-credentials -t ${__ns1_pid} kill $(cat "${STATESETUP}/qemu_1.pid")
> -	nsenter -U -p --preserve-credentials -t ${__ns2_pid} kill $(cat "${STATESETUP}/qemu_2.pid")
> +        ${NSTOOL} exec ${STATESETUP}/ns1.hold -- kill $(cat "${STATESETUP}/qemu_1.pid")
> +        ${NSTOOL} exec ${STATESETUP}/ns2.hold -- kill $(cat "${STATESETUP}/qemu_2.pid")
>  	context_wait qemu_1
>  	context_wait qemu_2
>  
> -	nsenter -U -p --preserve-credentials -t ${__ns1_pid} kill $(cat "${STATESETUP}/passt_1.pid")
> -	nsenter -U -p --preserve-credentials -t ${__ns2_pid} kill $(cat "${STATESETUP}/passt_2.pid")
> +        ${NSTOOL} exec ${STATESETUP}/ns1.hold -- kill $(cat "${STATESETUP}/passt_1.pid")
> +        ${NSTOOL} exec ${STATESETUP}/ns2.hold -- kill $(cat "${STATESETUP}/passt_2.pid")

...and here.

-- 
Stefano


  reply	other threads:[~2023-04-05 11:59 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-04-04  1:46 [PATCH 00/14] Improved tool for testing across multiple namespaces David Gibson
2023-04-04  1:46 ` [PATCH 01/14] nstool: Rename nsholder to nstool David Gibson
2023-04-04  1:46 ` [PATCH 02/14] nstool: Reverse parameters " David Gibson
2023-04-04  1:46 ` [PATCH 03/14] nstool: Move description of its operation modes from comment to usage David Gibson
2023-04-05 11:56   ` Stefano Brivio
2023-04-06  0:10     ` David Gibson
2023-04-04  1:46 ` [PATCH 04/14] nstool: Split some command line parsing and socket setup to subcommands David Gibson
2023-04-05 11:58   ` Stefano Brivio
2023-04-06  2:31     ` David Gibson
2023-04-06  6:47       ` Stefano Brivio
2023-04-06 10:35         ` David Gibson
2023-04-04  1:46 ` [PATCH 05/14] nstool: Replace "pid" subcommand with "info" subcommand David Gibson
2023-04-05 11:58   ` Stefano Brivio
2023-04-06  0:13     ` David Gibson
2023-04-04  1:46 ` [PATCH 06/14] nstool: Detect what namespaces target is in David Gibson
2023-04-05 11:58   ` Stefano Brivio
2023-04-06  0:15     ` David Gibson
2023-04-04  1:46 ` [PATCH 07/14] nstool: Add magic number to advertized information David Gibson
2023-04-05 11:58   ` Stefano Brivio
2023-04-06  0:18     ` David Gibson
2023-04-04  1:46 ` [PATCH 08/14] nstool: Helpers to iterate through namespace types David Gibson
2023-04-05 11:58   ` Stefano Brivio
2023-04-06  0:55     ` David Gibson
2023-04-04  1:46 ` [PATCH 09/14] nstool: Add nstool exec command to execute commands in an nstool namespace David Gibson
2023-04-05 11:58   ` Stefano Brivio
2023-04-06  0:59     ` David Gibson
2023-04-04  1:46 ` [PATCH 10/14] nstool: Add --keep-caps option to nstool exec David Gibson
2023-04-05 11:59   ` Stefano Brivio
2023-04-06  1:11     ` David Gibson
2023-04-04  1:46 ` [PATCH 11/14] test: Initialise ${TRACE} properly David Gibson
2023-04-04  1:46 ` [PATCH 12/14] test: Use "nstool exec" to slightly simplify tests David Gibson
2023-04-05 11:59   ` Stefano Brivio [this message]
2023-04-06  1:14     ` David Gibson
2023-04-04  1:46 ` [PATCH 13/14] nstool: Advertise the holder's cwd (in its mountns) across the socket David Gibson
2023-04-04  1:46 ` [PATCH 14/14] nstool: Enter holder's cwd when changing mount ns with nstool exec David Gibson
2023-04-05 12:01 ` [PATCH 00/14] Improved tool for testing across multiple namespaces Stefano Brivio

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230405135912.4b40c5a0@elisabeth \
    --to=sbrivio@redhat.com \
    --cc=david@gibson.dropbear.id.au \
    --cc=passt-dev@passt.top \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://passt.top/passt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).