Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Docker container crashes at startup #518

Closed
vmax opened this issue Jun 2, 2021 · 12 comments · Fixed by #519
Closed

Docker container crashes at startup #518

vmax opened this issue Jun 2, 2021 · 12 comments · Fixed by #519
Labels
Bug Confirmed bug in Oxalis. Pri/L Low priority

Comments

@vmax
Copy link
Contributor

vmax commented Jun 2, 2021 •
edited
Loading

Attempting to start norstella/oxalis:5.0.1 image (latest at the time) throws an error:

2021-05-29 19:01:52,392  INFO [network.oxalis.server.Main] Starting server 
Exception in thread "main" java.lang.IllegalStateException: StatisticsHandler has no Wrapped Handler
	at org.eclipse.jetty.server.handler.StatisticsHandler.doStart(StatisticsHandler.java:251)
	at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:73)
	at org.eclipse.jetty.util.component.ContainerLifeCycle.start(ContainerLifeCycle.java:169)
	at org.eclipse.jetty.util.component.ContainerLifeCycle.doStart(ContainerLifeCycle.java:117)
	at org.eclipse.jetty.server.handler.AbstractHandler.doStart(AbstractHandler.java:97)
	at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:73)
	at org.eclipse.jetty.util.component.ContainerLifeCycle.start(ContainerLifeCycle.java:169)
	at org.eclipse.jetty.server.Server.start(Server.java:423)
	at org.eclipse.jetty.util.component.ContainerLifeCycle.doStart(ContainerLifeCycle.java:110)
	at org.eclipse.jetty.server.handler.AbstractHandler.doStart(AbstractHandler.java:97)
	at org.eclipse.jetty.server.Server.doStart(Server.java:387)
	at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:73)
	at network.oxalis.server.Main.run(Main.java:82)
	at network.oxalis.server.Main.main(Main.java:58)

I'm starting the container by bind-mounting keystore file & a config which only contains oxalis.keystore section (for passing in password)

UPD: oxalis-server-5.0.1-full.tar.gz distribution is also affected
This was referenced Jun 2, 2021
Merged
Closed
@aaron-kumar
Copy link
Member

Please refer reply : OxalisCommunity/Oxalis-AS4#156 (comment)

@vmax
Copy link
Contributor Author

vmax commented Jun 3, 2021

@aaron-kumar I've discovered that not only the Docker image is affected, but also the release hosted on GitHub Releases

@aaron-kumar
Copy link
Member

aaron-kumar commented Jun 3, 2021 •
edited
Loading

@vmax : Yes, I can see problem there and I do not know historical reason for not adding statisticsHandler correctly in Jetty specific Main class.

I have gone through PR : 4095314 and will merge that soon.

@aaron-kumar aaron-kumar added Bug Confirmed bug in Oxalis. Pri/L Low priority labels Jun 3, 2021
@gozus19p
Copy link

Hi guys!

I am trying to upgrade Oxalis from 4.1.1 to 5.0.1 and I am facing off the same issue as @vmax.
We are trying to refer to Docker Hub norstella/oxalis-as4:5.0.1.

At the moment, I have rolled back to the version previously installed on our systems: digdir/oxalis-as4:4.1.5.

I would like to kindly ask you for an information.
I must plan the Oxalis upgrade date, in order to organize our development team.
Do you have any idea when the pull request will be merged and made available on Docker Hub?
I am asking this just to understand if it is better, for us, to proceed in another way in the meantime.
We would prefer to wait for the official release.

I thought to upgrade from 4.1.5 to 5.0.0, but 5.0.0 version contains a bug on PEPPOL lookup process (fixed in 5.0.1). So, I would go for 5.0.2, maybe.

Many thanks in advance.
I wish you a great day.

Best regards,
Manuel

@aaron-kumar
Copy link
Member

I will merge the change soon (probably this weekend) and create latest artifact.

Please verify that latest artifact, available in docker hub. Once you found that it is working as expected, I will create release e.g. 5.0.2.

@gozus19p
Copy link

Thank you so much. I will try to install the latest artifact on Monday. I will let you know the result as soon as possible.
Have a nice day!

@aaron-kumar
Copy link
Member

aaron-kumar commented Jun 14, 2021 •
edited
Loading

@gozus19p : Please check latest docker hub release (norstella/oxalis:latest) . FYI: @vmax

@gozus19p
Copy link

Hi Aaron,

thanks for the update!
I confirm you that I installed the latest Docker Hub release. I found no problems during Docker startup.

I wish you all a great day.

Best regards,
Manuel

@aaron-kumar
Copy link
Member

@gozus19p & @vmax : This change will be part of 5.0.2 release - I can't commit date now but will be released soon.

@aaron-kumar
Copy link
Member

@gozus19p & @vmax : fix released as part of Oxalis 5.0.3. Docker : norstella/oxalis:5.0.3

@vmax
Copy link
Contributor Author

vmax commented Jun 20, 2021

@aaron-kumar thanks! any chance we could also get a new version of norstella/oxalis-as4 too up on DockerHub? Then OxalisCommunity/Oxalis-AS4#156 would be fixed too 🙂

@aaron-kumar
Copy link
Member

@vmax : Here you go : norstella/oxalis-as4:5.0.2

Either I posted my earlier release message too quickly or you responded too quickly (I was not expecting early response due to weekend) 🙂

Please verify and let me know if any issue etc...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Confirmed bug in Oxalis. Pri/L Low priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix StatisticsHandler initialization oppgjorskontoret/oxalis
3 participants
@vmax @aaron-kumar @gozus19p