Skip to content

[autoscaler] Default min and max workers to 0 for the head node type. #17554

Closed
@DmitriGekhtman

Description

What is the problem?

Ray version and other system information (Python version, TensorFlow version, OS):

Most use-cases require a head node with a config distinct from the workers.
Having to explicitly state that there are no workers of head node type is confusing.
The default min and max workers for head node type should be 0. Head node type min and max workers should be removed from all cluster configs and from the helm chart.

Document the change and log a warning if necessary, as, strictly speaking, this change is backwards-incompatible.

Reproduction (REQUIRED)

Please provide a short code snippet (less than 50 lines if possible) that can be copy-pasted to reproduce the issue. The snippet should have no external library dependencies (i.e., use fake or mock data / environments):

If the code snippet cannot be run by itself, the issue will be closed with "needs-repro-script".

  • I have verified my script runs in a clean environment and reproduces the issue.
  • I have verified the issue also occurs with the latest wheels.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

P1Issue that should be fixed within a few weeksenhancementRequest for new feature and/or capabilityusability

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions