ibmruntimes / openj9-openjdk-jdk17

Extensions for OpenJDK 17 for Eclipse OpenJ9
GNU General Public License v2.0
23 stars 53 forks source link

Re-enable FIPS140-3 failures #369

Closed JinhangZhang closed 1 month ago

JinhangZhang commented 1 month ago

@llxia @jasonkatonica As discussed, re-enable update where test ignores were removed in https://github.com/ibmruntimes/openj9-openjdk-jdk17/pull/367.

llxia commented 1 month ago

@JinhangZhang I noticed com/sun/jdi/JdwpAttachTest.java.JdwpAttachTest https://github.com/ibmruntimes/openj9-openjdk-jdk17/issues/321 linux-x64,linux-ppc64le,linux-s390x,aix-all was added in https://github.com/ibmruntimes/openj9-openjdk-jdk17/pull/367/files, but it is not in this PR. It is expected?

JinhangZhang commented 1 month ago

It is in https://github.com/ibmruntimes/openj9-openjdk-jdk17/blob/25d1ee81b8a3e0c6ddce3e2049a139b79988fd20/test/jdk/ProblemList-FIPS140_3_OpenJcePlus.txt#L929

@JinhangZhang I noticed com/sun/jdi/JdwpAttachTest.java.JdwpAttachTest https://github.com/ibmruntimes/openj9-openjdk-jdk17/issues/321 linux-x64,linux-ppc64le,linux-s390x,aix-all was added in https://github.com/ibmruntimes/openj9-openjdk-jdk17/pull/367/files, but it is not in this PR. It is expected?

jasonkatonica commented 1 month ago

These updates look reasonable to me.

JinhangZhang commented 1 month ago

@JasonFengJ9 Please help to merge