Skip to content
This repository has been archived by the owner on Jul 3, 2020. It is now read-only.

docker-sync #10

Closed
wants to merge 4 commits into from
Closed

docker-sync #10

wants to merge 4 commits into from

Conversation

arosenhagen
Copy link
Contributor

implement docker-sync
use with unison to really speed up things on macOS

@bvogel
Copy link

bvogel commented Jul 17, 2018

I really like the idea behind docker-sync, but this should be optional.

@arosenhagen
Copy link
Contributor Author

the only way to make this optional is to have a separate docker-compose.yml

@shyim
Copy link

shyim commented Jul 18, 2018

This project uses psh. Whats about a command which generates a docker-compose.override.yml for the mac users?

@bvogel
Copy link

bvogel commented Jul 18, 2018

Point is that docker-sync is not required with neither Windows nor Linux and additionally the need to install ruby (which is a real hassle on Windows) just to get this to work will put off a couple of users.

@arosenhagen
Copy link
Contributor Author

I like the idea of docker-compose.override.yml(now that I read about it) - but I don't know how to integrate it cleanly. There has to be a setting on which one can turn on/off docker-sync - either as parameter to psh or in .psh.yaml, right?

@shyim shyim mentioned this pull request Jan 13, 2019

syncs:
#IMPORTANT: ensure this name is unique and does not match your other application container name
app_server-sync: #tip: add -sync and you keep consistent names as a convention
Copy link
Collaborator

Choose a reason for hiding this comment

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

Hey, please add the right sync_userid and sync_groupid. Also, exclude the unnecessary files e.g.
sync_excludes: ['.git', '.idea', 'dev-ops/docker/_volumes', 'var', '.docker-sync'].

Copy link
Collaborator

Choose a reason for hiding this comment

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

You can see an example here: shopware/development@d161be8

@ffuenf ffuenf closed this by deleting the head repository Aug 8, 2023
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.

None yet

6 participants