Skip to content
This repository has been archived by the owner on Aug 25, 2021. It is now read-only.

Increase client init resources 25Mi => 50Mi #832

Merged
merged 1 commit into from
Feb 18, 2021
Merged

Conversation

lkysow
Copy link
Member

@lkysow lkysow commented Feb 18, 2021

Increase the memory for the client daemonset's tls-init
container. We saw an out of memory error when running on OpenShift.

https://circleci.com/gh/hashicorp/consul-helm/8609 saw an OOMKilled error

Checklist:

  • CHANGELOG entry added (HashiCorp engineers only, community PRs should not add a changelog entry)

@@ -314,7 +314,7 @@ server:
# enable `server.exposeGossipAndRPCPorts` and `client.exposeGossipPorts`,
# that will configure the LAN gossip ports on the servers and clients to be
# hostPorts, so if you are running clients and servers on the same node the
# ports will conflict if they are both 8301. When you enable
# ports will conflict if they are both 8301. When you enable
Copy link
Member Author

Choose a reason for hiding this comment

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

@lkysow lkysow requested review from a team, kschoche and ishustava and removed request for a team February 18, 2021 18:34
Copy link
Contributor

@kschoche kschoche left a comment

Choose a reason for hiding this comment

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

cool!

Increase the memory for the client daemonset's tls-init
container. We saw an out of memory error when running on OpenShift.
@lkysow lkysow merged commit eb00d47 into master Feb 18, 2021
@lkysow lkysow deleted the client-resources branch February 18, 2021 18:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants