Changed using a map to k8s.io/apimachinery/pkg/utils/sets and added a test for adding a hostname

Author: JacobTanenbaum

pkg/ingress/admission/ingress_admission.go

@@ -10,6 +10,7 @@ import (
 	kadmission "k8s.io/kubernetes/pkg/admission"
 	kextensions "k8s.io/kubernetes/pkg/apis/extensions"
 	clientset "k8s.io/kubernetes/pkg/client/clientset_generated/internalclientset"
+	"k8s.io/client-go/pkg/util/sets"
 
 	configlatest "github.com/openshift/origin/pkg/cmd/server/api/latest"
 	"github.com/openshift/origin/pkg/ingress/admission/api"
@@ -80,13 +81,13 @@ func (r *ingressAdmission) Admit(a kadmission.Attributes) error {
 }
 
 func haveHostnamesChanged(oldIngress, newIngress *kextensions.Ingress) bool {
-	m := make(map[string]int)
+	hostnameSet := sets.NewString()
 	for _, element := range oldIngress.Spec.Rules {
-		m[element.Host] = 1
+		hostnameSet.Insert(element.Host)
 	}
 
 	for _, element := range newIngress.Spec.Rules {
-		if _, present := m[element.Host]; !present {
+		if present := hostnameSet.Has(element.Host); !present {
 			return false
 		}
 	}

pkg/ingress/admission/ingress_admission_test.go

@@ -67,6 +67,13 @@ func TestAdmission(t *testing.T) {
 			oldHost:  "bar.com",
 			testName: "Upstream Hostname updates enabled",
 		},
+		{
+			admit:    true,
+			config:   testConfigUpdateAllow(),
+			op:       admission.Update,
+			newHost:  "foo.com",
+			testName: "add new hostname with upstream rules",
+		},
 	}
 	for _, test := range tests {
 		if len(test.newHost) > 0 {