Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

align ako statefulset resource limit #1477

Merged
merged 1 commit into from
Jan 11, 2022

Conversation

XudongLiuHarold
Copy link
Member

@XudongLiuHarold XudongLiuHarold commented Jan 11, 2022

Signed-off-by: Xudong Liu xudongl@vmware.com

What this PR does / why we need it

AKO increases resource limit upper boundary in the new release version, we should align the default limit with it.

Which issue(s) this PR fixes

Fixes #

Fix ako pod OOMKilled crash when it works in large scale cluster.

Describe testing done for PR

Release note

NONE

PR Checklist

  • Squash the commits into one or a small number of logical commits
  • Use good commit messages
  • Ensure PR contains terms all contributors can understand and links all contributors can access

Additional information

Special notes for your reviewer

Signed-off-by: Xudong Liu <xudongl@vmware.com>
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/1477/20220111201839/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

Copy link
Contributor

@HanFa HanFa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM@

@XudongLiuHarold XudongLiuHarold added the ok-to-merge PRs should be labelled with this before merging label Jan 11, 2022
@XudongLiuHarold XudongLiuHarold merged commit 96d3537 into main Jan 11, 2022
@XudongLiuHarold XudongLiuHarold deleted the topic/xudongl/align-ako-resource-limit branch January 11, 2022 23:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants