From eed00eb17969791107ddb8df0e909790c17bd573 Mon Sep 17 00:00:00 2001
From: Jason Dillaman <dillaman@redhat.com>
Date: Wed, 8 Apr 2020 17:50:59 -0400
Subject: [PATCH] librbd: additional debug logs for mirror snapshot unlink peer

Signed-off-by: Jason Dillaman <dillaman@redhat.com>
---
 src/librbd/mirror/snapshot/UnlinkPeerRequest.cc | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/librbd/mirror/snapshot/UnlinkPeerRequest.cc b/src/librbd/mirror/snapshot/UnlinkPeerRequest.cc
index 67f51dc4bd654..cb059c760c1ce 100644
--- a/src/librbd/mirror/snapshot/UnlinkPeerRequest.cc
+++ b/src/librbd/mirror/snapshot/UnlinkPeerRequest.cc
@@ -72,12 +72,14 @@ void UnlinkPeerRequest<I>::unlink_peer() {
         &snap_it->second.snap_namespace);
     } else if (boost::get<cls::rbd::MirrorSnapshotNamespace>(
                  &snap_it->second.snap_namespace) != nullptr) {
+      ldout(cct, 20) << "located newer mirror snapshot" << dendl;
       newer_mirror_snapshots = true;
       break;
     }
   }
 
   if (r == -ENOENT) {
+    ldout(cct, 20) << "missing snapshot: snap_id=" << m_snap_id << dendl;
     m_image_ctx->image_lock.unlock_shared();
     finish(r);
     return;
@@ -100,7 +102,8 @@ void UnlinkPeerRequest<I>::unlink_peer() {
   }
   m_image_ctx->image_lock.unlock_shared();
 
-  ldout(cct, 20) << dendl;
+  ldout(cct, 20) << "snap_id=" << m_snap_id << ", "
+                 << "mirror_peer_uuid=" << m_mirror_peer_uuid << dendl;
   librados::ObjectWriteOperation op;
   librbd::cls_client::mirror_image_snapshot_unlink_peer(&op, m_snap_id,
                                                         m_mirror_peer_uuid);
@@ -175,6 +178,7 @@ void UnlinkPeerRequest<I>::remove_snapshot() {
   }
 
   if (r == -ENOENT) {
+    ldout(cct, 20) << "failed to locate snapshot " << m_snap_id << dendl;
     finish(0);
     return;
   }
@@ -185,6 +189,10 @@ void UnlinkPeerRequest<I>::remove_snapshot() {
 
   if (info->mirror_peer_uuids.size() > 1 ||
       info->mirror_peer_uuids.count(m_mirror_peer_uuid) == 0) {
+    ldout(cct, 20) << "skipping removal of snapshot: "
+                   << "snap_id=" << m_snap_id << ": "
+                   << "mirror_peer_uuid=" << m_mirror_peer_uuid << ", "
+                   << "mirror_peer_uuids=" << info->mirror_peer_uuids << dendl;
     finish(0);
     return;
   }
-- 
2.39.5