Skip to content
This repository was archived by the owner on Mar 11, 2022. It is now read-only.

Added resources limits in the configuration of helm chart#224

Merged
seanmalloy merged 3 commits intoKohlsTechnology:masterfrom
rajneesh-ops:resource_fix
Dec 26, 2019
Merged

Added resources limits in the configuration of helm chart#224
seanmalloy merged 3 commits intoKohlsTechnology:masterfrom
rajneesh-ops:resource_fix

Conversation

@rajneesh-ops
Copy link
Copy Markdown
Contributor

Description

Added support to create Kubernetes deployment with a resource limit.

#205

Type of change

  • Bugfix (non-breaking change which fixes an issue)

Checklist

  • Documentation updated

@rajneesh-ops
Copy link
Copy Markdown
Contributor Author

eunomia

Above is the screenshot of the changes.

Comment thread deploy/helm/eunomia-operator/values.yaml
Copy link
Copy Markdown
Contributor

@Smiley73 Smiley73 left a comment

Choose a reason for hiding this comment

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

Memory requests and limits must be the same.

@rajneesh-ops
Copy link
Copy Markdown
Contributor Author

I have made the suggested changes.

@seanmalloy seanmalloy merged commit 8835364 into KohlsTechnology:master Dec 26, 2019
@seanmalloy seanmalloy added this to the v0.1.0 milestone Dec 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants