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

feat: Reduced resource requests for several components #1532

Merged
merged 3 commits into from
Oct 21, 2024

Conversation

a-thaler
Copy link
Collaborator

@a-thaler a-thaler commented Oct 16, 2024

Description

  • After checking the idle consumptions, I reduced the CPU request for
    • fluentbit exporter from 100m to 10m
    • self-monitor from 100m to 10m
  • After checking the idle consumptions, I reduced the memory request for
    • fluentbit exporter from 50m to 20m
    • manager from 100m to 50m

Changes refer to particular issues, PRs or documents:

Traceability

  • The PR is linked to a GitHub issue.
  • The follow-up issues (if any) are linked in the Related Issues section.
  • If the change is user-facing, the documentation has been adjusted.
  • If a CRD is changed, the corresponding Busola ConfigMap has been adjusted.
  • The feature is unit-tested.
  • The feature is e2e-tested.

@a-thaler a-thaler added kind/feature Categorizes issue or PR as related to a new feature. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. area/manager Manager or module changes labels Oct 16, 2024
@a-thaler a-thaler requested a review from a team as a code owner October 16, 2024 12:57
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Oct 16, 2024
@skhalash
Copy link
Collaborator

Let us merge it after #1527 is merged

@a-thaler a-thaler added this to the 1.26.0 milestone Oct 16, 2024
@a-thaler a-thaler enabled auto-merge (squash) October 17, 2024 17:01
@a-thaler a-thaler removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 21, 2024
@hisarbalik hisarbalik self-assigned this Oct 21, 2024
@a-thaler a-thaler merged commit 977a8d7 into kyma-project:main Oct 21, 2024
39 of 40 checks passed
@kyma-bot kyma-bot added the lgtm Looks good to me! label Oct 21, 2024
@a-thaler a-thaler deleted the reduced-request branch October 29, 2024 07:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/manager Manager or module changes cla: yes Indicates the PR's author has signed the CLA. kind/feature Categorizes issue or PR as related to a new feature. lgtm Looks good to me! 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.

4 participants