Skip to content

Failing with Payara 5.181 #20

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

Closed
CSchulz opened this issue Apr 25, 2018 · 4 comments
Closed

Failing with Payara 5.181 #20

CSchulz opened this issue Apr 25, 2018 · 4 comments

Comments

@CSchulz
Copy link

CSchulz commented Apr 25, 2018

I have noticed that the samples are failing when updating to 5.181, the default domain was renamed to default1 and the container doesn't start correctly:

[INFO] Running org.javaee8.servlet.http2.Http2Test
Starting container using command: [java, -jar, /Users/ich/Documents/Daten/javaee8-samples/target/payara5/glassfish/modules/admin-cli.jar, start-domain, domain1, -t]
Exception in thread "main" java.lang.NullPointerException
	at com.sun.enterprise.module.common_impl.AbstractModulesRegistryImpl.initializeServiceLocator(AbstractModulesRegistryImpl.java:152)
	at com.sun.enterprise.module.common_impl.AbstractModulesRegistryImpl.newServiceLocator(AbstractModulesRegistryImpl.java:144)
	at com.sun.enterprise.module.common_impl.AbstractModulesRegistryImpl.createServiceLocator(AbstractModulesRegistryImpl.java:218)
	at com.sun.enterprise.module.common_impl.AbstractModulesRegistryImpl.createServiceLocator(AbstractModulesRegistryImpl.java:224)
	at com.sun.enterprise.module.single.StaticModulesRegistry.createServiceLocator(StaticModulesRegistry.java:88)
	at com.sun.enterprise.admin.cli.CLIContainer.getServiceLocator(CLIContainer.java:216)
	at com.sun.enterprise.admin.cli.CLIContainer.getLocalCommand(CLIContainer.java:254)
	at com.sun.enterprise.admin.cli.CLICommand.getCommand(CLICommand.java:231)
	at com.sun.enterprise.admin.cli.AdminMain.executeCommand(AdminMain.java:370)
	at com.sun.enterprise.admin.cli.AdminMain.doMain(AdminMain.java:305)
	at com.sun.enterprise.admin.cli.AdminMain.main(AdminMain.java:207)

@CSchulz
Copy link
Author

CSchulz commented Apr 25, 2018

Seems to be related with Java 9.

@dmatej
Copy link
Contributor

dmatej commented Jul 30, 2018

Current status (edited 31. 7. 2018 20:20 CEST):
fails with:

  • payara-ci-managed (NPE)
  • payara-micro-managed (with JDK9/10 as default java in system and arquillian-payara-micro-5-managed 1.0.beta3-m2)
  • glassfish-embedded (request timeout)
  • tomcat-ci-managed (missing jersey-client, when fixed, problems with invalid tomcat-users.xml management, when fixed [see Invalid management of tomcat-users.xml #25], fails like glassfish-embedded with request timeout)

passes with (with JDK8 in AS_JAVA and JAVA_HOME env properties):

@arjantijms
Copy link
Contributor

payara-micro-managed (hangs, missing JAXB classes - another issue, indeed JDK9/10)

I briefly tried getting the repo to work with JDK9 a while back, but it kept failing. For now everything is JDK 8 only, but I'd really like to get everything running on at least JDK 11 when that comes out soon.

dmatej added a commit to dmatej/javaee8-samples that referenced this issue Jul 30, 2018
- throws IllegalStateException if it cannot get session in 5 seconds
- (fail fast as possible)
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Jul 30, 2018
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Jul 30, 2018
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Jul 30, 2018
- added safety timeouts - test hangs sometimes.
- added assert for header even in the public variant
- shows the content of the war file in logs
- removed unuseful exception declarations
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Jul 30, 2018
- if injection is not supported by the container, shows meanungful error, not NPE
@dmatej
Copy link
Contributor

dmatej commented Jul 30, 2018

I expect there will be a bunch of work with the classloaders in Payara, they protect their contexts before each other and it is very complicated (and OSGI between that). But I think Payara team will move it soon ...
I have a question: which technologies should be supported also with Tomcat? Many modules crash, f.e. jaxrs - I'm not sure if it should be skipped or if I should add jersey-client and then find out how to fix it (perhaps together with the embedded-glassfish which ends up in same state then).

The problem with payara-micro-managed is that it seems it uses default java program from the OS, it should use (in this priority order) ${AS_JAVA}/bin/java or ${JAVA_HOME}/bin/java or default java.

dmatej added a commit to dmatej/javaee8-samples that referenced this issue Jul 30, 2018
- throws IllegalStateException if it cannot get session in 5 seconds
- (fail fast as possible)
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Jul 30, 2018
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Jul 30, 2018
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Jul 30, 2018
- if injection is not supported by the container, shows meanungful error, not NPE
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Jul 30, 2018
- added safety timeouts - test hangs sometimes.
- added assert for header even in the public variant
- shows the content of the war file in logs
- removed unuseful exception declarations
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Nov 3, 2018
- throws IllegalStateException if it cannot get session in 5 seconds
- (fail fast as possible)
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Nov 3, 2018
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Nov 3, 2018
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Nov 3, 2018
- added safety timeouts - test hangs sometimes.
- added assert for header even in the public variant
- shows the content of the war file in logs
- removed unuseful exception declarations
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Nov 3, 2018
- if injection is not supported by the container, shows meanungful error, not NPE
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Nov 3, 2018
- Payara + Glassfish + Arquillian + h2 + Tomcat
- fixes failing build of the payara-micro-managed profile
  - using fixed arquillian-payara-micro-5-managed:1.0.Beta3-m4
dmatej added a commit to dmatej/javaee8-samples that referenced this issue Nov 3, 2018
- Payara + Glassfish + Arquillian + h2 + Tomcat
- fixes failing build of the payara-micro-managed profile
  - using fixed arquillian-payara-micro-5-managed:1.0.Beta3-m4
Pandrex247 pushed a commit to Pandrex247/javaee8-samples that referenced this issue Jan 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants