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

docker quickstart #19

Merged
merged 3 commits into from
Aug 22, 2018
Merged

docker quickstart #19

merged 3 commits into from
Aug 22, 2018

Conversation

licarth
Copy link
Contributor

@licarth licarth commented Aug 20, 2018

This PR adds a quickstart to run opla with docker and docker-compose. See #18 for details.

Todo:

  • adapt CI to circleCI (instead of travis) docker is easier on travis (circleCI does not support connecting to the containers via localhost, travis is closer to dev machine setup than circleCI) keeping it for now.
  • optional could publish nginx image to avoid having to build it locally ==> no need for that, docker-compose --build is fast enough, easier to maintain.

@licarth licarth changed the title docker quickstart [WIP] docker quickstart Aug 20, 2018
@licarth licarth force-pushed the docker branch 24 times, most recently from e0e9390 to 55991ad Compare August 21, 2018 09:20
@licarth licarth force-pushed the docker branch 4 times, most recently from 7345ce0 to 56eae76 Compare August 21, 2018 09:52
@licarth licarth changed the title [WIP] docker quickstart docker quickstart Aug 21, 2018
@licarth licarth requested a review from mikbry August 21, 2018 10:02
@licarth licarth force-pushed the docker branch 2 times, most recently from 1cc1f72 to b35c19e Compare August 21, 2018 10:37
#!/bin/bash

# It is important for this not to output anything before it actually IS connected
# in order for Travis to stop the build if this retries forever (no stdout, Travis
Copy link
Contributor

Choose a reason for hiding this comment

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

Travis is not used anymore ?

@mikbry mikbry merged commit 2f3391f into master Aug 22, 2018
@licarth licarth deleted the docker branch August 22, 2018 13:13
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.

2 participants