Skip to content

qa/suites: set osd_pg_log_dups_tracked in cfuse_workunit_suites_fsync.yaml#21909

Merged
jdurgin merged 1 commit intoceph:masterfrom
neha-ojha:wip-fix-powercycle
May 9, 2018
Merged

qa/suites: set osd_pg_log_dups_tracked in cfuse_workunit_suites_fsync.yaml#21909
jdurgin merged 1 commit intoceph:masterfrom
neha-ojha:wip-fix-powercycle

Conversation

@neha-ojha
Copy link
Member

Increase osd_pg_log_dups_tracked for this test, so that we keep track of dup ops and don't end up treating dup ops as new ops.

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

Signed-off-by: Neha Ojha nojha@redhat.com

….yaml

Signed-off-by: Neha Ojha <nojha@redhat.com>
@neha-ojha
Copy link
Member Author

All 100 runs of the same test passed with this fix.

http://pulpito.ceph.com/nojha-2018-05-08_22:23:43-powercycle-luminous-distro-basic-smithi/

@neha-ojha neha-ojha requested a review from jdurgin May 9, 2018 16:33
@neha-ojha neha-ojha added the core label May 9, 2018
@jdurgin jdurgin merged commit 8582e57 into ceph:master May 9, 2018
@batrick
Copy link
Member

batrick commented May 9, 2018

I'm trying to better understand the bug and fix. I'm not getting how this is a duplicate op issue. Shouldn't replies from the OSD be ordered by the tid of the messages? Why do we care about duplicate messages?

@neha-ojha
Copy link
Member Author

When we do not track enough dup ops, the ops which are actually dup ops end up being treated like new ops. In the asserts that this bug has been causing, we've seen the osd_op_reply for dup ops(which were detected) reaching the client before the ones, which weren't tracked as dup ops.

@yuriw yuriw added the backport label Jul 19, 2018
@yuriw
Copy link
Contributor

yuriw commented Jul 19, 2018

@theanalyst @smithfarm we need mimic backport
https://tracker.ceph.com/issues/23827

@smithfarm
Copy link
Contributor

@yuriw #23136

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants