ceph / ceph-csi

CSI driver for Ceph
Apache License 2.0
1.19k stars 528 forks source link

cleanup: incorrect fuserecovery logging #4598

Closed iPraveenParihar closed 1 month ago

iPraveenParihar commented 1 month ago

Describe what this PR does

cleanup: incorrect fuserecovery logging This commit make sure that logs fuserecovery.go is only logged when the chosen mount is FUSE.

Checklist:


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/`: retest the `` after unrelated failure (please report the failure too!)
nixpanic commented 1 month ago

@mergifyio rebase

mergify[bot] commented 1 month ago

rebase

✅ Branch has been successfully rebased

nixpanic commented 1 month ago

@mergifyio queue

mergify[bot] commented 1 month ago

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.
ceph-csi-bot commented 1 month ago

/test ci/centos/upgrade-tests-cephfs

ceph-csi-bot commented 1 month ago

/test ci/centos/k8s-e2e-external-storage/1.27

ceph-csi-bot commented 1 month ago

/test ci/centos/upgrade-tests-rbd

ceph-csi-bot commented 1 month ago

/test ci/centos/mini-e2e-helm/k8s-1.27

ceph-csi-bot commented 1 month ago

/test ci/centos/k8s-e2e-external-storage/1.30

ceph-csi-bot commented 1 month ago

/test ci/centos/mini-e2e/k8s-1.27

ceph-csi-bot commented 1 month ago

/test ci/centos/k8s-e2e-external-storage/1.28

ceph-csi-bot commented 1 month ago

/test ci/centos/mini-e2e-helm/k8s-1.30

ceph-csi-bot commented 1 month ago

/test ci/centos/mini-e2e-helm/k8s-1.28

ceph-csi-bot commented 1 month ago

/test ci/centos/k8s-e2e-external-storage/1.29

ceph-csi-bot commented 1 month ago

/test ci/centos/mini-e2e/k8s-1.30

ceph-csi-bot commented 1 month ago

/test ci/centos/mini-e2e/k8s-1.28

ceph-csi-bot commented 1 month ago

/test ci/centos/mini-e2e-helm/k8s-1.29

ceph-csi-bot commented 1 month ago

/test ci/centos/mini-e2e/k8s-1.29

nixpanic commented 1 month ago

/retest ci/centos/k8s-e2e-external-storage/1.29

nixpanic commented 1 month ago

/retest ci/centos/k8s-e2e-external-storage/1.29

logs show 2 failures related to RBD, but this PR only changes CephFS code...

nixpanic commented 1 month ago

@mergifyio requeue

mergify[bot] commented 1 month ago

requeue

✅ The queue state of this pull request has been cleaned. It can be re-embarked automatically