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

osd: allow client throttler to be adjusted on-fly, without restart #13213

Merged
merged 1 commit into from Feb 3, 2017

Conversation

Projects
None yet
5 participants
@branch-predictor
Copy link
Member

branch-predictor commented Feb 1, 2017

This patch allows the osd_client_message_cap and
osd_client_message_size_cap to be adjusted on-fly, using admin socket
functionality.

Signed-off-by: Piotr Dałek piotr.dalek@corp.ovh.com

@ghost

This comment has been minimized.

Copy link

ghost commented Feb 2, 2017

could you please create an issue for this commit http://tracker.ceph.com/projects/ceph/issues/new ? It helps tracking backports and can be very simple, just with a URL to this PR :-)

OSD: allow client throttler to be adjusted on-fly, without restart
This patch allows the osd_client_message_cap and
osd_client_message_size_cap to be adjusted on-fly, using admin socket
functionality.

Fixes: http://tracker.ceph.com/issues/18791
Signed-off-by: Piotr Dałek <piotr.dalek@corp.ovh.com>

@branch-predictor branch-predictor force-pushed the ovh:bp-osd-updateable-throttles branch from eb594a5 to 64c309d Feb 2, 2017

@branch-predictor

This comment has been minimized.

Copy link
Member Author

branch-predictor commented Feb 2, 2017

@dachary done, and also updated commit message

@smithfarm

This comment has been minimized.

Copy link
Contributor

smithfarm commented Feb 2, 2017

@branch-predictor Next time, please wait until the master PR is merged before cherry-picking. If this PR gets rebased before merge, the "cherry picked from ..." lines in the backports will point to a non-existent commit.

@ktdreyer

This comment has been minimized.

Copy link
Member

ktdreyer commented Feb 2, 2017

does this need documentation?

@branch-predictor

This comment has been minimized.

Copy link
Member Author

branch-predictor commented Feb 2, 2017

@ktdreyer I don't think so, these options were already present, but not observed by OSDs after they started.

@liewegas liewegas changed the title OSD: allow client throttler to be adjusted on-fly, without restart osd: allow client throttler to be adjusted on-fly, without restart Feb 3, 2017

@yuriw yuriw merged commit f5b676a into ceph:master Feb 3, 2017

3 checks passed

Signed-off-by all commits in this PR are signed
Details
Unmodifed Submodules submodules for project are unmodified
Details
default Build finished.
Details

@branch-predictor branch-predictor deleted the ovh:bp-osd-updateable-throttles branch May 2, 2017

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.