Merge pull request #322 from mrunalp/remove_host_ping_test

Remove host ping test

test/helpers.bash

@@ -245,14 +245,6 @@ function parse_pod_ip() {
 	done
 }
 
-function ping_host_pod() {
-	pod_ip=`ocic pod status --id $1 | grep "IP Address" | cut -d ' ' -f 3`
-
-	ping -W 1 -c 5 $pod_ip
-
-	echo $?
-}
-
 function ping_pod() {
 	netns=`ocic pod status --id $1 | grep namespace | cut -d ' ' -f 3`
 	inet=`ip netns exec \`basename $netns\` ip addr show dev eth0 scope global | grep inet`

test/network.bats

@@ -60,37 +60,6 @@ load helpers
 	stop_ocid
 }
 
-@test "Ping host pod from the host" {
-	# this test requires docker, thus it can't yet be run in a container
-	if [ "$TRAVIS" = "true" ]; then # instead of $TRAVIS, add a function is_containerized to skip here
-		skip "cannot yet run this test in a container, use sudo make localintegration"
-	fi
-
-	if [ ! -f "$OCID_CNI_PLUGIN/bridge" ]; then
-		skip "missing CNI bridge plugin, please install it"
-	fi
-
-	if [ ! -f "$OCID_CNI_PLUGIN/host-local" ]; then
-		skip "missing CNI host-local IPAM, please install it"
-	fi
-
-	prepare_network_conf $POD_CIDR
-
-	start_ocid
-	run ocic pod run --config "$TESTDATA"/sandbox_config_hostnet.json
-	echo "$output"
-	[ "$status" -eq 0 ]
-	pod_id="$output"
-
-	ping_host_pod $pod_id
-
-	cleanup_pods
-	cleanup_network_conf
-
-	stop_ocid
-}
-
-
 @test "Ping pod from another pod" {
 	# this test requires docker, thus it can't yet be run in a container
 	if [ "$TRAVIS" = "true" ]; then # instead of $TRAVIS, add a function is_containerized to skip here