-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Allow for resource limits and requests for pre-deployment jobs #23077
Conversation
Adding the ability to allow for requests and limits to be added to the predeploy_job like already done for deployments.
Add the ability for the proxy predeploy_job to have limits and requests like the deployment. Using same values from Values file.
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.
The template change looks good :)
Can you add tests covering the feature?
You can find test guidelines here, and might want to copy the main deployment resource tests.
Adding tests for resource limits on predeploy job for auth and proxy.
@hugoShaka I have added the tests. It was my first time building those, please let me know if I missed something. |
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 once the suggestions are addressed
Co-authored-by: Hugo Shaka <hugo.hervieux@goteleport.com>
Co-authored-by: Hugo Shaka <hugo.hervieux@goteleport.com>
Co-authored-by: Hugo Shaka <hugo.hervieux@goteleport.com>
Co-authored-by: Hugo Shaka <hugo.hervieux@goteleport.com>
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.
@yjperez Do you want to backport this? If so, can you add respective backport/branch/xxx
labels to the PR.
@r0mant no, this is only applicable starting Teleport V12. |
@yjperez Then you need |
Adding the ability to allow for requests and limits to be added to the predeploy_job like already done for deployments.