Skip to content

ECK - Increase default Kibana memory limit to 2Gi#6770

Merged
eedugon merged 4 commits into
mainfrom
eedugon-patch-5
Jun 1, 2026
Merged

ECK - Increase default Kibana memory limit to 2Gi#6770
eedugon merged 4 commits into
mainfrom
eedugon-patch-5

Conversation

@eedugon
Copy link
Copy Markdown
Contributor

@eedugon eedugon commented May 29, 2026

Updated the default memory limits for {{kib}} from 1Gi to 2Gi.

Closes #6581

Summary

Generative AI disclosure

  1. Did you use a generative AI (GenAI) tool to assist in creating this contribution?
  • Yes
  • No

Updated the memory limits for {{kib}} from 1Gi to 2Gi to ensure sufficient resources for core functionality and prevent service interruptions.

Closes #6581
@eedugon eedugon requested a review from a team as a code owner May 29, 2026 11:13
@eedugon eedugon requested a review from TimBosman May 29, 2026 11:13
@github-actions
Copy link
Copy Markdown
Contributor

Elastic Docs AI PR menu

Check the box to run an AI review for this pull request.

  • Review docs changes (docs-review). Status: not started.

Powered by GitHub Agentic Workflows and docs-actions. For more information, reach out to the docs team.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 29, 2026

🔍 Preview links for changed docs

@github-actions
Copy link
Copy Markdown
Contributor

✅ Elastic Docs Style Checker (Vale)

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide. To use Vale locally or report issues, refer to Elastic style guide for Vale.

@eedugon
Copy link
Copy Markdown
Contributor Author

eedugon commented May 29, 2026

@shainaraskas , @yetanothertw : how would you address this change?

Kibana default memory limit has been increased to 2Gi from 1Gi in ECK 3.4.0, but in earlier versions the default memory limit, if not set explicitly, will still be 1Gi.

What's the best way to address this and future changes on this regard? In other docs we have created tabs for version ranges. Would you apply something similar here?

Comment thread deploy-manage/deploy/cloud-on-k8s/manage-compute-resources.md Outdated
@yetanothertw
Copy link
Copy Markdown
Member

In other docs we have created tabs for version ranges. Would you apply something similar here?

Yes, I think tabs for version ranges might work the cleanest. But would you have to replicate the entire table twice?
Could you play around and see what it would look like if you add another line and use different applies_to tags to signal the ranges that way as well?

You'll be able to tell which way is clearer once you've tried it. That's what I'd do anyway.


::::{note}
The default 1 Gi {{kib}} limit is sufficient for core functionality. For Platinum and Enterprise users, we recommend at least 2 Gi of memory for each {{kib}} instance. With less than 2 Gi, you can face service interruptions when using features such as Security detection rules, reporting, workflows, and Agent Builder.
The default 2 Gi {{kib}} limit is sufficient for core functionality. For Platinum and Enterprise users, we recommend at least 2 Gi of memory for each {{kib}} instance. With less than 2 Gi, you can face service interruptions when using features such as Security detection rules, reporting, workflows, and Agent Builder.
Copy link
Copy Markdown
Contributor

@TimBosman TimBosman May 29, 2026

Choose a reason for hiding this comment

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

This sounds a bit weird for me.

The default is 2 Gi, but for specific use cases we recommend 2 Gi.

We should either increase the recommendation, or remove this line from the documentation

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, I agree. Even with the at least it doesn't sound nice.
Thanks for the comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@eedugon I added this during the window where the default compute resources were lower.

I think what this actually needs is version-awareness - I assume in older ECK versions, regardless of stack version, the default would be lower? so the default in the table would need two bullets, one for the two states, and then this note would need to be contextualized to older versions.

@shainaraskas
Copy link
Copy Markdown
Member

shainaraskas commented May 29, 2026

just saw your tag. as mentioned, given that it's just one value that changed, I'd do an inline bullet, like this

heading heading heading
hi • {applies_to}eck: ga 3.4 2Gi
• {applies_to}eck: ga 3.0-3.3 1Gi
hello

+ then edit the note as stated with an :applies_to attribute

@eedugon
Copy link
Copy Markdown
Contributor Author

eedugon commented Jun 1, 2026

just saw your tag. as mentioned, given that it's just one value that changed, I'd do an inline bullet, like this

heading heading heading
hi • {applies_to}eck: ga 3.4 2Gi
• {applies_to}eck: ga 3.0-3.3 1Gi hello

  • then edit the note as stated with an :applies_to attribute

Awesome, thanks! I've done that but without bullets. Here you have an example showing both options, I think the bullets reduce the consistency with the rest of the table:

image

@eedugon eedugon requested a review from shainaraskas June 1, 2026 09:14
Copy link
Copy Markdown
Member

@shainaraskas shainaraskas left a comment

Choose a reason for hiding this comment

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

lgtm - ty

@eedugon eedugon enabled auto-merge (squash) June 1, 2026 13:53
@eedugon eedugon merged commit cdff0c2 into main Jun 1, 2026
7 of 8 checks passed
@eedugon eedugon deleted the eedugon-patch-5 branch June 1, 2026 13:55
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

Successfully merging this pull request may close these issues.

[Internal]: Update Kibana default Memory limit in ECK documentation

4 participants