Skip to content
This repository has been archived by the owner on Feb 14, 2023. It is now read-only.

Set memory and cpu requests and limit values for all containers #65

Closed
wants to merge 1 commit into from

Conversation

paulcwarren
Copy link
Member

Relint is currently working on a scaling and Quality of Service (QoS) set of stories.

We are targeting 1.0 to be configured out-of-the-box as a "developer" edition aimed at those users who want to kick the tyres. As part of this, we would like to set limits on mem/cpu.

Since a "developer" edition may not be preferred by everyone, we want each component to be configurable to scale both horizontally (replicas) and vertically (mem/cpu). This will also allow users to deliver a Guaranteed QoS when required (although we are recommending that all of our pods and containers use the Burstable QoS) As part of this we would like to ask you to do several things:

  1. consider which of your pods/containers you would like to expose for scaling properties for.
  2. expose said configuration properties.
  3. sets mem and cpu values for all containers in order to provide as much meta-data to k8s as possible so that its scheduler can do as good a job as possible. This PR is an initial attempt at setting these values, although we know you are much more likely to have insight into your components mem/cpu requirements than our guess.

If you have any questions or concerns, please let us know! Thanks!

#174462927

Co-Authored-By: Carson Long lcarson@vmware.com

@cf-gitbot
Copy link
Collaborator

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/174613040

The labels on this github issue will be updated when the story is started.

@XanderStrike
Copy link
Contributor

Closing: Will implement but using a different approach, see https://www.pivotaltracker.com/n/projects/2407973/stories/174613651

routing-ci pushed a commit that referenced this pull request Sep 15, 2020
Remove prometheus

[finishes #171829314]
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants