Skip to main content
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [jakartaee-tck-dev] More changes needed to compile after merging pruning of Jakarta Deployment change...

Hi Olivier,

Thanks for reporting this. I find this issue now. There are some earlier errors in the logs which are related to the bind.dtds ant task and hence following build steps are failing.

This issue is not reported in the master run [1] since for the CI job , other TCKs(connector) uses the same workspace and is able to compile the dtds in this task. So the bind.dtds does not run for servlet in the job run. This need to be investigated though to fix it for servlet and possibly some other standalone tcks.

So far as I see it, the below issue need not be related to the Deployment change.

Thanks,
Alwin

[1] https://ci.eclipse.org/jakartaee-tck/job/standalonetck-nightly-build-run-master/

On 06/06/20 7:26 am, Olivier Lamy wrote:
Hi  
not sure if related but I still have failures trying to build master
I'm running this `sh ./docker/build_standalone-tcks.sh servlet`

trying to build from master and I still have some failure such:
```
     [java] java.lang.ClassNotFoundException: com.sun.ts.lib.deliverable.servlet.ServletDeliverable
     [java] at java.net.URLClassLoader.findClass(URLClassLoader.java:382)
     [java] at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
     [java] at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:349)
     [java] at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
     [java] at java.lang.Class.forName0(Native Method)
     [java] at java.lang.Class.forName(Class.java:264)
     [java] at com.sun.ts.lib.deliverable.DeliverableFactory.createInstance(DeliverableFactory.java:55)
     [java] at com.sun.ts.lib.deliverable.DeliverableFactory.createInstance(DeliverableFactory.java:44)
     [java] at com.sun.ts.lib.deliverable.DeliverableFactory.getDeliverableInstance(DeliverableFactory.java:32)
     [java] at com.sun.ts.lib.harness.InteropTestManager.getDeliverableTable(InteropTestManager.java:138)
     [java] at com.sun.ts.lib.harness.InteropTestManager.getInteropDirections(InteropTestManager.java:56)
     [java] at com.sun.ts.lib.harness.TSTestFinder.scanFile(TSTestFinder.java:288)
     [java] at com.sun.javatest.finder.TagTestFinder.scan(TagTestFinder.java:95)
     [java] at com.sun.ts.lib.harness.TSTestFinder.scan(TSTestFinder.java:171)
     [java] at com.sun.javatest.TestFinder.read(TestFinder.java:439)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:403)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read(BinaryTestWriter.java:374)
     [java] at com.sun.javatest.finder.BinaryTestWriter.run(BinaryTestWriter.java:240)
     [java] at com.sun.javatest.finder.BinaryTestWriter.main(BinaryTestWriter.java:122)
     [java] java.lang.NullPointerException
     [java] at com.sun.ts.lib.harness.InteropTestManager.getInteropDirections(InteropTestManager.java:58)
     [java] at com.sun.ts.lib.harness.TSTestFinder.scanFile(TSTestFinder.java:288)
     [java] at com.sun.javatest.finder.TagTestFinder.scan(TagTestFinder.java:95)
     [java] at com.sun.ts.lib.harness.TSTestFinder.scan(TSTestFinder.java:171)
     [java] at com.sun.javatest.TestFinder.read(TestFinder.java:439)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:403)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read0(BinaryTestWriter.java:421)
     [java] at com.sun.javatest.finder.BinaryTestWriter.read(BinaryTestWriter.java:374)
     [java] at com.sun.javatest.finder.BinaryTestWriter.run(BinaryTestWriter.java:240)
```
the generated zip bundle contains only .java files.
see the console here https://jenkins.webtide.net/job/tck/job/tck_build/153/

On Sat, Jun 6, 2020 at 4:44 AM Alwin Joseph <alwin.joseph@xxxxxxxxxx> wrote:

Thanks Scott. This would complete pruning the Jakarta Deployment changes with successful builds.


On 05/06/20 11:02 pm, Scott Marlow wrote:
The Platform TCK compilation failure due to org.glassfish.deployment.client is now resolved. :)

Scott

On Thu, Jun 4, 2020 at 7:53 PM Scott Marlow <smarlow@xxxxxxxxxx> wrote:
https://github.com/eclipse-ee4j/jakartaee-tck/pull/306 removes the reference to org.glassfish.deployment.client classes, which I think will be fine, as the org.glassfish.deployment.client no longer can be called.  

If we do see any unexpected (appclient) deployment issues with missing EJB stub classes (on GlassFish 6), we should look deeper at the S1AS scripting.

Scott

On Thu, Jun 4, 2020 at 3:58 PM Scott Marlow <smarlow@xxxxxxxxxx> wrote:
Hi,

The Platform TCK com.sun.ts.lib.implementation.sun.javaee.glassfish.AutoDeployment class no longer compiles, which makes sense as it depends on the (removed) GlassFish Deployment client.  

I don't think this is due to merging the change to remove Jakarta Deployment support, unless I missed something. I think it is more that I missed that com.sun.ts.lib.implementation.sun.javaee.glassfish.AutoDeployment references the (now removed) GlassFish Jakarta Deployment classes when running in Full Platform mode.  

There was discussion before of continuing to work with GlassFish as if running in Web Profile mode.  So, we can try doing that by updating com.sun.ts.lib.implementation.sun.javaee.glassfish.AutoDeployment to not reference the org.glassfish.deployment.client.* classes, however, I am concerned about possible side effects of what will not be setup for TCK testing.

AutoDeployment.java [1] shows the code that can 
 that we would modify to not use the org.glassfish.deployment.client.ServerConnectionIdentifier + org.glassfish.deployment.client.DeploymentFacility classes.  

Perhaps we do not need the AutoDeployment.java [1] getClientPath() code since we removed the EJB interoperability tests already, but I'm not really sure.

Also, there is a comment at the bottom of the [1] (private) getClientPath() that explains that we no longer need to return the jar file with the EJB stubs, so that may be unused code now.  It sure looks like unused code to me but I'm not sure of what the GlassFish deployment client might of been doing previously, that definitely will not happen anymore. 

Scott




_______________________________________________
jakartaee-tck-dev mailing list
jakartaee-tck-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartaee-tck-dev
_______________________________________________
jakartaee-tck-dev mailing list
jakartaee-tck-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartaee-tck-dev


--
Olivier

_______________________________________________
jakartaee-tck-dev mailing list
jakartaee-tck-dev@xxxxxxxxxxx
To unsubscribe from this list, visit https://www.eclipse.org/mailman/listinfo/jakartaee-tck-dev

Back to the top