Skip to content
This repository has been archived by the owner on Apr 17, 2019. It is now read-only.

Vertical Pod Autoscaler : updater first version #2532

Closed
wants to merge 2 commits into from

Conversation

GFilipek
Copy link

@GFilipek GFilipek commented Apr 6, 2017

Updater component for Vertical Pod Autoscaler described in kubernetes/community#338

Runs in loop. On one iteration performs:

  • Fetching Vertical Pod Autoscaler configuration
  • Fetching Pods information and resource allocation recommendations for pods.
  • Calculating if pod update is required and how many replicas can be evicted
  • Evicting pods if recommended resources significantly vary from actual resources allocation.

Mocked parts - to be implemented:

  • Recommendation API for fetching data from Vertical Pod Autoscaler Recommender.
  • Vertical Pod Autoscaler lister for fetching Vertical Pod Autoscaler config.

cc: @mwielgus @kgrygiel @KarolKraskiewicz

@k8s-ci-robot
Copy link
Contributor

Thanks for your pull request. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please follow instructions at https://github.com/kubernetes/kubernetes/wiki/CLA-FAQ to sign the CLA.

Once you've signed, please reply here (e.g. "I signed it!") and we'll verify. Thanks.


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.

@k8s-ci-robot k8s-ci-robot added the cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. label Apr 6, 2017
@k8s-github-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

The following people have approved this PR: GFilipek

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

We suggest the following people:
cc @eparis
You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@k8s-reviewable
Copy link

This change is Reviewable

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels Apr 6, 2017
@mwielgus mwielgus assigned mwielgus and kgrygiel and unassigned bprashanth Apr 6, 2017
@bgrant0607
Copy link
Contributor

@GFilipek

Please don't add something this big to contrib. See #762.

We should create a repo in kubernetes-incubator instead.

The process is described here:

https://github.com/kubernetes/community/blob/master/incubator.md

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area/autoscaling cncf-cla: yes Indicates the PR's author has signed the CNCF CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants