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

increase registries memory limit to 256Mi in helm charts #14135

Merged
merged 1 commit into from
Aug 6, 2019
Merged

increase registries memory limit to 256Mi in helm charts #14135

merged 1 commit into from
Aug 6, 2019

Conversation

sparkoo
Copy link
Member

@sparkoo sparkoo commented Aug 6, 2019

Signed-off-by: Michal Vala mvala@redhat.com

What does this PR do?

Increase memory limit in helm charts for registries to 256Mi. This is to keep it consistent with Openshift operator (#14093).

What issues does this PR fix or reference?

#14100

Signed-off-by: Michal Vala <mvala@redhat.com>
@che-bot che-bot added status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. kind/task Internal things, technical debt, and to-do tasks to be performed. labels Aug 6, 2019
Copy link
Member

@sleshchenko sleshchenko left a comment

Choose a reason for hiding this comment

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

LGTM

@che-bot
Copy link
Contributor

che-bot commented Aug 6, 2019

E2E Happy path tests of Eclipse Che Single User on K8S (minikube v1.1.1) has been successful:

@eclipse-che eclipse-che deleted a comment from musienko-maxim Aug 6, 2019
@skabashnyuk skabashnyuk merged commit 1af529c into eclipse-che:master Aug 6, 2019
@che-bot che-bot removed the status/code-review This issue has a pull request posted for it and is awaiting code review completion by the community. label Aug 6, 2019
@sparkoo sparkoo deleted the helmRegistriesMemoryLimits branch August 6, 2019 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/task Internal things, technical debt, and to-do tasks to be performed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants