github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-30 15:05:27 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #72410 from pohly/storage-gcepd-config-fix

Browse Source 
e2e/storage: reset driver config in the test suite
This commit is contained in:
Kubernetes Prow Robot 2019-01-03 15:51:44 -08:00 committed by GitHub
commit 47f2f7a53a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 16 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
test/e2e/storage/csi_volumes.go
View File

@ -127,20 +127,23 @@ var _ = utils.SIGDescribe("CSI Volumes", func() {
for _, initDriver := range csiTestDrivers {
curDriver := initDriver(config)
curConfig := curDriver.GetDriverInfo().Config
Context(testsuites.GetDriverNameWithFeatureTags(curDriver), func() {
driver := curDriver
BeforeEach(func() {
// Reset config. The driver might have modified its copy
// in a previous test.
curDriver.GetDriverInfo().Config = curConfig
// setupDriver
driver.CreateDriver()
curDriver.CreateDriver()
})
AfterEach(func() {
// Cleanup driver
driver.CleanupDriver()
curDriver.CleanupDriver()
})
testsuites.RunTestSuite(f, driver, csiTestSuites, csiTunePattern)
testsuites.RunTestSuite(f, curDriver, csiTestSuites, csiTunePattern)
})
}

11
test/e2e/storage/drivers/in_tree.go
View File

@ -151,9 +151,6 @@ func (n *nfsDriver) CreateDriver() {
ns := f.Namespace
n.externalPluginName = fmt.Sprintf("example.com/nfs-%s", ns.Name)
// Reset config. It might have been modified by a previous CreateVolume call.
n.driverInfo.Config.ServerConfig = nil
// TODO(mkimuram): cluster-admin gives too much right but system:persistent-volume-provisioner
// is not enough. We should create new clusterrole for testing.
framework.BindClusterRole(cs.RbacV1beta1(), "cluster-admin", ns.Name,
@ -298,8 +295,6 @@ func (g *glusterFSDriver) GetPersistentVolumeSource(readOnly bool, fsType string
}
func (g *glusterFSDriver) CreateDriver() {
// Reset config. It might have been modified by a previous CreateVolume call.
g.driverInfo.Config.ServerConfig = nil
}
func (g *glusterFSDriver) CleanupDriver() {
@ -429,8 +424,6 @@ func (i *iSCSIDriver) GetPersistentVolumeSource(readOnly bool, fsType string, te
}
func (i *iSCSIDriver) CreateDriver() {
// Reset config. It might have been modified by a previous CreateVolume call.
i.driverInfo.Config.ServerConfig = nil
}
func (i *iSCSIDriver) CleanupDriver() {
@ -556,8 +549,6 @@ func (r *rbdDriver) GetPersistentVolumeSource(readOnly bool, fsType string, test
}
func (r *rbdDriver) CreateDriver() {
// Reset config. It might have been modified by a previous CreateVolume call.
r.driverInfo.Config.ServerConfig = nil
}
func (r *rbdDriver) CleanupDriver() {
@ -670,8 +661,6 @@ func (c *cephFSDriver) GetPersistentVolumeSource(readOnly bool, fsType string, t
}
func (c *cephFSDriver) CreateDriver() {
// Reset config. It might have been modified by a previous CreateVolume call.
c.driverInfo.Config.ServerConfig = nil
}
func (c *cephFSDriver) CleanupDriver() {

13
test/e2e/storage/in_tree_volumes.go
View File

@ -69,20 +69,23 @@ var _ = utils.SIGDescribe("In-tree Volumes", func() {
for _, initDriver := range testDrivers {
curDriver := initDriver(config)
curConfig := curDriver.GetDriverInfo().Config
Context(testsuites.GetDriverNameWithFeatureTags(curDriver), func() {
driver := curDriver
BeforeEach(func() {
// Reset config. The driver might have modified its copy
// in a previous test.
curDriver.GetDriverInfo().Config = curConfig
// setupDriver
driver.CreateDriver()
curDriver.CreateDriver()
})
AfterEach(func() {
// Cleanup driver
driver.CleanupDriver()
curDriver.CleanupDriver()
})
testsuites.RunTestSuite(f, driver, testSuites, intreeTunePattern)
testsuites.RunTestSuite(f, curDriver, testSuites, intreeTunePattern)
})
}
})