Add MapStore to the docker compose stack #58
Conversation
There was a problem hiding this comment.
I finally make it works, but it's not really "intuitive"
- expose the db on port 5432
- find https://github.com/geosolutions-it/geostore/blob/master/doc/sql/002_create_schema_postgres.sql download it.
- create the schema "geostore" in geochestra DB.
- launch the composition.
What I recommend, is : launching these migrations in an entrypoint of the container. georchestra/docker should work once "docker-compose up" has been fired.
You could also let autoddl of hibernate do the job for you.
Submodule config:
> ffd124e - Set consoleAppender
> b43ba7c - Add documentation
> a5c5d70 - Indentation fix
|
I think I have some trouble with the last image of mapstore. Minor troubles, but we are very close !!! Also once this is merge, you'll be able to remove a lot of content from http://docs.georchestra.geo-solutions.it/fr/latest/georchestra_integration/ |
|
Dear @RemiDesgrange I'm sorry for the late answer. We are trying to resume the work of @Gnafu for this PR since he is not available this period unfortunately. Are you still experiencing the above issue? We will be able to check it next days. I'm sorry for the inconvenience. Then we will solve conflicts on this PR and clean up the online doc, where needed. |
| @@ -138,3 +138,13 @@ services: | |||
| - JAVA_OPTIONS=-Dorg.eclipse.jetty.annotations.AnnotationParser.LEVEL=OFF | |||
| - XMS=256M | |||
| - XMX=1G | |||
|
|
|||
| mapstore: | |||
| image: geosolutionsit/mapstore2-georchestra:latest | |||
There was a problem hiding this comment.
Would be perfect if https://github.com/georchestra/mapstore2-georchestra/blob/master/.github/workflows/mapstore.yml could generate a georchestra/mapstore image, as for the other apps.
There was a problem hiding this comment.
yup, appending something like below in https://github.com/georchestra/mapstore2-georchestra/blob/master/.github/workflows/mapstore.yml should do the trick, but only then we can replace the docker hub image in the compose:
- name: "Building docker image"
if: github.repository == 'georchestra/georchestra'
working-directory: mapstore2-georchestra/
run: ../mvnw --no-transfer-progress clean package docker:build -Pdocker,log4j-logstash,sentry-log4j -DdockerImageName=georchestra/mapstore:${{ steps.version.outputs.VERSION }} -DskipTests
- name: "Logging in docker.io"
if: github.repository == 'georchestra/georchestra'
uses: azure/docker-login@v1
with:
username: '${{ secrets.DOCKER_HUB_USERNAME }}'
password: '${{ secrets.DOCKER_HUB_PASSWORD }}'
- name: "Pushing latest to docker.io"
if: github.ref == 'refs/heads/master' && github.repository == 'georchestra/georchestra'
run: |
docker tag georchestra/mapstore:${{ steps.version.outputs.VERSION }} georchestra/mapstore:latest
docker push georchestra/mapstore:latest
- name: "Pushing release branch to docker.io"
if: contains(github.ref, 'refs/heads/20.') && github.repository == 'georchestra/georchestra'
run: |
docker push georchestra/mapstore:${{ steps.version.outputs.VERSION }}
- name: "Pushing release tag to docker.io"
if: contains(github.ref, 'refs/tags/20.') && github.repository == 'georchestra/georchestra'
run: |
docker push georchestra/mapstore:${{ steps.version.outputs.VERSION }}
There was a problem hiding this comment.
@fvanderbiest I got up to here for workflow docker credentials need to be configured:
https://github.com/georchestra/mapstore2-georchestra/runs/1158835723?check_suite_focus=true
There was a problem hiding this comment.
Please create the PR and we'll review it.
Once merged, push will be allowed
docker-compose.yml
Outdated
| volumes: | ||
| - ./config:/etc/georchestra | ||
| environment: | ||
| - JAVA_OPTS=-Xms512m -Xmx512m -XX:MaxPermSize=128m -Dgeorchestra.datadir=/etc/georchestra |
There was a problem hiding this comment.
Missing -Dgeorchestra.extensions=<mapstore_dynamic_files_folder_path> ?
There was a problem hiding this comment.
leaving it as is extensions.json and dist directory are created in /etc/georchestra/mapstore on the container which is georchestra datadir for mapstore, if this is ok no modification is needed, if a different directory is needed let me know so I commit it here
There was a problem hiding this comment.
OK, you're the mapstore expert.
I let you decide which is best ;-)
|
Hello @RemiDesgrange @fvanderbiest @landryb I see that all applications in georchestra have their ddl files in the database Docker images and databases for everything else but mapstore are initialized here: |
Correct ! |
concerning this issue, I had to build up a fresh installation yesteday of the compose for taking over the work on this PR and I did not experience the problem above |
|
@fvanderbiest I tried to see how I cuold solve the conflicts but I do not have permissions, everything else is in place to be merged IMO |
|
Cannot merge if there are conflicts ... either update this one, or create a new one. |
ok @fvanderbiest I will do a new one |
No description provided.