Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

testsetup: Check for DNS working inside VMs #1660

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
7 changes: 6 additions & 1 deletion scripts/qa_crowbarsetup.sh
Expand Up @@ -3893,6 +3893,7 @@ function oncontroller_testsetup
local volumeattachret=0
local volumeresult=""
local portresult=0
local dnsresult=0

# do volume tests for non-docker scenario only
if [ -z "$want_docker" ] ; then
Expand All @@ -3915,6 +3916,9 @@ function oncontroller_testsetup
ssh $ssh_target fdisk -l $device | grep 1073741824 || volumeattachret=57
ssh $ssh_target "mount $device /mnt && grep -q $rand /mnt/test.txt" || volumeattachret=58
volumeresult="$volumecreateret & $volumeattachret"

# Check for DNS available inside instances
ssh $ssh_target getent hosts www.google.com || dnsresult=59
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would fail when deploying an offline cloud. might be nicer to do it thus #1661

else
volumeresult="tests skipped (not supported for docker)"
fi
Expand Down Expand Up @@ -3999,10 +4003,11 @@ function oncontroller_testsetup
echo "RadosGW Tests: $radosgwret"
echo "Tempest: $tempestret"
echo "Volume in VM: $volumeresult"
echo "DNS inside VM: $dnsresult"
echo "Ports in binding_failed: $portresult"

test $tempestret = 0 -a $volumecreateret = 0 -a $volumeattachret = 0 \
-a $radosgwret = 0 -a $portresult = 0 || exit 102
-a $radosgwret = 0 -a $volumeresult = 0 -a $portresult = 0 || exit 102
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

$dnsresult should be checked here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

d'oh, yea.

}


Expand Down