Skip to content

fix(configs): pin head unschedulable — ray-data (batch 3)#910

Merged
Aydin-ab merged 1 commit into
mainfrom
fix/batch-3-head-pin-deps
Jul 22, 2026
Merged

fix(configs): pin head unschedulable — ray-data (batch 3)#910
Aydin-ab merged 1 commit into
mainfrom
fix/batch-3-head-pin-deps

Conversation

@Aydin-ab

@Aydin-ab Aydin-ab commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Pins the head node unschedulable (resources: {CPU: 0}) in the aws + gce compute configs. These configs have worker groups or auto_select, so the head is a coordinator, not a work node: the launch-time conversion injects this when the field is absent, so setting it explicitly makes the repo config equal what ships to S3 (and rayapp test reproduces the real worker placement instead of co-locating on a schedulable head). Heads are CPU-only instances, so CPU: 0 alone is unschedulable. Config-only.

Templates: unstructured_data_ingestion, vhol-ray-data, audio-dataset-curation-llm-judge, e2e-timeseries-forecasting

Testing

Validated green via /test-template (Buildkite). The earlier CPU: 0, GPU: 0CPU: 0 normalization is a no-op on CPU-only heads (identical Ray resources), so not re-tested.

Part of the head-unschedulable sweep; a companion check (#924) enforces this policy repo-wide.

https://claude.ai/code/session_01QmLc4yWtmC3PX2NwWzPbzD

@Aydin-ab

Copy link
Copy Markdown
Contributor Author

/test-template unstructured_data_ingestion vhol-ray-data audio-dataset-curation-llm-judge

@Aydin-ab

Copy link
Copy Markdown
Contributor Author

/test-template e2e-timeseries-forecasting

These configs have worker groups or auto_select, so at launch the head is a coordinator, not a work node. Set head_node.resources CPU: 0 in aws.yaml and gce.yaml so the repo config equals what ships to S3 — the build-time compute-config conversion injects head resources when the field is absent, the silent drift behind the head-vs-worker framework skew. Heads are CPU-only instances, so CPU: 0 alone makes them unschedulable (no GPU to pin).

Templates: unstructured_data_ingestion, vhol-ray-data, audio-dataset-curation-llm-judge, e2e-timeseries-forecasting

https://claude.ai/code/session_01QmLc4yWtmC3PX2NwWzPbzD
Signed-off-by: Aydin Abiar <aydin@anyscale.com>
@Aydin-ab
Aydin-ab force-pushed the fix/batch-3-head-pin-deps branch from e95a959 to e7e03a3 Compare July 20, 2026 22:50
@Aydin-ab
Aydin-ab merged commit 23906ad into main Jul 22, 2026
4 checks passed
@Aydin-ab
Aydin-ab deleted the fix/batch-3-head-pin-deps branch July 22, 2026 22:15
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