]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
osd: initialize purged_snap on backfill start; restart backfill if change
authorSage Weil <sage@redhat.com>
Tue, 23 Sep 2014 23:21:33 +0000 (16:21 -0700)
committerSamuel Just <sjust@redhat.com>
Tue, 2 Dec 2014 21:48:42 +0000 (13:48 -0800)
commit1b656450ca75b12fb98dee82bace914ef5f45c44
tree17095d17e500a08f5a89aa1f75e6bca807a43df1
parentc982da44e0e9e0be3c3d4e8f5e0a186fb2fcebb3
osd: initialize purged_snap on backfill start; restart backfill if change

If we backfill a PG to a new OSD, we currently neglect to initialize
purged_snaps.  As a result, the first time the snaptrimmer runs it has to
churn through every deleted snap for all time, and to make matters worse
does so in one go with the PG lock held.  This leads to badness on any
cluster with a significant number of removed snaps that experiences
backfill.

Resolve this by initializing purged_snaps when we finish backfill.  The
backfill itself will clear out any stray snaps and ensure the object set
is in sync with purged_snaps.  Note that purged_snaps on the primary
that is driving backfill will not change during this period as the
snaptrimmer is not scheduled unless the PG is clean (which it won't be
during backfill).

If we by chance to interrupt backfill, go clean with other OSDs,
purge snaps, and then let this OSD rejoin, we will either restart
backfill (non-contiguous log) or the log will include the result of
the snap trim (the events that remove the trimmed snap).

Fixes: #9487
Backfill: firefly, dumpling
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit 255b430a87201c7d0cf8f10a3c1e62cbe8dd2d93)
src/osd/PG.cc