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
Conversation
To our surprise we learned yesterday that in some occassions DNS lookups ot the outside is not working with Cloud7, and we seem to have no check for that. Adding a test for this
There was a check for that in line 3884, but at some point I pointed that to $clouddata which happened to be an IP resolved outside of the VM. |
@@ -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 |
There was a problem hiding this comment.
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
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 |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
d'oh, yea.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please close in favour of PR #1661
PR #1661 was merged, closing this. |
To our surprise we learned yesterday that in some occassions
DNS lookups ot the outside is not working with Cloud7, and we
seem to have no check for that. Adding a test for this