Skip to content
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.

Sign up for GitHub

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

Jump to bottom

kraken: tests: qa/tasks: set pgp = pg num on thrashing finish #13757

Merged
merged 2 commits into from Mar 3, 2017
Merged

kraken: tests: qa/tasks: set pgp = pg num on thrashing finish #13757

merged 2 commits into from Mar 3, 2017

Conversation

liewegas
Copy link
Member

@liewegas liewegas commented Mar 3, 2017 •
edited by smithfarm

@tchaikov @liewegas
test: Thrasher: update pgp_num of all expanded pools if not yet
ab5538f 
otherwise wait_until_healthy will fail after timeout as seeing warning
like:

HEALTH_WARN pool cephfs_data pg_num 182 > pgp_num 172

Signed-off-by: Kefu Chai <kchai@redhat.com>
(cherry picked from commit 136483a)
@tchaikov @liewegas
test: Thrasher: do not update pools_to_fix_pgp_num if nothing happens
6adf39b 
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>
(cherry picked from commit c0f0cde)
@liewegas liewegas requested a review from tchaikov March 3, 2017 00:28
@tchaikov tchaikov added this to the kraken milestone Mar 3, 2017
@tchaikov tchaikov merged commit 7e16ea2 into ceph:kraken Mar 3, 2017
@smithfarm smithfarm changed the title qa/tasks: set pgp = pg num on thrashing finish kraken: tests: qa/tasks: set pgp = pg num on thrashing finish Aug 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tchaikov tchaikov tchaikov approved these changes

Assignees
No one assigned
Labels
bug-fix tests
Projects
None yet
Milestone
kraken
2 participants
@liewegas @tchaikov