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
kubelet/rkt: add container/image gc for rkt. #14686
Conversation
Can one of the admins verify that this patch is reasonable to test? (reply "ok to test", or if you trust the user, reply "add to whitelist") If this message is too spammy, please complain to ixdy. |
ok to test |
Labelling this PR as size/M |
Unit, integration and GCE e2e test build/test passed for commit 07ed08ac3e9f11107c342308936ba08601b62c59. |
// Duration to wait before expiring prepared pods. | ||
defaultExpirePrepared = "1m" | ||
defaultExpirePrepared = "0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why changing the periods to 0?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yujuhong Just for testing, I changed so I can see the result soonner. Changed back.
btw, we should derive this period from gc policy's MinAge, but that requires the gc definition stuff to be moved to kubelet/container or other packages, not in the kubelet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moving the gc policy out kubelet sounds good to me.
Unit, integration and GCE e2e test build/test passed for commit 6a328f75eaaa3f28c5659414610afac12e983216. |
#14706 got merged first...need a rebase here. |
Rebased |
Unit, integration and GCE e2e test build/test passed for commit 71f98eea7a321631c53e07b5f0055ddbff6dcdb1. |
Kicked off shippable again. Hope this time it'd pass |
Oops...need to rebase again |
rebased. |
Unit, integration and GCE e2e test build/test passed for commit b42d231. |
LGTM was before last commit, removing LGTM |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
GCE e2e test build/test passed for commit b42d231. |
Automatic merge from submit-queue |
Auto commit by PR queue bot
Not sure why the rkt container gc is disabled.
This PR reenables it. It also adds image gc.
/cc @dchen1107 @yujuhong