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: make message cap option usable again #32855

Merged
merged 1 commit into from Jun 24, 2020

Conversation

jdurgin
Copy link
Member

@jdurgin jdurgin commented Jan 25, 2020

This reverts commit 45d5ac3.

Without a msg throttler, we can't change osd_client_message_cap cap.
The throttler is designed to work with 0 as a max, so change the
default to 0 to disable it by default instead.

This doesn't affect the default behavior, it only lets us use this
option again.

Fixes: https://tracker.ceph.com/issues/46143

Conflicts:
src/ceph_osd.cc - new style of gconf() access

Signed-off-by: Josh Durgin jdurgin@redhat.com
Signed-off-by: Neha Ojha nojha@redhat.com

@jdurgin
Copy link
Member Author

jdurgin commented Jan 25, 2020

@yuyuyu101 do you recall more details about the livelock you mentioned in #14944?

It seems like the ops throttle would be helpful for small messages (like deletes, or cls calls - one we've seen is related to RGW gc) that are expensive for the OSD. I'm curious if this msg cap can be effective in at least stopping the OSD from becoming overloaded and suiciding in such cases.

@neha-ojha
Copy link
Member

retest this please

@stale
Copy link

stale bot commented May 9, 2020

This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days.
If you are a maintainer or core committer, please follow-up on this pull request to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@stale stale bot added the stale label May 9, 2020
@neha-ojha neha-ojha removed the stale label May 11, 2020
@neha-ojha neha-ojha added the DNM label May 13, 2020
@jdurgin jdurgin force-pushed the wip-message-cap branch 2 times, most recently from 56e28d0 to ac3ec25 Compare June 22, 2020 17:49
@jdurgin jdurgin changed the title DNM: osd: make message cap option usable again osd: make message cap option usable again Jun 22, 2020
@jdurgin jdurgin removed the DNM label Jun 22, 2020
This reverts commit 45d5ac3.

Without a msg throttler, we can't change osd_client_message_cap cap.
The throttler is designed to work with 0 as a max, so change the
default to 0 to disable it by default instead.

This doesn't affect the default behavior, it only lets us use this
option again.

Fixes: https://tracker.ceph.com/issues/46143

Conflicts:
	src/ceph_osd.cc - new style of gconf() access

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