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.

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

Resource limits are difficult to override downstream if the limits are removed in a downstream config. #531

Closed
nvo-aurora opened this issue Sep 20, 2023 · 7 comments
Labels
kind/feature New feature or request

Comments

@nvo-aurora
Copy link

Motivation

Default resource limits being set upstream in this chart makes it difficult to override downstream.
We have removed CPU limits in our config, however the below setting in values.yaml

  limits:
    cpu: 1000m

cannot be overridden if our config has no CPU limits set, and so this setting gets retained in our final deployment.

Feature

Would it be possible to remove these default limits or could you give us other options/alternatives to work around these default limits?

Alternatives

We can possibly set a limitrange in our namespace and just not set a maximum so that it could override the default limits, but I'm not sure how feasible that is in our current setup.

Additional context

This relates to falcosecurity/falco#1403 specifically with the debug option to remove CPU resource limits as a potential remediation.

@nvo-aurora nvo-aurora added the kind/feature New feature or request label Sep 20, 2023
@nvo-aurora
Copy link
Author

Would a config with the below override the default setting in values.yaml?

limits:
  cpu: ''

And remove the cpu limit?

@alacuku
Copy link
Member

alacuku commented Sep 22, 2023

Default resource limits being set upstream in this chart makes it difficult to override downstream. We have removed CPU limits in our config, however the below setting in values.yaml

  limits:
    cpu: 1000m

cannot be overridden if our config has no CPU limits set, and so this setting gets retained in our final deployment.

Hi @nvo-aurora, you need to set a new value in your configuration file (values.yaml) in order to overwrite the default values.
In this case, you will need to set the whole resources:

resources:
  limits:
    cpu: 

This will overwrite the resources for you deployment.

@nvo-aurora
Copy link
Author

A blank value like the below would not work, correct? We need a number value?
cpu: ''

@alacuku
Copy link
Member

alacuku commented Sep 25, 2023

Hi @nvo-aurora, a blank value like this would work:

resources:
  limits:
    cpu: 

@nvo-aurora
Copy link
Author

Thank you

@alacuku
Copy link
Member

alacuku commented Oct 25, 2023

/close

Feel free to comment if you have issues.

@poiana
Copy link
Contributor

poiana commented Oct 25, 2023

@alacuku: Closing this issue.

In response to this:

/close

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.

@poiana poiana closed this as completed Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants