Skip to content
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

Implement read only mode #423

Closed
ChrisBr opened this issue Nov 3, 2022 · 1 comment · Fixed by #428
Closed

Implement read only mode #423

ChrisBr opened this issue Nov 3, 2022 · 1 comment · Fixed by #428

Comments

@ChrisBr
Copy link

ChrisBr commented Nov 3, 2022

Implement a read only mode which does not update the bootsnap cache anymore e.g by supplying an env variable BOOTSNAP_READ_ONLY.

Background

Our Bootsnap cache in CI is quite significant and we run several CI workers on the same machine via docker. As bootsnap keeps writing to the cache, we need to keep a dedicated copy of the cache for each of our CI worker instances. If bootsnap would support a read only mode, we could only keep one copy of the cache around and mount it into the containers in read only mode. This would reduce our network traffic, I/o and disk size significantly.

@casperisfine
Copy link
Contributor

Ack. I think that would make sense, even beyond the use case you mention.

It may take me a tiny while before I can look into this (couple weeks perhaps).

casperisfine pushed a commit that referenced this issue Nov 24, 2022
Fix: #423

When you know that the cache won't be used again, it avoid
some useless work and IOs.

Typically this might be the case for dockerized applications. You
generate a cache when building the image, but then when you boot the
application any cache update won't be persisted, so thre is no point.
casperisfine pushed a commit that referenced this issue Nov 24, 2022
Fix: #423

When you know that the cache won't be used again, it avoid
some useless work and IOs.

Typically this might be the case for dockerized applications. You
generate a cache when building the image, but then when you boot the
application any cache update won't be persisted, so thre is no point.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants