Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add check/wait for PVC deletion after pod deletion in ephemeral volume support e2e case #3910

Closed
Rakshith-R opened this issue Jun 16, 2023 · 2 comments · Fixed by #4058
Closed
Assignees
Labels
bug Something isn't working

Comments

@Rakshith-R
Copy link
Contributor

Describe the bug

ephemeral volume support e2e fails rarely when pvc isn't immediately deleted by kublet after pod deletion.

A step needs to be added at line 811 below to wait for pvc deletion with timeout.

ceph-csi/e2e/rbd.go

Lines 793 to 819 in 37f1d72

By("verify generic ephemeral volume support", func() {
// create application
app, err := loadApp(appEphemeralPath)
if err != nil {
framework.Failf("failed to load application: %v", err)
}
app.Namespace = f.UniqueName
err = createApp(f.ClientSet, app, deployTimeout)
if err != nil {
framework.Failf("failed to create application: %v", err)
}
// validate created backend rbd images
validateRBDImageCount(f, 1, defaultRBDPool)
validateOmapCount(f, 1, rbdType, defaultRBDPool, volumesType)
err = deletePod(app.Name, app.Namespace, f.ClientSet, deployTimeout)
if err != nil {
framework.Failf("failed to delete application: %v", err)
}
// validate created backend rbd images
validateRBDImageCount(f, 0, defaultRBDPool)
validateOmapCount(f, 0, rbdType, defaultRBDPool, volumesType)
// validate images in trash
err = waitToRemoveImagesFromTrash(f, defaultRBDPool, deployTimeout)
if err != nil {
framework.Failf("failed to validate rbd images in pool %s trash: %v", defaultRBDPool, err)
}
})

similar step needs to be added to cephfs e2e too.

Failed ci:

ctrl+f pvc-3bff7e9f-5dea-48f8-8c97-92d8b0b942f5

@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the wontfix This will not be worked on label Jul 16, 2023
@riya-singhal31 riya-singhal31 removed the wontfix This will not be worked on label Jul 17, 2023
@riya-singhal31 riya-singhal31 self-assigned this Jul 17, 2023
@github-actions
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in a week if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the wontfix This will not be worked on label Aug 16, 2023
@riya-singhal31 riya-singhal31 removed the wontfix This will not be worked on label Aug 17, 2023
@riya-singhal31 riya-singhal31 added this to the release-v3.10.0 milestone Aug 22, 2023
@mergify mergify bot closed this as completed in #4058 Aug 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants