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

rbd: add a check for CSI pv #4808

Merged
merged 1 commit into from
Aug 27, 2024
Merged

rbd: add a check for CSI pv #4808

merged 1 commit into from
Aug 27, 2024

Conversation

Madhu-1
Copy link
Collaborator

@Madhu-1 Madhu-1 commented Aug 27, 2024

add a check for CSI as it can be nil for non-csi PV.

fixes: #4807

@mergify mergify bot added the component/rbd Issues related to RBD label Aug 27, 2024
yati1998
yati1998 previously approved these changes Aug 27, 2024
@Madhu-1 Madhu-1 changed the title rbd: add a check for CSI pvc rbd: add a check for CSI pv Aug 27, 2024
@Madhu-1 Madhu-1 requested review from yati1998 and a team August 27, 2024 10:32
@mergify mergify bot dismissed yati1998’s stale review August 27, 2024 10:32

Pull request has been modified.

@nixpanic nixpanic added bug Something isn't working backport-to-release-v3.12 Label to backport from devel to release-v3.12 branch labels Aug 27, 2024
@nixpanic
Copy link
Member

@Mergifyio queue

Copy link
Contributor

mergify bot commented Aug 27, 2024

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 3ac5968

add a check for CSI as it can be
nil for non-csi PV.

fixes: ceph#4807

Signed-off-by: Madhu Rajanna <madhupr007@gmail.com>
@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Aug 27, 2024
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Aug 27, 2024
@mergify mergify bot merged commit 3ac5968 into ceph:devel Aug 27, 2024
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-release-v3.12 Label to backport from devel to release-v3.12 branch bug Something isn't working component/rbd Issues related to RBD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

csi-rbdplugin on the node crashes on start with nil pointer
4 participants