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

mgr: move "osd pool stats" to mgr #14365

Merged
merged 2 commits into from Apr 8, 2017

Conversation

Projects
None yet
3 participants
@tchaikov
Copy link
Contributor

tchaikov commented Apr 6, 2017

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

@tchaikov tchaikov added mgr mon labels Apr 6, 2017

@tchaikov tchaikov requested a review from liewegas Apr 6, 2017

mgr: move "osd pool stats" to mgr
Signed-off-by: Kefu Chai <kchai@redhat.com>

@tchaikov tchaikov force-pushed the tchaikov:wip-another-mgr-command branch from a8cf925 to db9e531 Apr 6, 2017

@tchaikov

This comment has been minimized.

Copy link
Contributor Author

tchaikov commented Apr 6, 2017

The following tests FAILED:
	  3 - ceph_objectstore_tool.py (Timeout)

https://jenkins.ceph.com/job/ceph-pull-requests/21552/

retest this please

@liewegas liewegas added the needs-qa label Apr 6, 2017

@liewegas

This comment has been minimized.

Copy link
Member

liewegas commented Apr 7, 2017

retest this please

@dzafman

This comment has been minimized.

Copy link
Member

dzafman commented Apr 7, 2017

@tchaikov This ceph_objectstore_tool.py test sets MGR=0 for vstart. Just change that to 1 and it shouldn't hang.

@tchaikov tchaikov force-pushed the tchaikov:wip-another-mgr-command branch from db9e531 to b32d60e Apr 8, 2017

@tchaikov

This comment has been minimized.

Copy link
Contributor Author

tchaikov commented Apr 8, 2017

@dzafman thanks for looking at this hang. just realized that it's me who added "MGR=0"!

@tchaikov tchaikov requested a review from dzafman Apr 8, 2017

tests: ceph_objectstore_tool.py: run vstart with MGR=1
Signed-off-by: Kefu Chai <kchai@redhat.com>

@tchaikov tchaikov force-pushed the tchaikov:wip-another-mgr-command branch from b32d60e to a5b38e4 Apr 8, 2017

@liewegas liewegas merged commit 5b8ea86 into ceph:master Apr 8, 2017

2 of 3 checks passed

default Build finished.
Details
Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.