[jdk8u-dev] RFR: 8341057: Add 2 SSL.com TLS roots [v2]

[Andrew John Hughes]

On Wed, 2 Oct 2024 11:53:53 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:

>> Andrew John Hughes has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove manual invocation of tests for 8u
>
> jdk/test/security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java line 495:
> 
>> 493:  * @run main/othervm/manual -Djava.security.debug=certpath,ocsp CAInterop ssltlsrootecc2022 DEFAULT
>> 494:  * @run main/othervm/manual -Djava.security.debug=certpath,ocsp -Dcom.sun.security.ocsp.useget=false CAInterop ssltlsrootecc2022 DEFAULT
>> 495:  * @run main/othervm/manual -Djava.security.debug=certpath CAInterop ssltlsrootecc2022 CRL
> 
> In 8u we usually run those tests (not specifying them as manual). That's why they weren't run in GHA. Please remove `/manual`.

Ah yes, good catch. This is going to be a pain with these backports going forward.

-------------

PR Review Comment: https://git.openjdk.org/jdk8u-dev/pull/589#discussion_r1784850312