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

[WIP] Dockerise Colouring London #780

Draft
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

edwardchalstrey1
Copy link
Contributor

@edwardchalstrey1 edwardchalstrey1 commented Feb 14, 2022

Add a docker version of the colouring london software that can be quickly deployed - currently this will load test map data from OSM only (WIP)

To run app:
docker compose up

@edwardchalstrey1 edwardchalstrey1 changed the title WIP: Dockerise Colouring London [WIP] Dockerise Colouring London Feb 14, 2022
@matkoniecz
Copy link
Contributor

Are you stuck on something specific here? Or is it just WIP? Not sure should I try looking how it works (or how it does not work)

@edwardchalstrey1
Copy link
Contributor Author

edwardchalstrey1 commented Feb 21, 2022

Are you stuck on something specific here? Or is it just WIP? Not sure should I try looking how it works (or how it does not work)

@matkoniecz still a WIP, I made a separate PR for docker specifically as I was still working on getting my manually set up dev environment (#776) working correctly first and merging the documentation changes that make that work first. You can feel free to experiment with this docker setup if you like, but I plan to continue with it next.

Unlike the Vagrant version, this does not have Node issues which is good! but I do not have the postgres container working yet

RUN chmod +x /wait

RUN mkdir /colouring-london
COPY app /colouring-london/app
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see comment

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