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.

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

[Errors] We should not try to launch the process if no war files are found. #15

Open
johnsudaar opened this issue Apr 14, 2020 · 4 comments · May be fixed by #28
Open

[Errors] We should not try to launch the process if no war files are found. #15

johnsudaar opened this issue Apr 14, 2020 · 4 comments · May be fixed by #28
Assignees

Comments

@johnsudaar
Copy link

Currently if no war file is found, we fail with the following error message:

2020-04-13 09:49:41.625968226 +0000 UTC [web-1] Picked up JAVA_TOOL_OPTIONS: -Xmx300m -Xss512k -XX:CICompilerCount=2 -Dfile.encoding=UTF-8
2020-04-13 09:49:41.788248634 +0000 UTC [web-1] The specified path "./*.war" does not exist.
2020-04-13 09:49:41.791431613 +0000 UTC [web-1] Usage: <main class> [options]
[...] (Rest of the java help method)

We should fail with a better error message saying that no war files were found and link them to the Procfile docs.

@johnsudaar
Copy link
Author

@Frzk
Copy link
Contributor

Frzk commented Jul 5, 2023

@johnsudaar : Would it be OK to do the check in the bin/release script?

It feels a bit weird to me.

@EtienneM
Copy link
Member

EtienneM commented Jul 6, 2023

I think it should be checked earlier, in the bin/compile script

@Soulou Soulou transferred this issue from Scalingo/java-buildpack Sep 12, 2023
@Soulou
Copy link
Member

Soulou commented Sep 12, 2023

Discussion with Françosi what has been decided:

  • A method to find the default war file should be created
    • During bin/compile it should be called to mention which war file will be used during execution, and give a hint to modify WAR_PATH to use another one. If no .war could be found, the deployment should fail.
    • During bin/release a wrapper script should be used, created in bin/start-war-app which will re-use the find method or directly start the app defined in WAR_PATH

@Frzk Frzk self-assigned this Oct 9, 2023
@Frzk Frzk linked a pull request Feb 21, 2024 that will close this issue
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

Successfully merging a pull request may close this issue.

4 participants