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

[TRG 5.04] Define resource limit/request for Charts #720

Closed
SebastianBezold opened this issue Aug 21, 2023 · 1 comment · Fixed by #794
Closed

[TRG 5.04] Define resource limit/request for Charts #720

SebastianBezold opened this issue Aug 21, 2023 · 1 comment · Fixed by #794
Assignees
Labels
enhancement New feature or request

Comments

@SebastianBezold
Copy link
Contributor

Is your feature request related to a problem? Please describe.
In our release guidelines, that are defined for the whole Eclipse Tractus-X project, we specify, that each Chart should provide
default resource -request and -limit values.
In best case the values are gathered by monitoring the EDC under different load circumstances. This would give users a good initial understanding or resource consumption

Describe the solution you'd like
There are already request and limit values specified as comments in the values.yaml. To follow the release guideline, they could simply be used as actual defaults.

Describe alternatives you've considered
If there are valid reasons, why resource requests and limits cannot be set, we should change the existing TRG 5.04, so the guidelines are followed and we do not need to track exceptions.

Additional context
related to, but not blocking #714

Copy link
Contributor

This issue is stale because it has been open for 4 weeks with no activity.

@github-actions github-actions bot added the stale label Nov 21, 2023
@github-project-automation github-project-automation bot moved this from Open to Done in EDC Board Nov 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants