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

nautilus: mon: mark pgtemp messages as no_reply more consistenly in preprocess_… #37171

Merged
merged 1 commit into from Sep 16, 2020

Conversation

neha-ojha
Copy link
Member

backport tracker: https://tracker.ceph.com/issues/47092


backport of #36593
parent tracker: https://tracker.ceph.com/issues/46914

this backport was staged using ceph-backport.sh version 15.1.1.389
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh

…_pgtemp

If a message is forwarded, it's conceivable the leader's and peon's evaluation
will disagree about whether the message is useful or not, which could result
in the leader ignoring it and the peon having a dangling forwarded message.
Fix this by marking the op as no_reply whenever ignoring it.

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

Signed-off-by: Greg Farnum <gfarnum@redhat.com>
(cherry picked from commit 73a014f)
@neha-ojha neha-ojha added this to the nautilus milestone Sep 15, 2020
@neha-ojha neha-ojha added the core label Sep 15, 2020
@yuriw yuriw added wip-yuri3-testing nautilus-batch-1 nautilus point releases labels Sep 16, 2020
@yuriw
Copy link
Contributor

yuriw commented Sep 16, 2020

@yuriw yuriw merged commit 68a4be7 into ceph:nautilus Sep 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants