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
ImageMaximumGCAge documentation recommendations #124441
Comments
/sig-node |
/sig node |
/cc @haircommander @sohankunkerkar /triage accepted |
Beta feedback for kubernetes/enhancements#4210 |
when concieving of the feature I had the idea of something intermittent but not too infrequent: something like 2 weeks. We've yet to make official recommendations as I think we need more testing time to find the frequency that seems good. |
@haircommander how big of a disk usage spike will happen on cleanup? Is 2 weeks coming from the idea that the autoscaler may decide to use the node again? If there are no autoscaler, is there any issues with 1 hour? |
btw, one idea for improvement of this logic is to add some grace period to wait if there are download is hapenning at this moment. So if there is an image pull ongoing when GC want's to clean up those images, we can wait a bit to minimize disk churn |
I think 1 hour is pretty low TBH
which autoscaler?
no matter the grace period we give, we'll hit this issue theoretically. Even if no container is using an image, we could hit situations where an unused image for 1 hour will be created immediately after. My thought with the relatively high value is to increase the liklihood that an image is not just unused, but has been replaced by a newer version of the same image. The former case (unused for a bit, but will be used later) is an unfortunate side effect, and we'd use the new metric to track when the image was gc'd for reason |
related: https://kubernetes.slack.com/archives/C0BP8PW9G/p1713553524168989
The beta feature
ImageMaximumGCAge
that is enabled by default should have recommended ranges for people to configure. Right now out of the box it's currently set to0s
(which means it's disabled). This is a documentation request to provide a general rule of thumb for the community to shoot for with garbage collection cleaning up images./sig-node
The text was updated successfully, but these errors were encountered: