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 context in reported errors by GetVolumeReplicationInfo #5079

Merged
merged 1 commit into from
Jan 15, 2025

Conversation

nixpanic
Copy link
Member

@nixpanic nixpanic commented Jan 14, 2025

Logged errors are much more helpful when there is some context around
the message about what went wrong.

Depends-on: #5078

@nixpanic nixpanic added cleanup component/rbd Issues related to RBD labels Jan 14, 2025
@nixpanic nixpanic changed the title cleanup/rbd/GetVolumeReplicationInfo rbd: add context in reported errors by GetVolumeReplicationInfo Jan 14, 2025
@nixpanic nixpanic requested a review from a team January 14, 2025 21:11
@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jan 15, 2025

@Mergifyio rebase

Logged errors are much more helpful when there is some context around
the message about what went wrong.

Signed-off-by: Niels de Vos <[email protected]>
Copy link
Contributor

mergify bot commented Jan 15, 2025

rebase

✅ Branch has been successfully rebased

@Madhu-1 Madhu-1 force-pushed the cleanup/rbd/GetVolumeReplicationInfo branch from 1b7ee0d to 2e6faa1 Compare January 15, 2025 06:38
@Madhu-1
Copy link
Collaborator

Madhu-1 commented Jan 15, 2025

@Mergifyio queue

Copy link
Contributor

mergify bot commented Jan 15, 2025

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at e89fe5a

@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Jan 15, 2025
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@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
Copy link
Collaborator

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

@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.32

@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.32

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Jan 15, 2025
@mergify mergify bot merged commit e89fe5a into ceph:devel Jan 15, 2025
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup component/rbd Issues related to RBD
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants