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

Clean up old cache files #1961

Merged
merged 5 commits into from
Nov 28, 2023
Merged

Clean up old cache files #1961

merged 5 commits into from
Nov 28, 2023

Conversation

amsanghi
Copy link
Contributor

@amsanghi amsanghi commented Nov 10, 2023

Uptill now we were using tmp files to store machine hash caches, but this create a problem that if the node terminates for some reason all the cache that we have build till now, will be deleted, so to avoid this issue we will be using persistent files space so that the cache is present even after restart.

This creates another issue that the cache will keep on growing forever, so to avoid this issue, we are adding a cleanup mechanism which will delete any files under the cache directory which has not been modified for the last 14 days (equal to the challenge duration).

@cla-bot cla-bot bot added the s Automatically added by the CLA bot if the creator of a PR is registered as having signed the CLA. label Nov 10, 2023
@amsanghi amsanghi requested a review from rauljordan November 10, 2023 09:42
@amsanghi amsanghi marked this pull request as ready for review November 10, 2023 09:42
@amsanghi amsanghi changed the base branch from bold to fix_state_provider November 15, 2023 15:04
Base automatically changed from fix_state_provider to bold November 28, 2023 15:28
@amsanghi amsanghi merged commit b16afd7 into bold Nov 28, 2023
5 of 6 checks passed
@amsanghi amsanghi deleted the clean_cache_files branch November 28, 2023 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s Automatically added by the CLA bot if the creator of a PR is registered as having signed the CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant