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
qa/suites/crimson-rados/thrash: enable chance_down #56511
Conversation
@athanatos, testing with #56376 to verify that we catch the error now. thrash tests with snaps enabled with this PR: Edit: OSD logs are not available yet but teuthology log looks promising:
|
We have other issues that involve restarting that require handling before merging, tested this PR on main as well: Hopefully that's caused by the scrub tracked issue |
@Matan-B Ah, that makes sense. If you have time, it would be helpful to do a first pass of that run and file bugs for the defects. We can split up the work on fixing them. |
https://tracker.ceph.com/issues/65203 |
@athanatos, the previous trackers seem to be ok, new one popped out: |
Updating relevant PRs: CC: @xxhdx1985126, @athanatos |
As the thrash tests were introduced, some options were disabled until the tests are stabilized. Re-enable chance_down option (default is 0.4) to detect bugs on restart. Since it will probably take few iterations before thrash and recovery tests ('default.yaml') will pass successfully, add anoter 'simple.yaml' which should remain stable. Fixes: https://tracker.ceph.com/issues/65130 Signed-off-by: Matan Breizman <mbreizma@redhat.com>
6db0718
to
3727517
Compare
I'll merge this based on: @xuxuehan, we can start to test+merge the 'crimson-replicated-recovery' PRs based on the non-recovery thrash tests (as we did before) until both are stable. |
jenkins test api |
jenkins test make check |
jenkins test api |
Contribution Guidelines
To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an
x
between the brackets:[x]
. Spaces and capitalization matter when checking off items this way.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 dashboard cephadm
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox
jenkins test windows
jenkins test rook e2e