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

High CPU usage #733

Closed
janisii opened this issue Jul 24, 2023 · 5 comments
Closed

High CPU usage #733

janisii opened this issue Jul 24, 2023 · 5 comments

Comments

@janisii
Copy link

janisii commented Jul 24, 2023

Hello,

I have no problem running 0.16.1 version on my k8s cluster (deployed with helm - https://charts.keel.sh). At the moment I'm running helm with chart version keel-0.9.8 (app 0.16.1) no problem.

I wanted to upgrade keel to latest release keel-1.0.2 (app 0.18.1) and it does not start with default resources defined in default values.yaml file.

# Resources
resources:
  limits:
    cpu: 100m
    memory: 128Mi
  requests:
    cpu: 50m
    memory: 64Mi

I've set cpu limits to 1000m and it was killed later with OOM, because it hit more than 1core cpu usage. What could be the problem?

@janisii
Copy link
Author

janisii commented Sep 20, 2023

resources:
  limits:
    memory: 256Mi

Looks like pod now requires more memory. 256MB solved the issue in our case.

CPU was not the issue.

@janisii janisii closed this as completed Sep 20, 2023
@lrstanley
Copy link

There is no way this should require that much memory. I'd still consider it a bug if it actually required that much.

@janisii
Copy link
Author

janisii commented Sep 22, 2023

There is no way this should require that much memory. I'd still consider it a bug if it actually required that much.

Yes, 256Mi was enough in our case. I will update my reply.

@janisii
Copy link
Author

janisii commented Oct 5, 2023

I have found the issue with the keel (>= 0.18.0) versions (CPU and memory leak).

The root cause of the problem is:

helmProvider:
  version: "v3"

@janisii janisii reopened this Oct 5, 2023
@sreenivas-ps
Copy link
Contributor

i'm using keel-1.0.3 app: 0.19.1 and hit with a out of memory error. Prior to the upgrade, we had

memory = 1000Mi

Increasing the memory to the following also didn't fix it:

  memory:  1200Mi

any suggestions?

@janisii janisii closed this as completed Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants