-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
finalize docker-compose.yml
for v2
#661
Conversation
But HTTPS url which shown in the "project detail page" would been wrong.
Ideally I would have liked to use a I have also applied the |
This would possible if you use a volume plugin like local-persist |
Thanks for linking to |
@sameersbn. I put my test results in here as you mentioned me in #658 . I guess it will provide a litte bit more clarity.
I used the env variables from the compose file. In that case we sould use env_file to define them once. Lets rethink about #600. |
I guess the redis client version in the gitlab image may not be compatible.
Wouldn't you use docker-compose to launch the rake task? create a backupdocker-compose run --rm gitlab app:rake gitlab:backup:create restore a backupdocker-compose run --rm gitlab app:rake gitlab:backup:restore BACKUP=1460905799
I have commented on the issue #600 (comment) |
I mean Next time i will try to use docker-compose to run the rake tasks. It will make my work a lot of easier. 😅 |
Yes. I meant the
With docker-compose v2 I think this might be the only valid way of doing things correctly because of all the networking bits. |
environment: | ||
version: '2' | ||
|
||
networks: |
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.
The networks key isn't needed because docker-compose will create an own network by default. So this could be remove @sameersbn.
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.
Yep, its not "required".
Not sure if we should keep it or remove it. Guess we can remove it for now and add it later as we see fit?
Hello, any idea when you will merge it? |
@jgsqware can you try it out and let us know it works for you? will merge it on the weekend after doing some tests to check if backup/restore operations work without issues. |
No description provided.