search

okfn / docker-ckan

Docker images and Docker Compose setup for CKAN [Not Maintained]
GNU Affero General Public License v3.0
81 stars 91 forks source link

added solr volume #30

Closed hvwaldow closed 5 years ago

hvwaldow commented 5 years ago

This is opinionated. For me it seems clear that the Solr index in production should not be lost when the container is re-created. So I added a volume for the index. Feel free to close without comment, if you see things differently.

amercader commented 5 years ago

@hvwaldow I never close issues without comment :)

This seems reasonable, but is worth mentioning in the docs. I have added a note in the README in case it affects someone's workflow.

hvwaldow commented 5 years ago

Good, but docs are slightly out of sync with codenow: We should either change the docs to state that this only applies to docker-compose.yml or make the index persistent also in docker-compose.dev.yml.

I would prefer the latter (https://github.com/okfn/docker-ckan/pull/31)