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: common/ceph_time: tolerate mono time going backwards #34542

Merged
merged 1 commit into from Apr 24, 2020

Conversation

smithfarm
Copy link
Contributor

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


backport of #33699
parent tracker: https://tracker.ceph.com/issues/43365

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

@smithfarm smithfarm added this to the nautilus milestone Apr 13, 2020
@jdurgin jdurgin changed the base branch from nautilus to nautilus-saved April 15, 2020 17:29
@jdurgin jdurgin changed the base branch from nautilus-saved to nautilus April 15, 2020 17:29
Some kernels (and possibly some hardware?) can trigger a monotonic clock
that goes back in time.  That, in turn, can lead to a negative monotonic
time span.  This would trigger an assert.

This this problem seems to be widespread, tolerate the case and interpret
it as a 0-length interval (vs something negative).

Fixes: https://tracker.ceph.com/issues/44078
Fixes: https://tracker.ceph.com/issues/43365
Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit cd00378)
@smithfarm
Copy link
Contributor Author

Unrelated "make check" failure:

1 tests failed out of 168 Total Test time (real) = 974.04 sec
The following tests FAILED: 5 - run-tox-mgr-dashboard (Failed) 

@yuriw
Copy link
Contributor

yuriw commented Apr 15, 2020

@yuriw
Copy link
Contributor

yuriw commented Apr 24, 2020

test this please

@yuriw
Copy link
Contributor

yuriw commented Apr 24, 2020

@smithfarm I am hesitant to merge with failed tests in this PR, although seem unrelated, pls merge t will, it was tested

@smithfarm
Copy link
Contributor Author

This time it was smoke.sh that failed, apparently on some bluestore errors.

@smithfarm
Copy link
Contributor Author

jenkins test make check

@smithfarm smithfarm merged commit c01a587 into ceph:nautilus Apr 24, 2020
@smithfarm smithfarm deleted the wip-44486-nautilus branch April 24, 2020 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
5 participants