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.

Sign up for GitHub

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

Jump to bottom

osd: fix/clean up full map request handling #8446

Merged
merged 3 commits into from Apr 7, 2016
Merged

osd: fix/clean up full map request handling #8446

merged 3 commits into from Apr 7, 2016

Conversation

liewegas
Copy link
Member

@liewegas liewegas commented Apr 5, 2016

  • send the right number of maps from the mon
  • request more maps if the osd didn't get them all
  • clean up the code

@liewegas
osd: create rerequest_full_maps() helper
2e22f54 
Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas
osd: improve full map requests
81cc288 
If we don't get all the full maps we want, request more
immediately.

Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas
mon/OSDMonitor: fix off-by-one for osd_map_message_max
bc9607b 
For most messages we were sending osd_map_message_max + 1 maps.

Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas added this to the jewel milestone Apr 5, 2016
@tchaikov
Copy link
Contributor

tchaikov commented Apr 5, 2016

lgtm

@liewegas liewegas merged commit 66b7ae6 into ceph:master Apr 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
cleanup core needs-qa wip-sage-testing
Projects
None yet
Milestone
jewel
2 participants
@liewegas @tchaikov