Skip to content

Commit 26442a6

Browse files
authored
Merge pull request kubernetes#129359 from andyzhangx/fix-pv-deletion-timeout
test: fix pv deletion timeout
2 parents 14dafc3 + 176730b commit 26442a6

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

test/e2e/storage/framework/volume_resource.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -220,7 +220,7 @@ func (r *VolumeResource) CleanupResource(ctx context.Context) error {
220220
}
221221

222222
if pv != nil {
223-
err = e2epv.WaitForPersistentVolumeDeleted(ctx, f.ClientSet, pv.Name, 5*time.Second, f.Timeouts.PVDelete)
223+
err = e2epv.WaitForPersistentVolumeDeleted(ctx, f.ClientSet, pv.Name, 5*time.Second, f.Timeouts.PVDeleteSlow)
224224
if err != nil {
225225
cleanUpErrs = append(cleanUpErrs, fmt.Errorf(
226226
"persistent Volume %v not deleted by dynamic provisioner: %w", pv.Name, err))

test/e2e/storage/testsuites/provisioning.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1187,7 +1187,7 @@ func StopPodAndDependents(ctx context.Context, c clientset.Interface, timeouts *
11871187
// As with CSI inline volumes, we use the pod delete timeout here because conceptually
11881188
// the volume deletion needs to be that fast (whatever "that" is).
11891189
framework.Logf("Wait up to %v for pod PV %s to be fully deleted", timeouts.PodDelete, pv.Name)
1190-
framework.ExpectNoError(e2epv.WaitForPersistentVolumeDeleted(ctx, c, pv.Name, 5*time.Second, timeouts.PodDelete))
1190+
framework.ExpectNoError(e2epv.WaitForPersistentVolumeDeleted(ctx, c, pv.Name, 5*time.Second, timeouts.PVDeleteSlow))
11911191
}
11921192
}
11931193
}

0 commit comments

Comments
 (0)