-
Notifications
You must be signed in to change notification settings - Fork 184
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
Fix a cache corruption issue during revalidation #476
Conversation
7a854ce
to
dfe1e08
Compare
Fix: #474 Fix: #471 As noticed by Stan Hu, upon revalidation, we'd corrupt the `data_size` field, as it wasn't initialized at all on the cache key we computed from the source file. The only thing we actually want to update is the `mtime`. Co-Authored-By: Stan Hu <[email protected]>
dfe1e08
to
ad189d1
Compare
heh, I managed to hit this. I did
I'm using the new Dockerfile that comes with Rails7.1 instead of my own Dockerfile(s), so at first I was worried something was broken with my build entirely. found this bug, bumped to |
I had this error pop up yesterday and it's still happening to me even on 1.18.3, are there any workarounds? |
@dlsteuer have you cleared your cache? To be fair I should have bumped the cache version, my bad. |
Next release will. 811b34a |
Is this just a matter of deleting the cache folder? |
Yes. |
Fix: #474
Fix: #471
As noticed by Stan Hu, upon revalidation, we'd corrupt the
data_size
field, as it wasn't initialized at all on the cache key we computed from the source file.The only thing we actually want to update is the
mtime
.Co-Authored-By: @stanhu (we didn't pair, but spotting the root cause was 99% of the work, so...)
For now I leave the revalidation as optional, but will likely flip it back on in another release next week, once it has run for a while on our CI without issues.