rbd-mirror: suppress unnecessary log message during snapshot unlink

remote snapshots without a mirror peer UUID are filtered out early. Once the
peer UUID is removed from a remote snapshot, it no longer appears in
m_remote_group_snaps locally. As a result, mirror_group_snapshot_unlink_peer()
will not find that snapshot in m_remote_group_snaps, and this condition is
most expected now.

avoid printing the log message that warns about the snapshot not being present,
as this is not the true case (as it is filtered) and can be misleading.
Moreover, the message would otherwise be printed repeatedly until the local
snapshot is eventually removed, creating unnecessary noise in the logs.

Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>

diff --git a/src/tools/rbd_mirror/group_replayer/Replayer.cc b/src/tools/rbd_mirror/group_replayer/Replayer.cc
index 005fa28a46c1375fbc72ca6c0345b3077213338d..9db3596781a334bcf90e88174a6169ab13395a1c 100644 (file)
--- a/src/tools/rbd_mirror/group_replayer/Replayer.cc
+++ b/src/tools/rbd_mirror/group_replayer/Replayer.cc
@@ -1619,8 +1619,6 @@ void Replayer<I>::mirror_group_snapshot_unlink_peer(const std::string &snap_id)
       });
 
   if (remote_snap == m_remote_group_snaps.end()) {
-    derr << "remote group snapshot not found: "
-         << snap_id << dendl;
     return;
   }