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.

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

qa/tasks/ceph_manager: fix get_stuck_pgs from pg dump change #24689

Merged
merged 1 commit into from Oct 22, 2018

Conversation

liewegas
Copy link
Member

Fixes 95b7d23

Fixes: http://tracker.ceph.com/issues/36485
Signed-off-by: Sage Weil sage@redhat.com

Copy link
Contributor

@tchaikov tchaikov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

after #23622 , the stuck_pg_stats array is enclosed in an object named pg_info. so we need to access this array by its name.

@liewegas liewegas merged commit b678356 into ceph:nautilus Oct 22, 2018
liewegas added a commit that referenced this pull request Oct 22, 2018
* refs/pull/24689/head:
	qa/tasks/ceph_manager: fix get_stuck_pgs from pg dump change

Reviewed-by: Kefu Chai <kchai@redhat.com>
liewegas added a commit that referenced this pull request Oct 22, 2018
* refs/pull/24689/head:
	qa/tasks/ceph_manager: fix get_stuck_pgs from pg dump change
	Merge PR #24625 into nautilus
	qa/suites/rados/mgr/tasks/module_selftest: whitelist 'foo bar security'

Reviewed-by: Kefu Chai <kchai@redhat.com>
liewegas added a commit that referenced this pull request Oct 23, 2018
* refs/pull/24698/head:
	Merge PR #24697 into nautilus
	ceph_test_msgr: fix authorizer behavior
	Merge pull request #24667 from liewegas/wip-ec-thrash-full
	Merge PR #24689 into nautilus
	qa/suites/rados/thrash-erasure-code*/thrashers/*: less likely resv rejection injection

Reviewed-by: Josh Durgin <jdurgin@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants