Unverified Commit f91667e3 authored by Rohit Ramkumar's avatar Rohit Ramkumar Committed by GitHub

Patch ingress upgrade test logic to take note of SNI support in next release.

parent c820e31b
...@@ -177,13 +177,10 @@ func (t *IngressUpgradeTest) verify(f *framework.Framework, done <-chan struct{} ...@@ -177,13 +177,10 @@ func (t *IngressUpgradeTest) verify(f *framework.Framework, done <-chan struct{}
postUpgradeResourceStore := &GCPResourceStore{} postUpgradeResourceStore := &GCPResourceStore{}
t.populateGCPResourceStore(postUpgradeResourceStore) t.populateGCPResourceStore(postUpgradeResourceStore)
// Ignore certain fields in compute.Firewall that we know will change // Ignore compute.SslCertificates since we know it will change during an upgrade/downgrade.
// due to the upgrade/downgrade. // TODO(rramkumar): Remove this once glbc 1.1 is released.
// TODO(rramkumar): Remove this once glbc 0.9.8 is released. t.resourceStore.SslList = nil
t.resourceStore.Fw.Allowed = nil postUpgradeResourceStore.Fw.SslList = nil
t.resourceStore.Fw.SourceRanges = nil
postUpgradeResourceStore.Fw.Allowed = nil
postUpgradeResourceStore.Fw.SourceRanges = nil
framework.ExpectNoError(compareGCPResourceStores(t.resourceStore, postUpgradeResourceStore, func(v1 reflect.Value, v2 reflect.Value) error { framework.ExpectNoError(compareGCPResourceStores(t.resourceStore, postUpgradeResourceStore, func(v1 reflect.Value, v2 reflect.Value) error {
i1 := v1.Interface() i1 := v1.Interface()
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment