From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Gibson To: passt-dev@passt.top Subject: [PATCH 15/15] tests: Remove unused DNS6 calculation from fedora tests Date: Wed, 06 Jul 2022 17:29:09 +1000 Message-ID: <20220706072909.596805-16-david@gibson.dropbear.id.au> In-Reply-To: <20220706072909.596805-1-david@gibson.dropbear.id.au> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0080898356330770721==" --===============0080898356330770721== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable The Fedora test file extracts some information from the host resolv.conf into a DNS6 variable which is then never used. Remove this unnecessary step, which is presumably a leftover from an earlier iteration. This was the only user of 'head' and 'sed' in the test file, so those can also be removed from the required tools. The debian and ubuntu test files also listed 'head' and 'sed' as tools, although they don't use them, I'm guessing because of an earlier version which had the same DNS6 code. Remove those as well. The opensuse test file still actually uses DNS6, so leave it there for now. The DNS handling and network config handling for SuSE looks to be kind of broken, but fixing that is a job for another day. Signed-off-by: David Gibson --- test/distro/debian | 2 +- test/distro/fedora | 3 +-- test/distro/ubuntu | 2 +- 3 files changed, 3 insertions(+), 4 deletions(-) diff --git a/test/distro/debian b/test/distro/debian index ad0ec30..abbbaa2 100644 --- a/test/distro/debian +++ b/test/distro/debian @@ -12,7 +12,7 @@ # Author: Stefano Brivio =20 temp PIDFILE -htools head sed cat kill qemu-system-x86_64 qemu-system-aarch64 qemu-system-= ppc64 +htools cat kill qemu-system-x86_64 qemu-system-aarch64 qemu-system-ppc64 =20 # Quick pasta test: send message from init to ns, and from ns to init def distro_quick_pasta_test diff --git a/test/distro/fedora b/test/distro/fedora index f225a8e..56805c5 100644 --- a/test/distro/fedora +++ b/test/distro/fedora @@ -12,7 +12,7 @@ # Author: Stefano Brivio =20 temp PIDFILE -htools head sed cat kill qemu-system-x86_64 +htools cat kill qemu-system-x86_64 =20 # Quick pasta test: send message from init to ns, and from ns to init def distro_quick_pasta_test @@ -59,7 +59,6 @@ endef hostb ./passt -P __PIDFILE__ & sleep 1 host echo -hout DNS6 sed -n 's/^nameserver \([^:]*:\)\([^%]*\).*/\1\2/p' /etc/resolv.co= nf | head -1 =20 test Fedora 26, x86_64 =20 diff --git a/test/distro/ubuntu b/test/distro/ubuntu index 3a80e37..b9a0446 100644 --- a/test/distro/ubuntu +++ b/test/distro/ubuntu @@ -12,7 +12,7 @@ # Author: Stefano Brivio =20 temp PIDFILE -htools qemu-img virt-edit guestfish head sed cat kill qemu-system-x86_64 qem= u-system-ppc64 qemu-system-s390x +htools qemu-img virt-edit guestfish cat kill qemu-system-x86_64 qemu-system-= ppc64 qemu-system-s390x =20 # Quick pasta test: send message from init to ns, and from ns to init def distro_quick_pasta_test --=20 2.36.1 --===============0080898356330770721==--