Skip to content

[3.6] bpo-30649: test_os tolerates 50 ms delta for utime (#2156) #2175

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jun 14, 2017
Merged

[3.6] bpo-30649: test_os tolerates 50 ms delta for utime (#2156) #2175

merged 2 commits into from
Jun 14, 2017

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Jun 13, 2017

On Windows, tolerate a delta of 50 ms instead of 20 ms in
test_utime_current() and test_utime_current_old() of test_os.

On other platforms, reduce the delta from 20 ms to 10 ms.
(cherry picked from commit c94caca)

@vstinner
Copy link
Member Author

vstinner commented Jun 13, 2017

Wrong: need also #2176

vstinner added 2 commits June 14, 2017 11:57
@vstinner
bpo-30649: test_os tolerates 50 ms delta for utime (#2156)
9048503 
On Windows, tolerate a delta of 50 ms instead of 20 ms in
test_utime_current() and test_utime_current_old() of test_os.

On other platforms, reduce the delta from 20 ms to 10 ms.
(cherry picked from commit c94caca)
@vstinner
bpo-30649: Revert utime delta in test_os (#2176)
6b1309b 
PPC64 Fedora 3.x buildbot requires at least a delta of 14 ms: revert
the utime delta to 20 ms.

I tried 10 ms, but test_os failed on the PPC64 Fedora 3.x buildbot.
(cherry picked from commit 3402f72)
@vstinner
Copy link
Member Author

vstinner commented Jun 14, 2017

Wrong: need also #2176

Done, I also cherry-picked this other change.

@vstinner vstinner merged commit c2a506e into python:3.6 Jun 14, 2017
@vstinner vstinner deleted the utime36 branch June 14, 2017 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vstinner @the-knights-who-say-ni