Skip to content

ECE: Disabling CPU hard limit and podman limitation #1511

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

ECE: Disabling CPU hard limit and podman limitation #1511

wants to merge 4 commits into from

Conversation

eedugon
Copy link
Contributor

@eedugon eedugon commented May 28, 2025 •
edited
Loading

This PR introduces the CPU hard limit configuration (enable / disable) within the Resource overrides page, and it describes that it doesn't have any effect on Podman hosts.

I have also refined the introduction of the page.

Preview:

The reason of adding this is:

  • This is available in ECE UI for all users and we didn't have it documented anywhere (thanks @frconil for pointing this out). So there's no point of hiding this button of the UI.
  • We have some public KB articles that also mention this setting (CPU hard limit toggling).
  • We now want to introduce a banner saying that this doesn't work in Podman, but we didn't have this even documented at all.

I have added some warnings and implications of using this setting, and recommending to do it only when being guided by Support.

This PR will also be aligned with #1418 for the Podman limitation.

@eedugon eedugon requested a review from a team as a code owner May 28, 2025 13:56
@eedugon eedugon requested review from frconil, jakommo and kunisen May 28, 2025 14:03
@eedugon eedugon self-assigned this May 28, 2025
@eedugon eedugon requested a review from pfcoperez May 28, 2025 14:13
@eedugon eedugon added the Team:Admin Issues owned by the Admin Docs Team label May 28, 2025
shainaraskas
shainaraskas approved these changes May 28, 2025
View reviewed changes
Copy link
Collaborator

@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.

tiny suggestions for you, otherwise good. assume we have validated that this is safe to share

shainaraskas
shainaraskas approved these changes May 28, 2025
View reviewed changes
Copy link
Collaborator

@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.

tiny suggestions for you, otherwise good. assume we have validated that this is safe to share

@eedugon eedugon added the ece Elastic Cloud Enterprise label May 29, 2025
@eedugon eedugon mentioned this pull request Jun 3, 2025
Open
@eedugon eedugon changed the title ece hard limit and podman introduction ece hard limit and podman limitation Jun 4, 2025
@eedugon eedugon changed the title ece hard limit and podman limitation ECE: Disabling CPU hard limit and podman limitation Jun 4, 2025
@eedugon eedugon requested a review from a team June 5, 2025 12:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shainaraskas shainaraskas shainaraskas approved these changes

@frconil frconil Awaiting requested review from frconil

@jakommo jakommo Awaiting requested review from jakommo

@kunisen kunisen Awaiting requested review from kunisen

@pfcoperez pfcoperez Awaiting requested review from pfcoperez

Assignees

@eedugon eedugon

Labels
ece Elastic Cloud Enterprise Team:Admin Issues owned by the Admin Docs Team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eedugon @shainaraskas