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

Enable javax_net_ssl_fips with FIPS_VARIATION_PROBLEM_LIST_FILE #5662

Merged
merged 1 commit into from
Oct 2, 2024

Conversation

llxia
Copy link
Contributor

@llxia llxia commented Oct 2, 2024

javax_net_ssl_fips will remain disabled until openjdk javax/net/ssl is updated

related: backlog/issues/1529

depends on: adoptium/TKG#620 and ibmruntimes/openj9-openjdk-jdk11#816

llxia added a commit to llxia/TKG that referenced this pull request Oct 2, 2024
related: backlog/issues/1529 and adoptium/aqa-tests#5662

Signed-off-by: Lan Xia <[email protected]>
llxia added a commit to llxia/openj9-openjdk-jdk11 that referenced this pull request Oct 2, 2024
related: related: backlog/issues/1529, adoptium/TKG#620 and adoptium/aqa-tests#5662
Signed-off-by: Lan Xia <[email protected]>
@llxia
Copy link
Contributor Author

llxia commented Oct 2, 2024

Grinder ==> Grinder is killed as the tests will not pass at the moment.

00:03:33.356  ===============================================
00:03:33.356  Running test javax_net_ssl_fips_0 ...
00:03:33.356  ===============================================
00:03:33.356  javax_net_ssl_fips_0 Start Time: Wed Oct  2 11:20:54 2024 Epoch Time (ms): 1727893254305
00:03:33.356  Test is disabled due to:
00:03:33.356  - placeholder for now until the tests in javax/net/ssl are updated for FIPS
00:03:33.356  - only applicable for Semeru atm
00:03:33.356  variation: -Dsemeru.fips=true -Dsemeru.customprofile=OpenJCEPlusFIPS.FIPS140-3-Test-TLS-PKCS12 -Djava.security.properties=/home/jenkins/workspace/Grinder/aqa-tests/TKG/../openjdk/openjdk-jdk/test/jdk/javax/net/ssl/TLSTest_java.security
00:03:33.356  JVM_OPTIONS:  -Dsemeru.fips=true -Dsemeru.customprofile=OpenJCEPlusFIPS.FIPS140-3-Test-TLS-PKCS12 -Djava.security.properties=/home/jenkins/workspace/Grinder/aqa-tests/TKG/../openjdk/openjdk-jdk/test/jdk/javax/net/ssl/TLSTest_java.security -Xverbosegclog 
...
00:03:33.356  -exclude:"/home/jenkins/workspace/Grinder/aqa-tests/TKG/../openjdk/openjdk-jdk/test/jdk/ProblemList-OpenJCEPlusFIPS.FIPS140-3-Test-TLS-PKCS12.txt" \

related: backlog/issues/1529

Signed-off-by: Lan Xia <[email protected]>
@llxia llxia marked this pull request as ready for review October 2, 2024 18:32
Copy link
Contributor

@LongyuZhang LongyuZhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

llxia added a commit to llxia/TKG that referenced this pull request Oct 2, 2024
related: backlog/issues/1529 and adoptium/aqa-tests#5662

Signed-off-by: Lan Xia <[email protected]>
llxia added a commit to llxia/TKG that referenced this pull request Oct 2, 2024
related: backlog/issues/1529 and adoptium/aqa-tests#5662

Signed-off-by: Lan Xia <[email protected]>
@karianna karianna requested a review from smlambert October 2, 2024 18:56
llxia added a commit to llxia/TKG that referenced this pull request Oct 2, 2024
related: backlog/issues/1529 and adoptium/aqa-tests#5662

Signed-off-by: Lan Xia <[email protected]>
@JasonFengJ9 JasonFengJ9 merged commit d97cfa6 into adoptium:master Oct 2, 2024
3 checks passed
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