From bebdabc06cd5735dcc487461f566d88bd2ca1363 Mon Sep 17 00:00:00 2001 From: Aram Sargsyan Date: Thu, 19 Sep 2024 14:47:49 +0000 Subject: [PATCH] Fix the nslookup system test The nslookup system test checks the count of resolved addresses in the CNAME tests using a 'grep' match on the hostname, and ignoring lines containing the 'canonical name' string. In order to protect the check from intermittent failures like the 'address in use' warning message, which then automatically resolves after a retry, edit the 'grep' matching string to also ignore the comments (as the mentioned warning message is a comment which contains the hostname). (cherry picked from commit 345b0f9e5ca075243e4ac662b4be299666be3207) --- bin/tests/system/nslookup/tests.sh | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/bin/tests/system/nslookup/tests.sh b/bin/tests/system/nslookup/tests.sh index 6c8e29cda3..85dc1d72f0 100644 --- a/bin/tests/system/nslookup/tests.sh +++ b/bin/tests/system/nslookup/tests.sh @@ -85,7 +85,7 @@ lines=$(grep -c "Server:" nslookup.out${n}) test $lines -eq 1 || ret=1 lines=$(grep -c "canonical name" nslookup.out${n}) test $lines -eq 1 || ret=1 -lines=$(grep a-only.example.net nslookup.out${n} | grep -cv "canonical name") +lines=$(grep a-only.example.net nslookup.out${n} | grep -cEv "(canonical name|^;;)") test $lines -eq 1 || ret=1 grep "1.2.3.4" nslookup.out${n} >/dev/null || ret=1 if [ $ret != 0 ]; then echo_i "failed"; fi @@ -101,7 +101,7 @@ lines=$(grep -c "Server:" nslookup.out${n}) test $lines -eq 1 || ret=1 lines=$(grep -c "canonical name" nslookup.out${n}) test $lines -eq 1 || ret=1 -lines=$(grep aaaa-only.example.net nslookup.out${n} | grep -cv "canonical name") +lines=$(grep aaaa-only.example.net nslookup.out${n} | grep -cEv "(canonical name|^;;)") test $lines -eq 1 || ret=1 grep "2001::ffff" nslookup.out${n} >/dev/null || ret=1 if [ $ret != 0 ]; then echo_i "failed"; fi @@ -117,7 +117,7 @@ lines=$(grep -c "Server:" nslookup.out${n}) test $lines -eq 1 || ret=1 lines=$(grep -c "canonical name" nslookup.out${n}) test $lines -eq 1 || ret=1 -lines=$(grep dual.example.net nslookup.out${n} | grep -cv "canonical name") +lines=$(grep dual.example.net nslookup.out${n} | grep -cEv "(canonical name|^;;)") test $lines -eq 2 || ret=1 grep "1.2.3.4" nslookup.out${n} >/dev/null || ret=1 grep "2001::ffff" nslookup.out${n} >/dev/null || ret=1