-
Notifications
You must be signed in to change notification settings - Fork 120
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
File-Event propagation stopping (worse in current stable/edge) #2417
Comments
Thanks for the ticket -- I'm hoping to investigate event propagation soon. |
I don't get any |
@pkyeck yes, thats exactly the point. Those inotify events are not triggering and since we use unison to sync, that one is not triggered ( and you might use the usual node tooling watch tasks failing the same way ) |
Any updates on this one? I guess this is a serious problem that should be resolved as soon as possible. If there is no time, the mount protocol should be reverted to old one. |
An update here, mac55 with the fallback to raw did not help with this issue, this makes it clear, that those issues have rather to do with the new linux-distro from +42 and maybe specific ulimit/sysctl values which are now to limiting? At least, wanting to share that mac55 is no fix for that one, since this issue is on High Sierra and Sierra, APFS and non APFS |
Any updates on this issue? This is a blocker for those of us developing inside docker containers in macos to upgrade beyond 17.09 due to the broken inotify issues reported here. |
I think many people have switched to using the unison method instead. I've been using it with the latest docker CE (60) and it has worked fine AFAICT. |
@baohx2000 by no means that is a solution, thats a trade of issues, and when talking about unison, usually far worse issues - the reason for that is obviously the file-watcher implemented for unison ( unox ). |
D'oh! I must have missed that! |
Thanks for the info @baohx2000 - I did try using the unison strategy via the excellent docker-sync but the experience was less than ideal: it took a while to install Currently the perfect solution in terms of ease of setup and IO performance is using edit: clarification that we had issues with unox and macos file system change monitoring leading to high CPU and not with unison. |
@nazar @baohx2000 to clarify, that CPU usage does not come from unison, but from unox, the file watcher which triggers unison - but you need that one to run unison on OSX - that is exactly why native_osx has been created, to avoid any file watch on OSX since they all are fairly bad in performance due to the FS-Layer having aweful events. |
We're really struggling with this. Is there a known combination that works, using anything newer than 17.09? Is there anything we can do to help provide additional data? We have a couple of mac's we can test on. |
I discovered one possible way the event stream from |
@djs55 Thanks. Installed on two developer macbooks and first impressions from both devs are that it's better, i.e. no fs events have stopped. I will report back after some more intensive use. |
@djs55 Unfortunately I have a report from a dev on the team of the fs events stopping again, after a few hours use on the mac63 release. A restart of the d4m app hasn't helped. This is a new touchbar model with APFS (though earlier comments indicate this isn't a factor), with a qcow image. Truly appreciate the effort. |
@djs55 I have been running this for a whole day on my mid-2015 MBP. No issues yet, running with a .raw image though. |
I just want to chime in here guys with a reference to #2375. Since getting my new MBP running with APFS, I haven't felt the need for docker-sync with respect to speed, but I have a somewhat weird issue of some containers seemingly receiving notification of changes to files and others not. Eg. my Python containers notice changes to Python files, while my Webpack devserver container doesn't notice changes to any source files (JS, SCSS mainly). I wouldn't doubt that's because of different container images (and OSes) or possibly different file system watch strategies, but… In the comments to the aforementioned issue there was an interesting note of starting a separate container with Also running .raw image, on macOS 10.13.4 with APFS and Docker 18.05.0-ce-rc1-mac63 (24246). |
@fdanielsen interesting. What file editor you're using to edit the js and scss files? There's a thing with Sublime and maybe Atom (and potentially other file editors) where they do not save files immediately and therefore the event can't occur. Not sure how a separate container would fix this though. |
@michalkleiner I guess the editor does not matter. I’ll try @fdanielsen workaround when my fs-events dies (hope it won't happen) and let you know the results. |
@michalkleiner I'm using vim. I read in Webpack's documentation that the |
We are experiencing the same issue as @fdanielsen. Most projects appear to propagate filesystem changes fine, but a large ember project (~100k files in node_modules) consistently fails to pick up file changes. Recently a few developers have also reported issues with a smaller create-react-app based project. The ember app is not running along side in that case. Edge build 18.05.0-ce-rc1-mac63 doesn't appear to have fixed the problem. |
@djs55 I am using edge version and hasn't experienced issues yet with docker-sync. I'll try to update again on this issue after a longer period of time. I do not use dummy-container workaround. |
@djs55 we have quiet some testing on docker-sync with edge63 right now. Results are basically a) its better then anything after mac42 stable - but its worse then mac42 (significantly) You can see the test results and scenarios here EugenMayer/docker-sync#517 (comment) - several people have been testing it. Hope that helps to at least mark the progress. I am really looking forward to see a progress here :) |
mac63+raw without docker-sync (using :cached etc) has been "good enough" for developers in my team to use it continuously, so I'd encourage anyone reading to try it and see if it's stable enough for your own use. I can confirm that our experience echoes that mac63+qcow are unusable for fsevents. |
Have you tried NFS share? It is similar setup like docker-sync (using an extra volume), but synchronization is done by NFS in macOS. This link helps setting it up: https://medium.com/@sean.handley/how-to-set-up-docker-for-mac-with-native-nfs-145151458adc Our benchmark shows significant performance boost comparing to :cached (nfs 40ms vs cached 150ms response time), but it is done on very small php project. Haven't tried it on larger-scale projects. |
@michalkleiner For bug fixes/improvements we try to update the release notes (e.g. https://docs.docker.com/docker-for-mac/edge-release-notes/) with links to the issues. However I think this isn't 100% reliable at the moment -- we need to work on our internal processes to make this a bit better I think. Having said that, when we make a major bump to stable we usually base it on the last released edge, so all the fixes already in edge should make it across. |
@djs55 - I've updated to Thank you for the fix in stable - it's very much appreciated 👍 |
Issues go stale after 90d of inactivity. Prevent issues from auto-closing with an If this issue is safe to close now please do so. Send feedback to Docker Community Slack channels #docker-for-mac or #docker-for-windows. |
please reopen this issue, it has been by now means fixed or done, thanks |
/remove-lifecycle stale |
/lifecycle frozen |
Still closed / frozen? |
I'm still experiencing this issue as well. It should be reopened. |
Same here. File propagation works well until a large number of files are updated on the HOST - i.e. I have to restart docker on High Sierra anytime I run |
We encountered this Issue and we came to a conclusion, that it is the issue with the docker dmg file and install process on docker website. If you use mac install the docker with: |
@petkoneo are you suggesting that via brew the installation is somehow different to the dmg installation and one doesn't have file events propagation issues compared to the other? Can you reliably replicate that? |
Yes but it is a suggestion. For now, we could resolve this problem on 2 computers. As far as I know the installation should be the same, but it still worked for us. (We were surprised as well) maybe it just needs a proper re-install. |
Reinstalling docker user brew cask didn't make a difference for me. Same problem :( |
C18223FD-44DC-4ACD-B197-B660FC33917F/20191128104037 |
C18223FD-44DC-4ACD-B197-B660FC33917F/20200430200341 |
@daveisfera what are these hashes for? |
They're the id from when I uploaded the diagnostics when this error is happening |
C18223FD-44DC-4ACD-B197-B660FC33917F/20200616212522 |
Closed issues are locked after 30 days of inactivity. If you have found a problem that seems similar to this, please open a new issue. Send feedback to Docker Community Slack channels #docker-for-mac or #docker-for-windows. |
I am maintaining http://docker-sync.io where we try to overcome some effects host-mounts in d4m ( docker on OSX in general ) spawn, specifically the performance.
We offer different ways ( rsync, unison ) and since about 1 year, OSXFS based "native_osx" - this relys on the FS events propagated by d4m - please see the chart here https://github.com/EugenMayer/docker-sync/wiki/8.-Strategies#native_osx-osx
a) It is known, that the propagation does stop working after some time, even before mac42: EugenMayer/docker-sync#410 - the only way we fix that is by restarting d4m completely - are there any better ways to that?
b) (the actual reason for this issue, but related to a) Now, since 42+ ( 42 works well ) things got a lot worse, that said, FS even propagation stops after some few operations or just some time, see EugenMayer/docker-sync#517
People do downgrade to mac42 EugenMayer/docker-sync#517 (comment) and seem to have far better results
Referencing to your statement
a) can we "restart" that service, when it stuck? Without restarting all containers
b) can we fix FS system propagation?
c) can we somehow detect when FS propagation has been stuck ( to trigger a )
Thanks
The text was updated successfully, but these errors were encountered: