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

[release-2.6] feat(ai-navigator): set proper resource limits and requests (#1585) #1597

Merged
merged 1 commit into from
Sep 14, 2023

Conversation

github-actions[bot]
Copy link
Contributor

Backport

This will backport the following commits from main to release-2.6:

Questions ?

Please refer to the Backport tool documentation

@mstruebing mstruebing added ready-for-review ok-to-test Signals mergebot that CI checks are ready to be kicked off labels Sep 14, 2023
* fix: remove empty config map

* feat(ai-navigator): set proper resource limits and requests

I've ran 25 bot users which are constantly asking questions against the
service on the daily and it was inside the limits with a good margin.
It never went above 0.008 CPU usage and memory wasn't really much
affected, it stayed <30MB observed in our Grafana Dashboard.

* Closes D2IQ-99125

* chore: set limits == requests

(cherry picked from commit d97ee74)
@github-actions github-actions bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 14, 2023
@mstruebing mstruebing merged commit 5ff8105 into release-2.6 Sep 14, 2023
7 of 8 checks passed
@mstruebing mstruebing deleted the backport/release-2.6/pr-1585 branch September 14, 2023 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Signals mergebot that CI checks are ready to be kicked off ready-for-review size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant