-
-
Notifications
You must be signed in to change notification settings - Fork 290
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
Unison invoked incorrectly too many roots #103
Comments
Could you paste the output of FYI, if not provided |
Sure! This is with verbose: true:
|
Thanks got it, there is a problem with |
sw_vers -ProductVersion is 10.11.6.. |
Mmm, sorry, have to guess which |
Ha sure -- I don't think I have anything installed locally that would override the OS X installation of
|
@mickaelperrin FWIW other folks on my team here at http://www.savaslabs.com are seeing the same issue on their machines |
And please, what |
|
Thanks a lot, have sufficient informations to work on patch. |
Thanks!! 🌠 |
Thanks for reporting, and sorry for the issue... PR #104 resolves it. I will see with @EugenMayer to quickly release a new version. |
…ibility [BUGFIX #103] Unison reports too many roots
@mickaelperrin thank you and also @timstallmann thank you for taking your time to report that detailed 0.1.1 has just been release. Well done ! |
Sorry to re-open this one, but I have the same issue:
docker-sync.yml
docker-compose-dev.yml
And here is the ouput of
|
Thanks for reporting, Could you provide the output of :
Thanks |
sed --version /usr/bin/sed --version /usr/local/bin/sed --version Not much info here I'm afraid ;) |
How did you install |
With gem:
|
What's the output of Could you enable the EDIT: change the sync container name |
And could you give also the version of docker-sync used ? |
I dropped all containers, rebooted and started again, and replace |
Great 👍 |
@michael See my lastest checkin were i introduced 127.0.0.1 and removed localhost |
This seems to happen when you have a volume that exists that matches your syncs that isn't unison. I had a volume left as rsync since it's the default strategy and this error was happening because of it. |
Having a similar issue this was my output when
I too had a rsync container before I switched so I just shutdown everything and restarted....but still no dice even though I changed the name. I'm using latest docker-sync(0.1.4) |
@karneaud please open a new issue, i think yours is different. Please read the migration guide of rsync -> unison (the syntax of excludes is different ). But lets cover that in a different issue |
I've got the same error. And I have tried to reboot but it doesn't help docker-compose.ymlversion: '2'
services:
ai:
build: .
command: /opt/conda/bin/jupyter notebook --notebook-dir=/opt/notebooks --ip='*' --port=8888 --no-browser
ports:
- "8888:8888" docker-compose.dev.ymlversion: '2'
services:
ai:
volumes:
- ai-sync:/opt/notebooks/:rw
volumes:
ai-sync:
external: true docker-sync.ymloptions:
verbose: true # make the whole sync verbose
syncs:
#IMPORTANT: ensure this name is unique and does not match your other application container name
ai-sync: #tip: add -sync and you keep consistent names als a convention
src: './'
dest: '/opt/notebooks'
sync_strategy: "unison"
sync_excludes: ['.*/.git']
sync_host_ip: '127.0.0.1' #localhost gives the same result
sync_host_port: 10872 result of
|
ok, It seems I have move one step farther - I have removed sync volume (it was initially created with rsync) and now I get new error:
|
ok, I got where was problem, I have used incorrect
it won't be stable and can change from time to time. Maybe it will be better way to use docker-machine name instead? |
Just started seeing this issue on the 0.1.0 branch. I updated my docker sync config per the migration docs.
docker-sync.yml
contents:docker ps output
:I've tried this with and without the
sync_host_ip
and that flag doesn't seem to impact the performanceThe text was updated successfully, but these errors were encountered: