osd: remove bad assertion in pick_read_snap

The clone oid.snap does not necessary correspond to the newest
snap, since snaps may be deleted, or because the snap the snap is
named based on the snap context seq and not the oldest snap it
contains.

diff --git a/src/osd/ReplicatedPG.cc b/src/osd/ReplicatedPG.cc
index 4c2b768054ce946d0df33784a9c67b9af87165b4..814461ef58352c2696b282a693baded299736657 100644 (file)
--- a/src/osd/ReplicatedPG.cc
+++ b/src/osd/ReplicatedPG.cc
@@ -617,14 +617,13 @@ bool ReplicatedPG::pick_read_snap(pobject_t& poid)
   dout(20) << "pick_read_snap  " << poid << " snaps " << snaps << dendl;
   snapid_t first = snaps[snaps.size()-1];
   snapid_t last = snaps[0];
-  assert(last == poid.oid.snap);
   if (first <= want) {
     dout(20) << "pick_read_snap  " << poid << " [" << first << "," << last << "] contains " << want << " -- HIT" << dendl;
     return true;
+  } else {
+    dout(20) << "pick_read_snap  " << poid << " [" << first << "," << last << "] does not contain " << want << " -- DNE" << dendl;
+    return false;
   }
-
-  dout(20) << "pick_read_snap  " << poid << " [" << first << "," << last << "] does not contain " << want << " -- DNE" << dendl;
-  return false;
 }