Skip to content

Conversation

AClarkie
Copy link
Contributor

@AClarkie AClarkie commented Sep 16, 2025

What this PR does / why we need it:

Adds a sensible default ephemeral-storage resource request to the nitro container to ensure it has enough local, node-attached storage to function properly.

Which issue this PR fixes

Fixes an issue where the nitro pod can be rescheduled due to exceeding it's requested ephemeral-storage when the node it's running on is low on ephemeral-storage

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • CI checks pass
  • Chart Version bumped

@AClarkie AClarkie requested a review from a team as a code owner September 16, 2025 13:35
@AClarkie AClarkie changed the title feat(nitro): add default resource.requests.ephemeral-storage feat(nitro): add a default resource.requests.ephemeral-storage Sep 16, 2025
Copy link

github-actions bot commented Sep 16, 2025

Chart Installation Test succeeded ✅

The chart installation test for commit ba04b9b99f5d1fca528e082dd72726fbe3b2cc64 has succeeded.

Changed charts: charts/nitro

View workflow run
View unprivileged test run

@AClarkie AClarkie merged commit c661c24 into main Sep 18, 2025
7 checks passed
@AClarkie AClarkie deleted the resources-update branch September 18, 2025 15:05
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.

2 participants