Skip to content

Add Eviction when EmptyDir exceeds user's requested limit #43330

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.

Sign up for GitHub

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

Jump to bottom
Closed
jingxu97 wants to merge 1 commit into from
Closed

Add Eviction when EmptyDir exceeds user's requested limit #43330

jingxu97 wants to merge 1 commit into from

Conversation

@jingxu97
Copy link
Contributor

@jingxu97 jingxu97 commented Mar 17, 2017

User can specify a size limit for emptyDir volume. Eviction manager will
evict the pod if the usage of EmptyDIr volume exceeds the limit.

@jingxu97
Add Eviction when EmptyDir exceeds user's requested limit
7853d1f 
User can specify a size limit for emptyDir volume. Eviction manager will
evict the pod if the usage of EmptyDIr volume exceeds the limit.
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Mar 17, 2017
@k8s-github-robot
Copy link

k8s-github-robot commented Mar 17, 2017

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jingxu97
We suggest the following additional approver: @lavalamp

Details Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-github-robot k8s-github-robot added kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API size/L Denotes a PR that changes 100-499 lines, ignoring generated files. release-note-label-needed labels Mar 17, 2017
@k8s-reviewable
Copy link

k8s-reviewable commented Mar 17, 2017

This change is Reviewable

@k8s-ci-robot
Copy link
Contributor

k8s-ci-robot commented Mar 17, 2017 •
edited
Loading

@jingxu97: The following test(s) failed:

Test name Commit Details Rerun command
Jenkins verification 7853d1f link @k8s-bot verify test this
Jenkins Bazel Build 7853d1f link @k8s-bot bazel test this
Jenkins unit/integration 7853d1f link @k8s-bot unit test this

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@jbeda
Copy link
Contributor

jbeda commented Apr 10, 2017

This is an API change and should probably follow the (nascent) API review process.

@pwittrock -- you are writing up the process in kubernetes/community#419. Care to comment on what should happen for a change like this?

@pwittrock
Copy link
Member

pwittrock commented Apr 11, 2017

@jbeda WDYT of starting at step 3 and creating a proposal from the template

@k8s-github-robot
Copy link

k8s-github-robot commented May 18, 2017

@jingxu97 PR needs rebase

@k8s-github-robot k8s-github-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 18, 2017
@ddysher
Copy link
Contributor

ddysher commented Jun 9, 2017

@jingxu97 This one can be closed?

@jingxu97
Copy link
Contributor Author

jingxu97 commented Jun 9, 2017

Have another PR #45686 for this feature. Close it

@jingxu97 jingxu97 closed this Jun 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@jbeda jbeda

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@jingxu97 @k8s-github-robot @k8s-reviewable @k8s-ci-robot @jbeda @pwittrock @ddysher