-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Possible cache race condition #5890
Comments
And again - https://travis-ci.org/crystal-lang/crystal/jobs/373199400#L540-L546:
|
We've never ever had any reports of this outside CI, which is extremely strange considering how often it happens on CI. Very interesting. |
It seems to only occurs in Travis? Interesting, very likely a system issue. For example increasing the value of I've also read this, not sure if it's related but we may also try to clear the caches before compiling: echo 1 >/proc/sys/vm/drop_caches
echo 2 >/proc/sys/vm/drop_caches
echo 3 >/proc/sys/vm/drop_caches
sync |
I'm not sure how to fix this. One can reproduce for example by going to the cache and changing one |
Just today at mid-day, I started getting this error consistently when trying to deploy (morning deploys were fine). I feel like I've seen it in the past, too, but at that time, it wasn't happening often enough for me to go out and look for solutions. Since this issue mentioned a race condition, I tried adding a Oh, and in my case, it's not one undefined reference error, I get pages and pages and pages and pages of these errors... |
That sounds weird. Are you sure they're not running concurrently? Crystal currently doesn't support concurrent invocations of the compiler (with the same cache directory). |
It does sound weird, I know. But they are definitely running sequentially, unless bash and/or docker are doing something crazy. Here are the exact commands being run:
One thing I thought of was that Docker might not have finished writing something to disk before the second invocation starts, but the container is not stopping between those two |
Refs:
/cc @asterite @RX14
The text was updated successfully, but these errors were encountered: