Skip to content

Commit b8c4b80

Browse files
committed
Removed clientset param from InjectContent
As one of previous changes added framework param to InjectContent, it does not need separate param for clientset any more.
1 parent ab8444d commit b8c4b80

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

test/e2e/framework/volume/fixtures.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -587,19 +587,19 @@ func TestVolumeClient(f *framework.Framework, config TestConfig, fsGroup *int64,
587587
// InjectContent inserts index.html with given content into given volume. It does so by
588588
// starting and auxiliary pod which writes the file there.
589589
// The volume must be writable.
590-
func InjectContent(f *framework.Framework, client clientset.Interface, config TestConfig, fsGroup *int64, fsType string, tests []Test) {
590+
func InjectContent(f *framework.Framework, config TestConfig, fsGroup *int64, fsType string, tests []Test) {
591591
privileged := true
592592
if framework.NodeOSDistroIs("windows") {
593593
privileged = false
594594
}
595-
injectorPod, err := runVolumeTesterPod(client, config, "injector", privileged, fsGroup, tests)
595+
injectorPod, err := runVolumeTesterPod(f.ClientSet, config, "injector", privileged, fsGroup, tests)
596596
if err != nil {
597597
framework.Failf("Failed to create injector pod: %v", err)
598598
return
599599
}
600600
defer func() {
601-
e2epod.DeletePodOrFail(client, injectorPod.Namespace, injectorPod.Name)
602-
e2epod.WaitForPodToDisappear(client, injectorPod.Namespace, injectorPod.Name, labels.Everything(), framework.Poll, framework.PodDeleteTimeout)
601+
e2epod.DeletePodOrFail(f.ClientSet, injectorPod.Namespace, injectorPod.Name)
602+
e2epod.WaitForPodToDisappear(f.ClientSet, injectorPod.Namespace, injectorPod.Name, labels.Everything(), framework.Poll, framework.PodDeleteTimeout)
603603
}()
604604

605605
ginkgo.By("Writing text file contents in the container.")

test/e2e/storage/testsuites/volumes.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ func (t *volumesTestSuite) defineTests(driver TestDriver, pattern testpatterns.T
178178
// local), plugin skips setting fsGroup if volume is already mounted
179179
// and we don't have reliable way to detect volumes are unmounted or
180180
// not before starting the second pod.
181-
volume.InjectContent(f, f.ClientSet, config, fsGroup, pattern.FsType, tests)
181+
volume.InjectContent(f, config, fsGroup, pattern.FsType, tests)
182182
if driver.GetDriverInfo().Capabilities[CapPersistence] {
183183
volume.TestVolumeClient(f, config, fsGroup, pattern.FsType, tests)
184184
} else {

0 commit comments

Comments
 (0)