mon/OSDMonitor: lookup_pruned_snap -> lookup_purged_snap

This confused me and sent me into the weeds.

Signed-off-by: Sage Weil <sage@redhat.com>
This commit is contained in:
Sage Weil 2019-06-07 10:21:16 -05:00
parent 94ebe0eab9
commit a12d80a813
2 changed files with 5 additions and 5 deletions

View File

@ -6254,7 +6254,7 @@ string OSDMonitor::make_snap_purged_key_value(
return make_snap_purged_key(pool, snap + num - 1);
}
int OSDMonitor::lookup_pruned_snap(int64_t pool, snapid_t snap,
int OSDMonitor::lookup_purged_snap(int64_t pool, snapid_t snap,
snapid_t *begin, snapid_t *end)
{
string k = make_snap_key(pool, snap);
@ -6311,14 +6311,14 @@ bool OSDMonitor::try_prune_purged_snaps()
snapid_t begin = i.get_start();
auto end = i.get_start() + i.get_len();
snapid_t pbegin = 0, pend = 0;
int r = lookup_pruned_snap(p.first, begin, &pbegin, &pend);
int r = lookup_purged_snap(p.first, begin, &pbegin, &pend);
if (r == 0) {
// already purged.
// be a bit aggressive about backing off here, because the mon may
// do a lot of work going through this set, and if we know the
// purged set from the OSDs is at least *partly* stale we may as
// well wait for it to be fresh.
dout(20) << __func__ << " we've already pruned " << pbegin
dout(20) << __func__ << " we've already purged " << pbegin
<< "~" << (pend - pbegin) << dendl;
break; // next pool
}
@ -12701,7 +12701,7 @@ bool OSDMonitor::_is_removed_snap(int64_t pool, snapid_t snap)
return true;
}
snapid_t begin, end;
int r = lookup_pruned_snap(pool, snap, &begin, &end);
int r = lookup_purged_snap(pool, snap, &begin, &end);
if (r == 0) {
dout(10) << __func__ << " pool " << pool << " snap " << snap
<< " - purged, [" << begin << "," << end << ")" << dendl;

View File

@ -512,7 +512,7 @@ private:
string make_snap_purged_key_value(int64_t pool, snapid_t snap, snapid_t num,
epoch_t epoch, bufferlist *v);
bool try_prune_purged_snaps();
int lookup_pruned_snap(int64_t pool, snapid_t snap,
int lookup_purged_snap(int64_t pool, snapid_t snap,
snapid_t *begin, snapid_t *end);
bool prepare_set_flag(MonOpRequestRef op, int flag);