Skip to content

add time check #8

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

Merged
merged 1 commit into from
Jun 2, 2017
Merged

add time check #8

merged 1 commit into from
Jun 2, 2017

Conversation

mnaboka
Copy link
Contributor

@mnaboka mnaboka commented Jun 2, 2017

use linux system call adjtimex and STA_UNSYNC magic constant from linux kernel.
The idea is taken from DC/OS check-time package.

This check will be built on Linux only.

screen shot 2017-06-02 at 12 57 09 pm

@amitaekbote
Copy link
Contributor

test?

@mnaboka mnaboka force-pushed the mnaboka/add-timesync-check branch 2 times, most recently from e9a708d to 10b5fe2 Compare June 2, 2017 19:52
Copy link
Contributor

@collinvandyck collinvandyck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clean and straightforward, but needs tests.

@mnaboka mnaboka force-pushed the mnaboka/add-timesync-check branch from 10b5fe2 to f09ece0 Compare June 2, 2017 21:00
@mnaboka
Copy link
Contributor Author

mnaboka commented Jun 2, 2017

@collinvandyck done

@mnaboka mnaboka merged commit 6bf4c42 into master Jun 2, 2017
@mnaboka mnaboka deleted the mnaboka/add-timesync-check branch June 2, 2017 21:39
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.

3 participants