-
Notifications
You must be signed in to change notification settings - Fork 80
add sample compose file #877
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -158,6 +158,40 @@ multi-machine scaling. | |||||
|
|
||||||
| For organizations, we recommend one Docker host per team of 5-10 developers. | ||||||
|
|
||||||
| ## Docker Compose | ||||||
|
|
||||||
| For a convenent way to run Coder in Docker, you can use [Docker Compose](https://docs.docker.com/compose/). | ||||||
|
|
||||||
| Simply create a new directory (for example, named `c4d`) and place the following | ||||||
| content in a file named `docker-compose.yml`: | ||||||
|
|
||||||
| ```yaml | ||||||
|
||||||
| ```yaml | |
| ```yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I just tried keeping the current indentation level; is there anything magical about that here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah, maybe my eyes deceive me - I thought this line was off compared to the rest. Disregard!
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you nee the { }?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's just a habit of mine, it's probably not necessary.
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we be recommending docker compose up -d?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I intentionally chose the docker-compose syntax in case folks on older versions want to try this. The new version is backwards-compatible:
$ ls -l $(which docker-compose)
lrwxr-xr-x 1 root wheel 68 Feb 27 21:05 /usr/local/bin/docker-compose -> /Applications/Docker-4.5.0.app/Contents/Resources/bin/docker-compose
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this was something I had suggested before (#818 (comment)), I suppose if we're good with the additional maintenance then more options is always good
cc @Emyrk