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

add DYLD_LIBRARY_PATH for osx native testing #3765

Merged
merged 1 commit into from
Nov 26, 2018

Conversation

llxia
Copy link
Contributor

@llxia llxia commented Nov 21, 2018

  • temporarily exclude cmdLineTester_pltest
  • temporarily exclude cmdLineTest_gpTest
  • add DYLD_LIBRARY_PATH for oxs native testing

[ci skip]

Signed-off-by: lanxia [email protected]

@llxia
Copy link
Contributor Author

llxia commented Nov 21, 2018

jenkins test sanity osx all

@llxia
Copy link
Contributor Author

llxia commented Nov 21, 2018

jenkins test sanity osx jdk8,jdk11

@llxia
Copy link
Contributor Author

llxia commented Nov 21, 2018

jenkins test sanity osx jdk11

1 similar comment
@llxia
Copy link
Contributor Author

llxia commented Nov 21, 2018

jenkins test sanity osx jdk11

@llxia
Copy link
Contributor Author

llxia commented Nov 21, 2018

jenkins test extended osx jdk11

@AdamBrousseau
Copy link
Contributor

jenkins test sanity osx jdk8

@AdamBrousseau
Copy link
Contributor

jenkins test extended osx jdk8

@llxia
Copy link
Contributor Author

llxia commented Nov 21, 2018

osx jdk8 PR builds are not ready to use. They both failed due to no matching label

There are no nodes with the label ‘ci.role.build&&hw.arch.x86&&sw.os.osx.10_11’

@llxia
Copy link
Contributor Author

llxia commented Nov 21, 2018

@pshipton Could you please review the change? Thanks

@babsingh
Copy link
Contributor

@llxia llxia changed the title add DYLD_LIBRARY_PATH for oxs native testing WIP: add DYLD_LIBRARY_PATH for oxs native testing Nov 23, 2018
@llxia llxia force-pushed the test branch 2 times, most recently from 05cb9cb to 0fef4c1 Compare November 23, 2018 14:20
@llxia
Copy link
Contributor Author

llxia commented Nov 23, 2018

jenkins test sanity osx jdk11

@llxia
Copy link
Contributor Author

llxia commented Nov 23, 2018

jenkins test extended osx jdk11

@llxia
Copy link
Contributor Author

llxia commented Nov 23, 2018

DYLD_LIBRARY_PATH should be added so native tests can find libs. (i.e., pthreadDestructor, reattachAfterExit, etc)

Since we had issue of hanging at Testing: abort in cmdLineTest_gpTest_0 before #3686 (comment), I suspect it maybe related. Further investgation is needed. Exclude it for now so we can have a build will not hang.

cmdLineTester_pltest is excluded due to #3685. It crashes in both DYLD_LIBRARY_PATH and LD_LIBRARY_PATH case.

@llxia
Copy link
Contributor Author

llxia commented Nov 23, 2018

PR builds failed during compilation (#3810)

@llxia llxia changed the title WIP: add DYLD_LIBRARY_PATH for oxs native testing add DYLD_LIBRARY_PATH for oxs native testing Nov 23, 2018
- temporarily exclude cmdLineTester_pltest
- temporarily exclude cmdLineTest_gpTest
- add DYLD_LIBRARY_PATH for oxs native testing

[ci skip]

Signed-off-by: lanxia <[email protected]>
@llxia
Copy link
Contributor Author

llxia commented Nov 26, 2018

jenkins test sanity osx jdk11

@llxia
Copy link
Contributor Author

llxia commented Nov 26, 2018

jenkins test extended osx jdk11

1 similar comment
@llxia
Copy link
Contributor Author

llxia commented Nov 26, 2018

jenkins test extended osx jdk11

@llxia
Copy link
Contributor Author

llxia commented Nov 26, 2018

rebased and relaunch the PR builds as #3810 is fixed

@llxia
Copy link
Contributor Author

llxia commented Nov 26, 2018

sanity.functional and extended.functional can complete without problem. Issues are opened in openj9. This PR is ready for review.

02:08:20.370 FAILED test targets:
02:08:20.370 	cmdLineTester_jvmtitests_2 ===> #3839
02:08:20.370 	testSCCMLTests1_openj9_1 ===>#3790
02:08:20.370 	cmdLineTest_sigxfszHandlingTest_0 ==> #3342
02:08:20.370 	cmdLineTester_SCURLClassLoaderNPTests_1 ===>#3790
02:08:20.370 	reattachAfterExit_0 ===> #3794
02:08:20.370 	jsigjnitest_0 ===> #3795
02:08:20.370 	thrstatetest_0 ==> #3795
02:08:20.370 
02:08:20.370 TOTAL: 209   EXECUTED: 117   PASSED: 110   FAILED: 7   SKIPPED: 92
02:58:54.504 FAILED test targets:
02:58:54.504 	testSCCMLTests4_1 ===> #3775
02:58:54.504 	testSCCMLTests5_1 ===> #3775
02:58:54.504 	testSCCMLTests6_1 ===> #3776
02:58:54.504 	TestSunAttachClasses_0 ===> #3842
02:58:54.504 	ContendedFieldsTests_90_1 ===> #3777
02:58:54.504 	threadMXBeanTestSuite2_3 ===> #3778
02:58:54.504 	testSoftMxDisclaimMemory_2 ===>#3841
02:58:54.504 	testDefaultDisclaimMemory_2 ===>#3841
02:58:54.504 
02:58:54.504 TOTAL: 301   EXECUTED: 113   PASSED: 105   FAILED: 8   SKIPPED: 188
02:58:54.504 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
02:58:54.504 
02:58:54.504 _extended done

@llxia llxia requested a review from smlambert November 26, 2018 19:28
@smlambert smlambert merged commit 7ea0a5f into eclipse-openj9:master Nov 26, 2018
@smlambert smlambert changed the title add DYLD_LIBRARY_PATH for oxs native testing add DYLD_LIBRARY_PATH for osx native testing Jan 28, 2021
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.

4 participants