From 6d73de1154a413d5b7d24e30d111597e6f4c4f10 Mon Sep 17 00:00:00 2001 From: Matthew Wong Date: Tue, 12 Feb 2019 10:18:18 -0500 Subject: [PATCH] Fix glusterfs e2e not using correct storageclass --- test/e2e/storage/testsuites/provisioning.go | 1 + test/e2e/storage/volume_provisioning.go | 1 + 2 files changed, 2 insertions(+) diff --git a/test/e2e/storage/testsuites/provisioning.go b/test/e2e/storage/testsuites/provisioning.go index 4c03b00280..b884019d7a 100644 --- a/test/e2e/storage/testsuites/provisioning.go +++ b/test/e2e/storage/testsuites/provisioning.go @@ -227,6 +227,7 @@ func testProvisioning(input *provisioningTestInput) { func TestDynamicProvisioning(t StorageClassTest, client clientset.Interface, claim *v1.PersistentVolumeClaim, class *storage.StorageClass) *v1.PersistentVolume { var err error if class != nil { + Expect(*claim.Spec.StorageClassName).To(Equal(class.Name)) By("creating a StorageClass " + class.Name) _, err = client.StorageV1().StorageClasses().Create(class) // The "should provision storage with snapshot data source" test already has created the class. diff --git a/test/e2e/storage/volume_provisioning.go b/test/e2e/storage/volume_provisioning.go index 983c507040..20c23ff0ad 100644 --- a/test/e2e/storage/volume_provisioning.go +++ b/test/e2e/storage/volume_provisioning.go @@ -805,6 +805,7 @@ var _ = utils.SIGDescribe("Dynamic Provisioning", func() { By("creating a claim object with a suffix for gluster dynamic provisioner") claim := newClaim(test, ns, suffix) + claim.Spec.StorageClassName = &class.Name testsuites.TestDynamicProvisioning(test, c, claim, class) })