p4u1/docker-cli
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

integ-cli: remove ICMP dependency from test, use nslookup

Browse Source 
As we started running CI machines on Azure cloud and Azure
doesn't have ICMP stack implemented by replacing
`ping 8.8.8.8` with `nslookup google.com`.

Signed-off-by: Ahmet Alp Balkan <ahmetalpbalkan@gmail.com>
Upstream-commit: 004ac85aa25f08e350465140ee7404ad31602d75
Component: engine
This commit is contained in:
Ahmet Alp Balkan
2015-03-10 22:09:09 -07:00
parent 0cbd226ff4
commit 4ae0ae0aee
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
components/engine/integration-cli/docker_cli_run_test.go
View File

@ -144,18 +144,17 @@ func TestRunLeakyFileDescriptors(t *testing.T) {
logDone("run - check file descriptor leakage")
}
// it should be possible to ping Google DNS resolver
// it should be possible to lookup Google DNS
// this will fail when Internet access is unavailable
func TestRunPingGoogle(t *testing.T) {
func TestRunLookupGoogleDns(t *testing.T) {
defer deleteAllContainers()
runCmd := exec.Command(dockerBinary, "run", "busybox", "ping", "-c", "1", "8.8.8.8")
out, _, _, err := runCommandWithStdoutStderr(runCmd)
out, _, _, err := runCommandWithStdoutStderr(exec.Command(dockerBinary, "run", "busybox", "nslookup", "google.com"))
if err != nil {
t.Fatalf("failed to run container: %v, output: %q", err, out)
}
logDone("run - ping 8.8.8.8")
logDone("run - nslookup google.com")
}
// the exit code should be 0