Skip to content
This repository has been archived by the owner on Jun 6, 2024. It is now read-only.

[Pod Eviction] Disable kubernetes's hard Eviction #2124

Merged
merged 3 commits into from
Feb 11, 2019
Merged

Conversation

ydye
Copy link
Contributor

@ydye ydye commented Feb 2, 2019

1: Disable pod eviction behavior
2: Set the image gc threshold to 100%. And when image gc is triggered, kubelet will continue do image gc until the file system's space is below 95%.

@coveralls
Copy link

coveralls commented Feb 2, 2019

Coverage Status

Coverage remained the same at 52.904% when pulling bff368d on yuye/pod-evtic into f0c3c19 on master.

@fanyangCS
Copy link
Contributor

have you tested it?

@ydye
Copy link
Contributor Author

ydye commented Feb 6, 2019

have you tested it?

Haven't test it yet.

@fanyangCS
Copy link
Contributor

please have it test

@ydye ydye requested a review from wangdian February 11, 2019 08:40
Copy link
Member

@wangdian wangdian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

According to test result, this setting will run image gc when disk usage at 100%.
Confirmed with Yundong, this is for disk space and the memory restriction was not tested and removed.

@ydye ydye merged commit 6b2842f into master Feb 11, 2019
@ydye ydye deleted the yuye/pod-evtic branch February 12, 2019 06:56
wangdian added a commit that referenced this pull request Feb 13, 2019
* Cleaner logic

* Fix bug

* 1. Remove worker
2. Add interval as var
3. Kill docker, send signal

* Fix '\t'

* Bug fix

* Import re in common

* Update openjdk build version

* [JDK] Remove JDK version hardcode and add print in pai_build for debuging (#2123)

* Change kill signal to 10 (SIGUSR1)

* [Pod Eviction] Disable kubernetes's pod eviction (#2124)

* According to kubernetes/kubernetes#71661

* add imagegc threshold

* Add k8s_POD to white list
Add markdown document for cleaner configuration

* Minor bug fix

* Cleaner logic

* Fix bug

* 1. Remove worker
2. Add interval as var
3. Kill docker, send signal

* Fix '\t'

* Bug fix

* Import re in common

* Update openjdk build version

* Change kill signal to 10 (SIGUSR1)

* Add k8s_POD to white list
Add markdown document for cleaner configuration

* Cleaner logic

* Fix bug

* 1. Remove worker
2. Add interval as var
3. Kill docker, send signal

* Fix '\t'

* Bug fix

* Import re in common

* Update openjdk build version

* Change kill signal to 10 (SIGUSR1)

* Add k8s_POD to white list
Add markdown document for cleaner configuration

* Add comment on cleaner kill handler in rest-server
@scarlett2018 scarlett2018 changed the title [Pod Eviction] Disable kubernetes's hard enicvtion [Pod Eviction] Disable kubernetes's hard Eviction Feb 25, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants