Skip to content
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

pod antiaffinity strategy evicts all pods. #48

Closed
ravisantoshgudimetla opened this issue Nov 28, 2017 · 1 comment
Closed

pod antiaffinity strategy evicts all pods. #48

ravisantoshgudimetla opened this issue Nov 28, 2017 · 1 comment

Comments

@ravisantoshgudimetla
Copy link
Contributor

Pod anti affinity strategy doesn't have a check on type of pod to be evicted. It can evict critical, mirror pods. As this is some functionality that needs to be respected by all strategies in descheduler, I am planning to move this to pods.go to avoid code duplication so that people implementing strategies won't have to think about them.
@ravisantoshgudimetla
Copy link
Contributor Author

cc @aveshagarwal

Merged
damemi pushed a commit to damemi/descheduler that referenced this issue Dec 10, 2020
@openshift-merge-robot
Merge pull request kubernetes-sigs#48 from damemi/bump-bz1904501
968b4b4 
Bug 1904501: sync upstream
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ravisantoshgudimetla