-
Notifications
You must be signed in to change notification settings - Fork 94
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
docker - unzip webapps in the container #2849
Conversation
See #2644 and others for the motivation / inspiration. tests: runtime tested a generated image * unzipped OK * docker run seems OK (webapp loaded, waiting for postgres to come alive)
Tests: * docker run bash: webapp unzipped in webapps/ * docker run: waiting for the db to become ready
Tests: * docker run bash: webapp unzipped in webapps/ OK * docker run: webapp init (but fails because datadir missing) OK
tests: same as on the other ones
To be reworked after #2848 is fixed, as i am not sure we have to work with the actual webapp, maybe the target/geowebcache directory does not contain every needed resources.
Tests: same as previously on the other ones
Tests: same as previously
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me Pierre, thanks !
Shall we consider merging, knowing the fact that it won't address #2848 ? |
As discussed, we can consider merging (once the checks are OK). |
Pending travis build is actually green, merging. |
This makes the customization (via volumes or so) easier.
inspired from what as already been done on CAS & SP, like the following PR:
https://github.com/georchestra/georchestra/pull/2644/files
Note: I encountered an issue while working on this, which might be worth waiting for a fix:
#2848
Tests:
Each module tested with
mvn clean package docker:build
, then manually checking that:/var/lib/jetty/webapps