-
Notifications
You must be signed in to change notification settings - Fork 54
Add Evict operator #54
base: master
Are you sure you want to change the base?
Conversation
493e6b4
to
ce4fbc1
Compare
I am sorry that this PR is WIP for a while already. I am busy with other work and have not found the time to finish the tests and remaining todos, yet. I will do but feel free to close the PR for now to keep the list of PRs clean. ⌛️ |
No worries, @alpe, take your time. All of your contributions are appreciated. I'll try to get the policy end of this sorted at some point, but I have been busy with a few things as well. |
Another usecase came up today for this: A maxAge annotation for Pods. Pods would get evicted after passing the duration specified. This is handy for triggering re-provisioning of certificates, and also just enforce good hygiene and ensure that disruptions are tolerable. |
This reverts commit 7bc0b34.
f68476e
to
5e08f88
Compare
This PR was WIP for quite some time. I think it would be better if we can get the code into master and work on improvements together. There are some open questions and extensions that can be addresses by separate and smaller PRs later. For testing the evictor, please see 7bc0b34 for a configuration example. I have reverted the commit as it requires some testing. |
Sounds good to me. I'll take a look at doing that. Thanks for the update @alpe. |
👋 The k-rail project has been deprecated and is no longer under active development. We recommend taking a look at OPA Gatekeeper to see if it might meet your needs going forward. Thanks for your contribution(s) to the project! |
Resolves #18
Implementation of the evict operator - based on #47
Status: works on my machine.... 😆
Follow ups and improvements
Open questions