From 4a0da21c35566991c8a804b194959d3e8776ddad Mon Sep 17 00:00:00 2001
From: k8s-merge-robot <k8s.production.user@gmail.com>
Date: Fri, 24 Jun 2016 12:07:50 -0700
Subject: [PATCH] Merge pull request #28030 from nikhiljindal/revertPR

Automatic merge from submit-queue

Revert "Federation e2e supports aws"

Reverting https://github.com/kubernetes/kubernetes/pull/27791 to get our Jenkins tests green again.

cc @kubernetes/sig-cluster-federation

(cherry picked from commit e93608c859db84b063814f95bd186593e40b354c)
---
 cluster/aws/util.sh                           |  2 +-
 federation/cluster/common.sh                  | 24 -------------------
 .../federation-apiserver-deployment.yaml      |  4 ----
 3 files changed, 1 insertion(+), 29 deletions(-)

diff --git a/cluster/aws/util.sh b/cluster/aws/util.sh
index b68dc76868126..0c2c3fc124448 100755
--- a/cluster/aws/util.sh
+++ b/cluster/aws/util.sh
@@ -1491,7 +1491,7 @@ function test-build-release {
 # Assumed vars:
 #   Variables from config.sh
 function test-setup {
-  . "${KUBE_ROOT}/cluster/kube-up.sh"
+  "${KUBE_ROOT}/cluster/kube-up.sh"
 
   VPC_ID=$(get_vpc_id)
   detect-security-groups
diff --git a/federation/cluster/common.sh b/federation/cluster/common.sh
index 3f628291be0d9..7955dd65817f3 100644
--- a/federation/cluster/common.sh
+++ b/federation/cluster/common.sh
@@ -135,30 +135,6 @@ function create-federation-api-objects {
 	    fi
 	    sleep 5
 	done
-
-	# Poll until DNS record for federation-apiserver is propagated
-	# TODO(colhom): REQUIREMENT for any other providers which use a DNS name for routing to loadbalancers
-	# right now it's just AWS that does this.
-	which nslookup
-	set +o errexit
-	if [[ "$KUBERNETES_PROVIDER" == "aws" ]];then
-	    for i in {1..60};do
-		echo "attempting to nslookup ${FEDERATION_API_HOST} ($i / 60)"
-		nslookup "${FEDERATION_API_HOST}"
-		if [[ $? -eq 0 ]];then
-		    echo "Lookup for ${FEDERATION_API_HOST} successful!"
-		    break
-		fi
-		sleep 10
-	    done
-
-	    if [[ $i -eq 60 ]];then
-		echo "Failed to resolve A record for ${FEDERATION_API_HOST}. Will exit"
-		exit 1
-	    fi
-	fi
-	set -o errexit
-
 	KUBE_MASTER_IP="${FEDERATION_API_HOST}:443"
     else
 	echo "provider ${KUBERNETES_PROVIDER} is not (yet) supported for e2e testing"
diff --git a/federation/manifests/federation-apiserver-deployment.yaml b/federation/manifests/federation-apiserver-deployment.yaml
index 7c4bd6ee19b98..5656298b696ce 100644
--- a/federation/manifests/federation-apiserver-deployment.yaml
+++ b/federation/manifests/federation-apiserver-deployment.yaml
@@ -22,11 +22,7 @@ spec:
         - --etcd-servers=http://localhost:2379
         - --service-cluster-ip-range={{.FEDERATION_SERVICE_CIDR}}
         - --secure-port=443
-        {{if eq .KUBERNETES_PROVIDER "aws"}}
-        - --external-hostname={{.FEDERATION_API_HOST}}
-        {{else}}
         - --advertise-address={{.FEDERATION_API_HOST}}
-        {{end}}
         # TODO: --admission-control values must be set when support is added for each type of control.
         - --token-auth-file=/srv/kubernetes/known-tokens.csv
         ports: