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

mon: get pools health'info have error #5369

Merged
1 commit merged into from Aug 30, 2015
Merged

mon: get pools health'info have error #5369

1 commit merged into from Aug 30, 2015

Conversation

ghost
Copy link

@ghost ghost commented Jul 28, 2015

@renhwztetecs @ldachary
Update OSDMonitor.cc
4457d3e 
OSDMonitor::get_pools_health(), s/objects/bytes/

Fixes: #12402
Signed-off-by: ren.huanwen@zte.com.cn
(cherry picked from commit 7fc13c9)
@ghost ghost self-assigned this Jul 28, 2015
@ghost ghost added this to the hammer milestone Jul 28, 2015
@ghost ghost added bug-fix core labels Jul 28, 2015
@loic-bot
Copy link

SUCCESS: http://jenkins.ceph.dachary.org/job/ceph/6689/

ghost pushed a commit that referenced this pull request Jul 29, 2015
@ldachary
Merge pull request #5369: get pools health'info have error
ed6e328 
Reviewed-by:
@ghost
Copy link
Author

ghost commented Aug 30, 2015

@tchaikov does this backport look good to merge ? It passed a run of the hammer rados suite ( see http://tracker.ceph.com/issues/12701#rados for details ).

ghost pushed a commit that referenced this pull request Aug 30, 2015
@ldachary
Merge pull request #5369: get pools health'info have error
9f8612b 
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 30, 2015
@ldachary
Merge pull request #5369: get pools health'info have error
cf995e8 
Reviewed-by: Loic Dachary <ldachary@redhat.com>
ghost pushed a commit that referenced this pull request Aug 30, 2015
@ldachary
Merge pull request #5369: get pools health'info have error
01b81a9 
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@tchaikov
Copy link
Contributor

@dachary lgtm
On Sun, 30 Aug 2015 at 18:01 Loic Dachary notifications@github.com wrote:

@tchaikov https://github.com/tchaikov does this backport look good to
merge ? It passed a run of the hammer rados suite ( see
http://tracker.ceph.com/issues/12701#rados for details ).

—
Reply to this email directly or view it on GitHub
#5369 (comment).

ghost pushed a commit that referenced this pull request Aug 30, 2015
Merge pull request #5369 from dachary/wip-12498-hammer
c3a2bf3 
get pools health'info have error

Reviewed-by: Kefu Chai <kchai@redhat.com>
@ghost ghost merged commit c3a2bf3 into ceph:hammer Aug 30, 2015
@ghost ghost changed the title get pools health'info have error mon: get pools health'info have error Sep 13, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug-fix core
Projects
None yet
Milestone
hammer
3 participants
@loic-bot @tchaikov @renhwztetecs