Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Labkey app #2

Merged
merged 5 commits into from
Jun 5, 2019
Merged

Labkey app #2

merged 5 commits into from
Jun 5, 2019

Conversation

mamelara
Copy link
Contributor

@mamelara mamelara commented Jun 5, 2019

Updates the documentation in the README and adds a locally working docker-compose with Dockerfile. Still needs to be tested out in the SPIN world.

There was an issue setting it up between the NFS docker volume mounting and the postgres database. Also issue with getting labkey.xml to find the postgres database. The volume mounting issue was due to the fact that docker volumes are persistent and I couldn't create the postgres superuser. Solution was here: docker-library/postgres#41 (comment) (simply delete the volume mount).

I used the links keyword to give the container an alias so that we can provide that as an URL in labkey.xml (probably not necessary but it worked).

The tarball for Labkey was too big to upload so make sure to have that present and extracted if you want to try this out first. I separated out some of the directories (look at the README) and if you look at the Dockerfile, you can see how they are being copied over.

mamelara added 5 commits June 4, 2019 17:53
The Dockerfile follows the instructions for setting up LabKey. It loads
files from the tarball into the appropriate directories.
Removes db directory since we are using the base image for Postgres
Removes web directory since we are placing the labkey server into the
tomcat container. Much easier to run.
@mamelara mamelara requested a review from benbowen June 5, 2019 01:03
@benbowen
Copy link
Member

benbowen commented Jun 5, 2019

Thanks Mario! I'll read for understanding and merge prior to our meet-up tomorrow.

@benbowen benbowen merged commit 1ec0920 into master Jun 5, 2019
@benbowen benbowen deleted the labkey-app branch June 5, 2019 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants