Skip to content

Commit

Permalink
mgr: fix pgp_num vs merge pacer
Browse files Browse the repository at this point in the history
This condition should only kick in for pgp_num reductions, or else we'll
end up jump to the end instead of slowing down merges.  (For splits, we
don't need to pace like this at all.)

Fixes 76503a1

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit e6113c1)
  • Loading branch information
liewegas committed Apr 29, 2019
1 parent c18a3fb commit 89e8e2a
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/mgr/DaemonServer.cc
Expand Up @@ -2601,7 +2601,8 @@ void DaemonServer::adjust_pgs()
}
dout(20) << " room " << room << " estmax " << estmax
<< " delta " << delta << " next " << next << dendl;
if (p.get_pgp_num_target() == p.get_pg_num_target()) {
if (p.get_pgp_num_target() == p.get_pg_num_target() &&
p.get_pgp_num_target() < p.get_pg_num()) {
// since pgp_num is tracking pg_num, ceph is handling
// pgp_num. so, be responsible: don't let pgp_num get
// too far out ahead of merges (if we are merging).
Expand Down

0 comments on commit 89e8e2a

Please sign in to comment.