From: Sage Weil <sage@redhat.com>
Date: Mon, 12 Jun 2017 19:40:49 +0000 (-0400)
Subject: osd/PG: make non-empty PastIntervals non-fatal
X-Git-Tag: v12.1.0~137^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=74da154a2116d2386258399641a39f6d830d3650;p=ceph.git

osd/PG: make non-empty PastIntervals non-fatal

Yes, we should not have past intervals in this case.  However, we don't
need to crash the OSD; the ERR in clog is sufficient to fail the teuthology
run, and users in production don't actually care.

See: http://tracker.ceph.com/issues/20167
Signed-off-by: Sage Weil <sage@redhat.com>
---

diff --git a/src/osd/PG.cc b/src/osd/PG.cc
index c186548a6c69..cea01f320028 100644
--- a/src/osd/PG.cc
+++ b/src/osd/PG.cc
@@ -799,7 +799,6 @@ void PG::check_past_interval_bounds() const
       derr << info.pgid << " required past_interval bounds are"
 	   << " empty [" << rpib << ") but past_intervals is not: "
 	   << past_intervals << dendl;
-      assert(past_intervals.empty());
     }
   } else {
     if (past_intervals.empty()) {