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

jewel: osd: limit omap data in push op #11991

Merged
merged 1 commit into from Feb 1, 2017
Merged

Conversation

asheplyakov
Copy link

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)
@asheplyakov asheplyakov changed the title osd: limit omap data in push op jewel: osd: limit omap data in push op Nov 15, 2016
@smithfarm smithfarm added this to the jewel milestone Nov 15, 2016
@smithfarm smithfarm assigned ghost Nov 15, 2016
ghost pushed a commit that referenced this pull request Nov 23, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost assigned athanatos Dec 5, 2016
@ghost
Copy link

ghost commented Dec 5, 2016

@athanatos does this backport look good to merge ? It passed the rados (http://tracker.ceph.com/issues/17851#note-12) suite. It also passed the upgrade/jewel-x and upgrade/hammer-x (http://tracker.ceph.com/issues/17851#note-15) suites. Note that it won't be included in 10.2.4, reason why it targets jewel-next.

ghost pushed a commit that referenced this pull request Dec 5, 2016
Reviewed-by: Loic Dachary <ldachary@redhat.com>
@ghost ghost changed the base branch from jewel-next to jewel December 21, 2016 23:29
@smithfarm
Copy link
Contributor

@jdurgin This passed another rados run at http://tracker.ceph.com/issues/17851#note-45 with some unrelated failures. OK to merge?

@smithfarm
Copy link
Contributor

This passed another rados run at http://tracker.ceph.com/issues/17851#note-53

@smithfarm smithfarm merged commit ef7d826 into ceph:jewel Feb 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants