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

Fix jvmnativestest test_EnsureWallClockTime() #13896

Merged
merged 2 commits into from
Nov 10, 2021

Conversation

keithc-ca
Copy link
Contributor

Fixes: #11293.

Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
Signed-off-by: Keith W. Campbell <keithc@ca.ibm.com>
@keithc-ca
Copy link
Contributor Author

Jenkins test sanity aix,osx,plinux,win32 jdk8

@pshipton
Copy link
Member

There aren't any OSX machines available atm, but all other platforms passed. I'll assume OSX won't be a problem.

@pshipton pshipton merged commit 8806529 into eclipse-openj9:master Nov 10, 2021
@keithc-ca
Copy link
Contributor Author

The implementation of omrtime_current_time_nanos() is different on OSX, but not enough so to be concerned that we'll see failures there.

@keithc-ca keithc-ca deleted the nanotime branch November 10, 2021 18:47
@pshipton
Copy link
Member

I'll check the grinder later.

@pshipton
Copy link
Member

The OSX grinder did pass.

@pshipton
Copy link
Member

I'm going to revert this. In PR testing for #13899, jvmnativestest_0 failed twice, both on zlinux.

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.

jvmnativestest_0 nanotime is not similar to millitime
2 participants