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

hammer: osd: ReplicatedBackend::build_push_op: add a second config to limit omap entries/chunk independently of object data #12417

Merged
merged 1 commit into from Jan 5, 2017

Conversation

smithfarm
Copy link
Contributor

We already have the config osd_recovery_max_chunk to limit the total
size of omap entries and omap data. But we need an individual config
to limit the number of omap entries independently. We call this config
osd_recovery_max_omap_entries_per_chunk here with the default number
of 64000.

Signed-off-by: Wanlong Gao <wanlong.gao@easystack.cn>
(cherry picked from commit 56064c5)
@smithfarm smithfarm self-assigned this Dec 9, 2016
@smithfarm smithfarm added this to the hammer milestone Dec 9, 2016
@smithfarm smithfarm changed the title hammer: ReplicatedBackend::build_push_op: add a second config to limit omap entries/chunk independently of object data hammer: osd: ReplicatedBackend::build_push_op: add a second config to limit omap entries/chunk independently of object data Dec 13, 2016
smithfarm added a commit to SUSE/ceph that referenced this pull request Dec 13, 2016
…push_op: add a second config to limit omap entries/chunk independently of object data

Reviewed-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit that referenced this pull request Dec 14, 2016
…_op: add a second config to limit omap entries/chunk independently of object data

Reviewed-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit that referenced this pull request Dec 14, 2016
…_op: add a second config to limit omap entries/chunk independently of object data

Reviewed-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit that referenced this pull request Jan 2, 2017
…_op: add a second config to limit omap entries/chunk independently of object data

Reviewed-by: Nathan Cutler <ncutler@suse.com>
@smithfarm
Copy link
Contributor Author

@athanatos This just passed a rados run at http://tracker.ceph.com/issues/17151#note-38

OK to merge?

@athanatos athanatos merged commit 7bfb05b into ceph:hammer Jan 5, 2017
@smithfarm smithfarm deleted the wip-18132-hammer branch January 5, 2017 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants