-
Notifications
You must be signed in to change notification settings - Fork 137
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.
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
Add resourcequotas and limitrange components. #439
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I very much dislike the names - what we're playing on here? Like.. gold
, silver
, bronze
lol. 😄
What about renaming those to idk, small
, medium
, large
, and so on? I like that better.
cluster-scope/components/resourcequotas/bronze/resourcequotas.yaml
Outdated
Show resolved
Hide resolved
2b2208d
to
8eb248c
Compare
Thoughts on the actual resource values for each tier? |
@HumairAK does it make sense to look at the current resource utilization of the cluster to help determine reasonable quota values? |
8eb248c
to
57fac27
Compare
I've updated the values based on discussions in the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: tumido The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Related: #438
The
resourcequotas
aren't included in any of the namespaces yet, this is one way we can go about this. The idea is to include these like we do every other component within the cluster-scope/base/namespaces.We don't have to use tiers if there are good arguments against it, this is a suggestion.
I just willy nilly picked amounts for each tier and
limitrange
. We can discuss what amounts are reasonable.WDYT?