From f91667e364aafa0413dcba9c42f43fa62438dcc4 Mon Sep 17 00:00:00 2001 From: Rohit Ramkumar Date: Mon, 9 Apr 2018 12:54:04 -0700 Subject: [PATCH] Patch ingress upgrade test logic to take note of SNI support in next release. --- test/e2e/upgrades/ingress.go | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/test/e2e/upgrades/ingress.go b/test/e2e/upgrades/ingress.go index a51a8c6a8cb..c804d628d31 100644 --- a/test/e2e/upgrades/ingress.go +++ b/test/e2e/upgrades/ingress.go @@ -177,13 +177,10 @@ func (t *IngressUpgradeTest) verify(f *framework.Framework, done <-chan struct{} postUpgradeResourceStore := &GCPResourceStore{} t.populateGCPResourceStore(postUpgradeResourceStore) - // Ignore certain fields in compute.Firewall that we know will change - // due to the upgrade/downgrade. - // TODO(rramkumar): Remove this once glbc 0.9.8 is released. - t.resourceStore.Fw.Allowed = nil - t.resourceStore.Fw.SourceRanges = nil - postUpgradeResourceStore.Fw.Allowed = nil - postUpgradeResourceStore.Fw.SourceRanges = nil + // Ignore compute.SslCertificates since we know it will change during an upgrade/downgrade. + // TODO(rramkumar): Remove this once glbc 1.1 is released. + t.resourceStore.SslList = nil + postUpgradeResourceStore.Fw.SslList = nil framework.ExpectNoError(compareGCPResourceStores(t.resourceStore, postUpgradeResourceStore, func(v1 reflect.Value, v2 reflect.Value) error { i1 := v1.Interface()