ceph-csi
ceph-csi copied to clipboard
rbd: include trashed parent images while calculating the clone depth
The getCloneDepth() function did not account for images that are in
the trash. A trashed image can only be opened by the image-id, and not
by name anymore.
Closes: #4013 Depends-on: #4064 #4273
Show available bot commands
These commands are normally not required, but in case of issues, leave any of the following bot commands in an otherwise empty comment in this PR:
/retest ci/centos/<job-name>: retest the<job-name>after unrelated failure (please report the failure too!)
/test ci/centos/mini-e2e/k8s-1.27
This pull request now has conflicts with the target branch. Could you please resolve conflicts and force push the corrected changes? 🙏
/test ci/centos/mini-e2e/k8s-1.27
/test ci/centos/mini-e2e/k8s-1.27
Manual stress testing with the scripts from rook/rook#12312 passes.
Need to fix the golangci-lint issues and have the go-ceph rebase merged before this is completely ready.
@mergifyio rebase
rebase
✅ Branch has been successfully rebased
@mergifyio refresh
refresh
✅ Pull request refreshed
@mergifyio queue
queue
🛑 The pull request has been removed from the queue default
The queue conditions cannot be satisfied due to failing checks.
You can take a look at Queue: Embarked in merge queue check runs for more details.
In case of a failure due to a flaky test, you should first retrigger the CI.
Then, re-embark the pull request into the merge queue by posting the comment
@mergifyio refresh on the pull request.
/test ci/centos/upgrade-tests-cephfs
/test ci/centos/upgrade-tests-rbd
/test ci/centos/k8s-e2e-external-storage/1.27
/test ci/centos/mini-e2e-helm/k8s-1.27
/test ci/centos/mini-e2e/k8s-1.27
/test ci/centos/k8s-e2e-external-storage/1.26
/test ci/centos/mini-e2e-helm/k8s-1.26
/test ci/centos/k8s-e2e-external-storage/1.28
/test ci/centos/mini-e2e/k8s-1.26
/test ci/centos/mini-e2e-helm/k8s-1.28
/test ci/centos/mini-e2e/k8s-1.28
/test ci/centos/upgrade-tests-rbd
/test ci/centos/mini-e2e-helm/k8s-1.28
/test ci/centos/mini-e2e/k8s-1.26
/test ci/centos/mini-e2e/k8s-1.27
It seems that there was some issue with snapshots. not sure why e2e passed earlier?!
/test ci/centos/mini-e2e-helm/k8s-1.28
e2e without the last go fmt fix is available at https://jenkins-ceph-csi.apps.ocp.cloud.ci.centos.org/blue/organizations/jenkins/mini-e2e-helm_k8s-1.28/detail/mini-e2e-helm_k8s-1.28/78/pipeline . This PR can be reviewed once that passed.
/test ci/centos/mini-e2e-helm/k8s-1.28