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

Support pex3 cache prune --older-than .... #2586

Merged
merged 2 commits into from
Nov 4, 2024

Conversation

jsirois
Copy link
Member

@jsirois jsirois commented Oct 30, 2024

Work towards #2528.

@jsirois
Copy link
Member Author

jsirois commented Nov 4, 2024

Pants folks - this is FYI. It looks like Pants has not adopted Pex features in quite some time, but pex3 cache prune may be compelling. Unlike pex3 cache purge, it requires no knowledge of Pex internals and just does the right thing. The defaults (prune anything not used in the last 2 weeks) probably make sense to use as-is. If Pants had a cron job facility, this should probably be one of the canned cron jobs it ships with for example.

And clearly, the scie-jump should support the same technology, but that's for another time.

N.B.: As with pex3 cache purge, the prune will block for in-flight PEX runs / Pex CLI invocations to complete. This blocking is not super user-friendly out of the box, but is more so when psutil is installed. The released Pex PEX scies all have psutil included.

@jsirois jsirois merged commit 06b8850 into pex-tool:main Nov 4, 2024
23 checks passed
@jsirois jsirois deleted the cache/prune branch November 4, 2024 02:25
@huonw
Copy link
Collaborator

huonw commented Nov 4, 2024

Thanks for the heads-up.

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 this pull request may close these issues.

2 participants