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

Update openssl testing for the 0.12.1 release #4558

Merged
merged 3 commits into from
Feb 3, 2019
Merged

Update openssl testing for the 0.12.1 release #4558

merged 3 commits into from
Feb 3, 2019

Conversation

pshipton
Copy link
Member

@pshipton pshipton commented Feb 3, 2019 •
edited
Loading

Required so openssl (CryptoTest) testing passes with ibmruntimes/openj9-openjdk-jdk11#122 and ibmruntimes/openj9-openjdk-jdk8#263

Corresponds to #4551, #4547 and #4555

llxia and others added 3 commits February 3, 2019 10:49
@llxia @pshipton
Remove obsolete OpenSSL test
8f9b7a7 
[ci skip]

Signed-off-by: lanxia <lan_xia@ca.ibm.com>
@pshipton
Update CryptoTest.java to load CipherCore
e0e3bc8 
MessageDigest openssl support is being disabled due to eclipse-openj9#4530, the test
needs to check a different algorithm.

ibmruntimes/openj9-openjdk-jdk11#117
ibmruntimes/openj9-openjdk-jdk8#259

[ci skip]

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
@pshipton
Fix CryptoTest to accept "native" or "Native" in output
ad22dbe 
Required so the tests can pass when ibmruntimes/openj9-openjdk-jdk11#122
is merged.

[ci skip]

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
@pshipton
Copy link
Member Author

pshipton commented Feb 3, 2019

jenkins test sanity zlinux jdk11

@pshipton
Copy link
Member Author

pshipton commented Feb 3, 2019

This can't be PR tested, as the PR testing gets the head of OMR, which isn't compatible.
Testing via a custom build, in combination with ibmruntimes/openj9-openjdk-jdk11#123
https://ci.eclipse.org/openj9/view/Pipelines/job/Pipeline-Release-0.12.1-Build/10/

@pshipton pshipton added this to the Release 0.12.1 milestone Feb 3, 2019
@pshipton
Copy link
Member Author

pshipton commented Feb 3, 2019 •
edited
Loading

Testing successful.

https://ci.eclipse.org/openj9/view/Pipelines/job/Pipeline-Release-0.12.1-Build/10/
12:05:47 cmdLineTester_CryptoTest_0_PASSED

Confirmed MessageDigest is disabled in the build.

bin/java -Djdk.cryptoTrace=true CryptoTest
MessageDigest load - Native crypto library disabled.

openjdk version "11.0.2-internal" 2019-01-15
OpenJDK Runtime Environment (build 11.0.2-internal+0-adhoc.jenkins.Build-JDK11-linux390-64cmprssptrs)
Eclipse OpenJ9 VM (build openssl-0.12.1-ad22dbe, JRE 11 Linux s390x-64-Bit Compressed References 20190203_514 (JIT enabled, AOT enabled)
OpenJ9   - ad22dbe
OMR      - d2f4534
JCL      - 72e40a0 based on jdk-11.0.2+7)

OpenJ9 sha is this PR.
OMR sha is the openj9-0.12.0 and openj9-0.12.1 release level
JCL sha is ibmruntimes/openj9-openjdk-jdk11#123
Not sure why jdk-11.0.2+7 but this should be resolved by ibmruntimes/openj9-openjdk-jdk11#119 which is not included in this build. I will re-validate the -version output after this PR and ibmruntimes/openj9-openjdk-jdk11#123 are merged.

@DanHeidinga, or @llxia @smlambert for merge

@smlambert smlambert merged commit 90dd8cb into eclipse-openj9:v0.12.1-release Feb 3, 2019
@pshipton pshipton deleted the openssl-0.12.1 branch March 1, 2019 19:06
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
Release 0.12.1
Development

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