osd: scrub: no need to expect snapdir objects

Signed-off-by: Sage Weil <sage@redhat.com>

diff --git a/src/osd/PG.cc b/src/osd/PG.cc
index 09592e3ede83e415ae4da2a033db6a48d58bfa59..e9bbba46cd23dbfe92edf2753811e9c0a70c83fa 100644 (file)
--- a/src/osd/PG.cc
+++ b/src/osd/PG.cc
@@ -3834,7 +3834,8 @@ void PG::_scan_snaps(ScrubMap &smap)
     const hobject_t &hoid = i->first;
     ScrubMap::object &o = i->second;
 
-    if (hoid.is_head() || hoid.is_snapdir()) {
+    assert(!hoid.is_snapdir());
+    if (hoid.is_head()) {
       // parse the SnapSet
       bufferlist bl;
       if (o.attrs.find(SS_ATTR) == o.attrs.end()) {

diff --git a/src/osd/PGBackend.cc b/src/osd/PGBackend.cc
index 468e29297d0de41e14bad096c23d4ee82d2219ed..eb1e0055f39d9e9204f2432f89f990c0a300bf1b 100644 (file)
--- a/src/osd/PGBackend.cc
+++ b/src/osd/PGBackend.cc
@@ -820,7 +820,8 @@ map<pg_shard_t, ScrubMap *>::const_iterator
     }
 
     // We won't pick an auth copy if the snapset is missing or won't decode.
-    if (obj.is_head() || obj.is_snapdir()) {
+    assert(!obj.is_snapdir());
+    if (obj.is_head()) {
       k = i->second.attrs.find(SS_ATTR);
       if (k == i->second.attrs.end()) {
        shard_info.set_ss_attr_missing();