qa: fix test_cephfs_mirror_stats failure#61076
Conversation
4d09335 to
5c125ce
Compare
|
I see some valgrind failures on this one. Putting it DNM for now. Will push the update soon. |
100MB files would take less than a second to sync, which makes no difference in 'last_synced_end' and the test fails intermittently. We need to increase the size of the files, as the time/duration is determined only in seconds. Because of this, it also needs more sleep time before checking the status. Fixes: https://tracker.ceph.com/issues/69232 Signed-off-by: Jos Collin <jcollin@redhat.com>
5c125ce to
005e492
Compare
Did this change introduce something new which you fixed in the latest push? |
I doubt it did since this is a QA only fix, but just checking is some change was removed in-between commits. |
I've just added more sleep time in the new push, as I'm syncing bigger snapshots in this change. So obviously we need to wait more time to get the updated status. |
|
jenkins test make check |
|
This PR is under test in https://tracker.ceph.com/issues/69939. |
Caused by #59875.
100MB files would take less than a second to sync, which makes no difference
in 'last_synced_end' and the test fails intermittently. We need to increase the
size of the files, as the time/duration is determined only in seconds.
Because of this, it also needs more sleep time before checking the status.
Fixes: https://tracker.ceph.com/issues/69232
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
xbetween the brackets:[x]. Spaces and capitalization matter when checking off items this way.Checklist
Show available Jenkins commands
jenkins retest this pleasejenkins test classic perfjenkins test crimson perfjenkins test signedjenkins test make checkjenkins test make check arm64jenkins test submodulesjenkins test dashboardjenkins test dashboard cephadmjenkins test apijenkins test docsjenkins render docsjenkins test ceph-volume alljenkins test ceph-volume toxjenkins test windowsjenkins test rook e2e