New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: Thrasher: do not update pools_to_fix_pgp_num if nothing happens #13518

Merged
merged 1 commit into from Feb 20, 2017

Conversation

Projects
None yet
2 participants
@tchaikov
Contributor

tchaikov commented Feb 19, 2017

we should not update pools_to_fix_pgp_num if the pool is not expanded or
the pg_num is not increased due to pgs being created. this prevent us
from fixing the pgp_num after done with thrashing if we actually did
nothing when fixing the pgp_num when thrashing, but we removed the pool
from pools_to_fix_pgp_num after set_pool_pgpnum() returns.

Signed-off-by: Kefu Chai kchai@redhat.com

test: Thrasher: do not update pools_to_fix_pgp_num if nothing happens
we should not update pools_to_fix_pgp_num if the pool is not expanded or
the pg_num is not increased due to pgs being created. this prevent us
from fixing the pgp_num after done with thrashing if we actually did
nothing when fixing the pgp_num when thrashing, but we removed the pool
from pools_to_fix_pgp_num after set_pool_pgpnum() returns.

Signed-off-by: Kefu Chai <kchai@redhat.com>

@tchaikov tchaikov requested a review from liewegas Feb 19, 2017

@tchaikov

This comment has been minimized.

Contributor

tchaikov commented Feb 19, 2017

@tchaikov tchaikov requested a review from yuriw Feb 19, 2017

@tchaikov tchaikov merged commit b3e516f into ceph:master Feb 20, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@tchaikov tchaikov deleted the tchaikov:wip-fix-pgp-num branch Feb 20, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment