Skip to content
This repository has been archived by the owner on Apr 28, 2023. It is now read-only.

Make docker compose setup self contained #581

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

KonradHoeffner
Copy link

@KonradHoeffner KonradHoeffner commented Apr 20, 2022

Adds the ols-config-importer and the ols-indexer to the docker compose setup.
Instead of the multi-step method, a single docker-compose up is all that is needed.
See #575.

@matentzn
Copy link
Contributor

If this works I will cry a few tears of joy..

@jamesamcl
Copy link
Member

@matentzn it does but it means you have to run the config importer and indexer every time you start OLS. I think this is probably fine though as OLS only reindexes ontologies that have changed.

@matentzn
Copy link
Contributor

Forgot about the indexing - I always had to redo it anyways, but yes, this is a problem! This wont work for a setup like ebis with many ontologies - but most external use cases load a handful of ontologies into OLS. This interplay of docker containers for services and indexing is a real nightmare.

@KonradHoeffner
Copy link
Author

I could try adding profiles so that it could be configured whether the importer and indexer should be run or not.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants