-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
osd/osd-rep-recov-eio.sh: TEST_rados_repair_warning: return 1 #37483
Conversation
Signed-off-by: David Zafman <dzafman@redhat.com>
Testing passed with other failures |
@@ -224,6 +228,7 @@ function TEST_rados_repair_warning() { | |||
rados_get $dir $poolname ${objbase}-$i || return 1 | |||
done | |||
|
|||
wait_for_clean |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
has the test been failing for lack of clean just at this stage or everywhere else?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After a read that needs a repair, recovery is started. I never noticed on my build machine that the query could happen before the recovery is done, but teuthology did have this race. I did a sleep then Brad asked if there was some other way to tell of the repair is finished. So I realized that wait_for_clean() would do it. The only other possible race, is if recovery was async to the read repair. So if after the read the PG was still active+clean we would have a problem.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've just seen the test fail here so wondering whether we need to add wait_for_clean
in all the other places or not, certainly doesn't hurt
Fixes: https://tracker.ceph.com/issues/46405 Signed-off-by: David Zafman <dzafman@redhat.com>
@dzafman is this ready to merge? |
@neha-ojha Yes |
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox