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

luminous: osd: get loadavg per cpu for scrub load threshold check #24593

Merged
merged 1 commit into from Oct 25, 2018

Conversation

Projects
None yet
5 participants
@dzafman
Member

dzafman commented Oct 15, 2018

osd: get loadavg per cpu for scrub load threshold check
Signed-off-by: kungf <yang.wang@easystack.cn>
(cherry picked from commit c1dba46)

@dzafman dzafman added this to the luminous milestone Oct 15, 2018

@dzafman

This comment has been minimized.

Member

dzafman commented Oct 15, 2018

Including #24532 commits, testing passed:
run-standalone.sh osd-recovery-scrub.sh osd-scrub-distrust.sh osd-scrub-repair.sh osd-scrub-snaps.sh osd-scrub-test.sh osd-unexpected-clone.sh
...
6 TESTS PASSED

@dzafman dzafman requested a review from neha-ojha Oct 15, 2018

@neha-ojha

This comment has been minimized.

Member

neha-ojha commented Oct 15, 2018

@dzafman let's add the tracker reference here, otherwise looks good!

@dzafman

This comment has been minimized.

Member

dzafman commented Oct 15, 2018

@neha-ojha I've usually left commit comments generated by cherry-pick alone. But can't think of a real reason for it. Do I use the tracker for the original or backport tracker? Would I add the tracker if the source commit already referenced the original tracker?

@neha-ojha

This comment has been minimized.

Member

neha-ojha commented Oct 15, 2018

@dzafman Just adding a reference to https://tracker.ceph.com/issues/36419 in this PR(as a comment) will be useful, I think.

@yuriw

This comment has been minimized.

Contributor

yuriw commented Oct 19, 2018

@yuriw yuriw merged commit 869ce7e into ceph:luminous Oct 25, 2018

4 checks passed

Docs: build check OK - docs built
Details
Signed-off-by all commits in this PR are signed
Details
Unmodified Submodules submodules for project are unmodified
Details
make check make check succeeded
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment