Skip to content

[Fix] Add gcsReadyOptions to override wait-gcs-ready init container resources - #5072

Open
aabrower wants to merge 1 commit into
ray-project:masterfrom
aabrower:gcs_resources_option
Open

[Fix] Add gcsReadyOptions to override wait-gcs-ready init container resources#5072
aabrower wants to merge 1 commit into
ray-project:masterfrom
aabrower:gcs_resources_option

Conversation

@aabrower

@aabrower aabrower commented Aug 3, 2026

Copy link
Copy Markdown

The KubeRay-injected "wait-gcs-ready" init container has hard-coded resources (200m CPU / 256Mi memory). With these resources requests, we are seeing the init container repeatedly OOMKilled

Add an optional spec.gcsReadyOptions.resources field to RayCluster that overrides the injected init container's resource requests/limits while leaving the defaults unchanged

Why are these changes needed?

Related issue number

#2735

Labels

  • If this PR has user-facing changes that require documentation updates at release time, I have added the doc-updates-required label.
  • If this PR contains breaking changes, I have added the breaking-change label.

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • This PR is not tested :(

Manual test instructions

…er resources

The KubeRay-injected "wait-gcs-ready" init container has hard-coded
resources (200m CPU / 256Mi memory). For large Ray images (e.g. bazelized
containers), `ray health-check` needs more than 256Mi to run and the init
container is repeatedly OOMKilled, wedging worker pods in PodInitializing.

Add an optional `spec.gcsReadyOptions.resources` field to RayCluster that
overrides the injected init container's resource requests/limits, mirroring
the existing `autoscalerOptions.resources` pattern (default values are kept
when unset).
@aabrower aabrower changed the title [Feature] Add gcsReadyOptions to override wait-gcs-ready init container resources [Fix] Add gcsReadyOptions to override wait-gcs-ready init container resources Aug 3, 2026
@aabrower
aabrower marked this pull request as ready for review August 3, 2026 18:30
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