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

hack: don't limit memory in Metricbeat daemonset #2534

Merged
merged 1 commit into from
Oct 30, 2023

Conversation

msanft
Copy link
Contributor

@msanft msanft commented Oct 27, 2023

Context

We recently ran into OOM kills on the metricbeat deployment (since metricbeat seems to be quite memory-hungry), causing issues when updating a cluster with metricbeat deployed.

Proposed change(s)

  • Remove the 200MiB default limit on the daemonset.

Additional info

Checklist

  • Add labels (e.g., for changelog category)
  • Is PR title adequate for changelog?
  • Link to Milestone

@netlify
Copy link

netlify bot commented Oct 27, 2023

Deploy Preview for constellation-docs canceled.

Name Link
🔨 Latest commit 32fb655
🔍 Latest deploy log https://app.netlify.com/sites/constellation-docs/deploys/653bd5a0ffe47f000868aa1d

@msanft msanft changed the title metricbeat: don't limit memory hack: don't limit memory in Metricbeat daemonset Oct 27, 2023
@msanft msanft added bug fix Fixing a bug no changelog Change won't be listed in release changelog and removed bug fix Fixing a bug labels Oct 27, 2023
@msanft msanft added this to the v2.13.0 milestone Oct 27, 2023
@msanft msanft requested a review from daniel-weisse October 27, 2023 15:22
@msanft msanft merged commit 78c5d09 into main Oct 30, 2023
4 checks passed
@msanft msanft deleted the fix/hack/metricbeat-oom branch October 30, 2023 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog Change won't be listed in release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants