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

mon: OSDMonitor: do not trust small values in osd epoch cache #3823

Merged
1 commit merged into from Mar 11, 2015
Merged

mon: OSDMonitor: do not trust small values in osd epoch cache #3823

1 commit merged into from Mar 11, 2015

Conversation

ghost
Copy link

@ghost ghost commented Feb 27, 2015

If the epoch cache says the osd has epoch 100 and the osd is asking for
epoch 200+, do not send it 100+.

Fixes: #10787
Backport: giant, firefly
Signed-off-by: Sage Weil sage@redhat.com
(cherry picked from commit a5759e9)

If the epoch cache says the osd has epoch 100 and the osd is asking for
epoch 200+, do not send it 100+.

Fixes: #10787
Backport: giant, firefly
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit a5759e9)
@ghost ghost added bug-fix core labels Feb 27, 2015
@ghost ghost added this to the firefly milestone Feb 27, 2015
@ghost ghost assigned jecluis Feb 27, 2015
@loic-bot
Copy link

SUCCESS: the output of run-make-check.sh on centos-centos7 for 740da24 is http://paste2.org/_Dk8njYtW

:octocat: Sent from GH.

@jecluis
Copy link
Member

jecluis commented Mar 1, 2015

@dachary looks good!

@jecluis jecluis assigned ghost and unassigned jecluis Mar 1, 2015
ghost pushed a commit that referenced this pull request Mar 11, 2015
mon/OSDMonitor: do not trust small values in osd epoch cache

Reviewed-by: Joao Eduardo Luis <joao@redhat.com>
@ghost ghost merged commit f5525a1 into ceph:firefly Mar 11, 2015
@ghost ghost changed the title mon/OSDMonitor: do not trust small values in osd epoch cache mon: OSDMonitor: do not trust small values in osd epoch cache Jul 14, 2015
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants