Currently, the geoserver data directory is downloaded when the geoserver_data container is being created:
|
ADD download.sh ${TEMP_DOWNLOADED} |
|
RUN chmod +x ${TEMP_DOWNLOADED}/download.sh |
|
RUN ${TEMP_DOWNLOADED}/download.sh $GEOSERVER_VERSION $TEMP_DOWNLOADED |
The data dir is a zip hosted at https://artifacts.geonode.org/geoserver/:
|
echo "GeoServer Data Directory is going to be downloaded" |
|
artifact_url="https://artifacts.geonode.org/geoserver/$GEOSERVER_VERSION/geonode-geoserver-ext-web-app-data.zip" |
|
echo "Downloading: $artifact_url" |
Furthermore, for what I can see, the geoserver_data service's only job is to download that zip to provide it as shared volume to be available for other services, like django and geoserver.
|
VOLUME ${BASE_GEOSERVER_DATA_DIR}/data |
This however, forces the container to be up and running. Otherwise, the volume is not available anymore.
I have several questions:
- Is there any reason (I cannot see), why we do not host the data dir in this repository?
- Do you have particular requirements which argue for a dedicated
geoserver_data container?
- What do you think about
- adding the data dir to the
geoserver container
- removing the
geoserver_data image completely
- move shared volume provisioning to
geoserver container
Anything else, I miss?
/cc @mwallschlaeger
Currently, the geoserver data directory is downloaded when the
geoserver_datacontainer is being created:geonode-docker/docker/geoserver_data/Dockerfile
Lines 14 to 16 in cf60723
The data dir is a zip hosted at https://artifacts.geonode.org/geoserver/:
geonode-docker/docker/geoserver_data/download.sh
Lines 14 to 16 in cf60723
Furthermore, for what I can see, the
geoserver_dataservice's only job is to download that zip to provide it as shared volume to be available for other services, like django and geoserver.geonode-docker/docker/geoserver_data/Dockerfile
Line 25 in cf60723
This however, forces the container to be up and running. Otherwise, the volume is not available anymore.
I have several questions:
geoserver_datacontainer?geoservercontainergeoserver_dataimage completelygeoservercontainerAnything else, I miss?
/cc @mwallschlaeger