search

jakartaee / jaxb-tck

Other
4 stars 16 forks source link

JAXB TCK JDK21 build + run with security manager disabled. #81

Closed gurunrao closed 6 months ago

gurunrao commented 1 year ago

JAXB TCK JDK21 build + run with security manager disabled.

scottmarlow commented 8 months ago

@gurunrao have we tried to build these changes on https://ci.eclipse.org/jakartaee-tck yet?

Should we do that via one of our ci setups like https://ci.eclipse.org/jakartaee-tck/job/guru or https://ci.eclipse.org/jakartaee-tck/job/jakartaee-tck-scottmarlow or perhaps https://ci.eclipse.org/jakartaee-tck/job/10/job/eftl-jaxb-tck-build-run-100?

gurunrao commented 7 months ago

@gurunrao have we tried to build these changes on https://ci.eclipse.org/jakartaee-tck yet?

Should we do that via one of our ci setups like https://ci.eclipse.org/jakartaee-tck/job/guru or https://ci.eclipse.org/jakartaee-tck/job/jakartaee-tck-scottmarlow or perhaps https://ci.eclipse.org/jakartaee-tck/job/10/job/eftl-jaxb-tck-build-run-100?

The changes have been built and tested locally, CI Job https://ci.eclipse.org/jakartaee-tck/job/guru/job/jaxb-tck-guru/ was used for build and test runs. We have used same job to test the JAXB TCK challenge https://github.com/jakartaee/jaxb-tck/issues/82, hence Job was modified to run with https://github.com/lukasj/jaxb-tck.

PS: note the PR is 8months old and had used JDK21 beta [ https://download.java.net/java/early_access/jdk21/15/GPL/openjdk-21-ea+15_linux-x64_bin.tar.gz ]. I will update PR to include JDK 21 GA [ https://download.java.net/java/GA/jdk21.0.1/415e3f918a1f4062a0074a2794853d0d/12/GPL/openjdk-21.0.1_linux-x64_bin.tar.gz ] during free cycles.

gurunrao commented 6 months ago

42 failures are observed for JDK21 + JAXB RI run + PR changes. Details of failures can be found with Junit report. JAXB-TCK-junit-report.zip

lukasj commented 6 months ago

@gurunrao #85 fixes (at least some) failures you see, can you try to apply & retry, please?

gurunrao commented 6 months ago

@gurunrao #85 fixes (at least some) failures you see, can you try to apply & retry, please?

Thanks lukas for the fix. I have applied the fix and built new JAXB TCK, test runs are in-progress.

gurunrao commented 6 months ago

@gurunrao #85 fixes (at least some) failures you see, can you try to apply & retry, please?

Thanks lukas for the fix. I have applied the fix and built new JAXB TCK, test runs are in-progress.

The 42 failing Tests with JDK21 are passing with the fix https://github.com/jakartaee/jaxb-tck/pull/85

gurunrao commented 6 months ago

I am planning to raise new PR with TCK runs for JDK21, without TCK build using JDK21.