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

Add resource limits for waf-enforcer and waf-config-mgr #7119

Open
anderius opened this issue Jan 13, 2025 · 2 comments
Open

Add resource limits for waf-enforcer and waf-config-mgr #7119

anderius opened this issue Jan 13, 2025 · 2 comments
Labels
proposal An issue that proposes a feature request ready for refinement An issue that was triaged and it is ready to be refined

Comments

@anderius
Copy link

Some organizations require resource limits on all resources.

It would be nice if the Helm chart could add values for that on all containers.

My team currently has to patch the chart, and change _helpers.tpl for waf-enforcer and waf-config-mgr.

@anderius anderius added the proposal An issue that proposes a feature request label Jan 13, 2025
Copy link

Hi @anderius thanks for reporting!

Be sure to check out the docs and the Contributing Guidelines while you wait for a human to take a look at this 🙂

Cheers!

@pdabelf5 pdabelf5 added the needs triage An issue that needs to be triaged label Jan 13, 2025
@pdabelf5
Copy link
Collaborator

@anderius thank you for raising this issue, we will investigate adding the functionality proposed.

@pdabelf5 pdabelf5 added ready for refinement An issue that was triaged and it is ready to be refined and removed needs triage An issue that needs to be triaged labels Jan 13, 2025
@lucacome lucacome moved this to Todo ☑ in NGINX Ingress Controller Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proposal An issue that proposes a feature request ready for refinement An issue that was triaged and it is ready to be refined
Projects
Status: Todo ☑
Development

No branches or pull requests

2 participants