Skip to content
This repository was archived by the owner on Jul 18, 2024. It is now read-only.

fix docker-compose.yml link #18

Merged
merged 2 commits into from
Jun 28, 2017
Merged

fix docker-compose.yml link #18

merged 2 commits into from
Jun 28, 2017

Conversation

loafyloaf
Copy link
Member

No description provided.

@@ -34,7 +34,7 @@ $ sudo docker rm gitlab

## Multiple Container deployment via Docker Compose

It's easy to connect gitlab to external PostgreSQL and Redis services by making a few tweaks to the environment variables passed to the gitlab container. In the root of this repo is a [docker-compose.yml](docker-compose.yml) file which will demonstrate a multiple container deployment that you can run locally or against a remote docker server.
It's easy to connect gitlab to external PostgreSQL and Redis services by making a few tweaks to the environment variables passed to the gitlab container. In the root of this repo is a [docker-compose.yml](../../docker-compose.yml) file which will demonstrate a multiple container deployment that you can run locally or against a remote docker server.
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the correct link should be ../docker-compose.yml

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, of course. new patch inbound

Copy link
Contributor

@Tomcli Tomcli left a comment

Choose a reason for hiding this comment

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

can you make sure the link is correct?

Copy link
Contributor

@Tomcli Tomcli left a comment

Choose a reason for hiding this comment

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

LGTM

@animeshsingh
Copy link

@loafyloaf so we are just getting the 2nd PR in - i believe we decided we dont need the first one? or do we need pieces from first one?
image

@loafyloaf
Copy link
Member Author

I've closed #16

@animeshsingh animeshsingh merged commit 67c1b0b into IBM:master Jun 28, 2017
@loafyloaf loafyloaf deleted the compose.yml_link branch June 28, 2017 22:41
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