search

goat-community / goat

This is the home of Geo Open Accessibility Tool (GOAT)
GNU General Public License v3.0
89 stars 47 forks source link

Check geoserver image #515

Closed EPajares closed 4 years ago

EPajares commented 4 years ago

We are currently using this docker image: https://hub.docker.com/r/oscarfonts/geoserver/dockerfile

We should double check if the Tomcat settings are appropriate for our deloyments. It could be that there is the need to increase the heap-memory here. Some further recommendations to improve performance are documented here:

https://docs.geoserver.org/latest/en/user/production/index.html

EPajares commented 4 years ago

@alfredopalhares

alfredopalhares commented 4 years ago

Do you want to build image based on that one, I see that we can increase the java heap size on a very simplistic way, yet it does not seem to check multiple flags.

EPajares commented 4 years ago

Yes I would say we should build on this image. Right it would be quite simple to pass the java heap size as additional variable. I guess we could do so in a fork. These changes might be relevant also for the maintainer of the image so we could check to do a PR on his repo after the changes.

EPajares commented 4 years ago

@alfredopalhares we would like to use this version of Geoserver: 2.16.2

alfredopalhares commented 4 years ago

@EPajares can you test the image here

EPajares commented 4 years ago

Thanks Alfredo! The image is working and the container is starting. Though I am having an issue with the data that should be mounted to the container. We are currently mounting this folder: https://github.com/goat-community/goat/tree/development/app/geoserver/geoserver_data

We are getting the following logs: " Found Java environment variable GEOSERVER_DATA_DIR set to /var/local/geoserver , which is not writeable"

EPajares commented 4 years ago

It seems to be a permission error. Basically when you start the service on localhost/geoserver and login with the credentials you should see our layers being loaded as on the following screenshot but instead this dashboard is empty.

grafik

alfredopalhares commented 4 years ago

I see, its the file that permission that should have the tomcat UID, I add some modifications, can you retest again please ?

EPajares commented 4 years ago

We fixed a type and it works now. I will do some further performance test tomorrow . But everything seems to be fine!

alfredopalhares commented 4 years ago

Awesome!

EPajares commented 4 years ago

I tested it! Everything is fine. I think we can close this issue.