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.

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

Disable cmdLineTester_jvmtitests_extended on win #2441

Merged
merged 1 commit into from
Jul 23, 2018
Merged

Disable cmdLineTester_jvmtitests_extended on win #2441

merged 1 commit into from
Jul 23, 2018

Conversation

llxia
Copy link
Contributor

@llxia llxia commented Jul 20, 2018 •
edited
Loading

Issue: #2129 #2147
[ci skip]

Signed-off-by: lanxia lan_xia@ca.ibm.com

@llxia
Copy link
Contributor Author

llxia commented Jul 20, 2018

jenkins test sanity win jdk8

@llxia llxia force-pushed the jvmtitest branch 2 times, most recently from f65a940 to 5298607 Compare July 23, 2018 14:52
@llxia
Copy link
Contributor Author

llxia commented Jul 23, 2018

jenkins test sanity win jdk8

@llxia
Copy link
Contributor Author

llxia commented Jul 23, 2018 •
edited
Loading

Tested build: https://ci.eclipse.org/openj9/job/PullRequest-Sanity-JDK8-win_x86-64_cmprssptrs-OpenJ9/155/

cmdLineTester_decompilationTests are excluded. Failed test is decompileAtMethodResolve_0 which is not related to this PR.

@llxia llxia changed the title WIP: Disable cmdLineTester_jvmtitests_extended on win Disable cmdLineTester_jvmtitests_extended on win Jul 23, 2018
@llxia llxia requested a review from smlambert July 23, 2018 18:06
@llxia
Disable cmdLineTester_jvmtitests_extended on win
a8fa3c3 
- disable cmdLineTester_jvmtitests_extended on win until eclipse-openj9#2129 is fixed
- rename cmdLineTester_jvmtitests_extended to
cmdLineTester_jvmtitests_debug

Issue: eclipse-openj9#2129 eclipse-openj9#2147
[ci skip]

Signed-off-by: lanxia <lan_xia@ca.ibm.com>
@smlambert smlambert merged commit 78279d9 into eclipse-openj9:master Jul 23, 2018
@pshipton
Copy link
Member

pshipton commented Jul 25, 2018 •
edited
Loading

Note there is still one test failing due to NPT ERROR.
For example https://ci.eclipse.org/openj9/job/PullRequest-Sanity-JDK8-win_x86-64_cmprssptrs-OpenJ9/157/

===============================================
Running test decompileAtMethodResolve_0 ...
===============================================
decompileAtMethodResolve_0 Start Time: Tue Jul 24 16:04:24 2018 Epoch Time (ms): 1532466264219
test with NoOptions
NPT ERROR: Cannot open library

decompileAtMethodResolve_0_FAILED

@llxia
Copy link
Contributor Author

llxia commented Jul 25, 2018

We also see other tests (i.e., decompileAtMethodResolve_0 and TestFlushReflectionCache_SE80_2) failing due to NPT error. https://ci.eclipse.org/openj9/job/PullRequest-Sanity-JDK8-win_x86-OpenJDK8/2
@theresa-m is working on the issue and I think she found the root cause, so I am not going to exclude these tests atm.

@pshipton
Copy link
Member

pshipton commented Aug 2, 2018

#2129 is now fixed by ibmruntimes/openj9-openjdk-jdk8#107 and the tests can be unexcluded @llxia @keithc-ca

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smlambert smlambert smlambert approved these changes

Assignees
No one assigned
Labels
comp:test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@llxia @pshipton @smlambert